fixes #5024, keyboard events no more blocked when table is focused

svn changeset:13232/svn branch:6.4

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
index 81dba3f1269571244af1b1c146bea58884d92f68..694c5aac0b70bbee9f70b821ee6389d8879fbd91 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
@@ -4699,6 +4699,7 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
         if (selectMode == SELECT_MODE_NONE && keycode == getNavigationDownKey()) {
             bodyContainer.setScrollPosition(bodyContainer.getScrollPosition()
                     + scrollingVelocity);
+            return true;
         } else if (keycode == getNavigationDownKey()) {
             if (selectMode == SELECT_MODE_MULTI && moveFocusDown()) {
                 selectFocusedRow(ctrl, shift);
@@ -4707,12 +4708,14 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                     && moveFocusDown()) {
                 selectFocusedRow(ctrl, shift);
             }
+            return true;
         }
 
         // Up navigation
         if (selectMode == SELECT_MODE_NONE && keycode == getNavigationUpKey()) {
             bodyContainer.setScrollPosition(bodyContainer.getScrollPosition()
                     - scrollingVelocity);
+            return true;
         } else if (keycode == getNavigationUpKey()) {
             if (selectMode == SELECT_MODE_MULTI && moveFocusUp()) {
                 selectFocusedRow(ctrl, shift);
@@ -4720,16 +4723,18 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                     && moveFocusUp()) {
                 selectFocusedRow(ctrl, shift);
             }
+            return true;
         }
 
-        // Left navigation
         if (keycode == getNavigationLeftKey()) {
+            // Left navigation
             bodyContainer.setHorizontalScrollPosition(bodyContainer
                     .getHorizontalScrollPosition()
                     - scrollingVelocity);
+            return true;
 
-            // Right navigation
         } else if (keycode == getNavigationRightKey()) {
+            // Right navigation
             bodyContainer.setHorizontalScrollPosition(bodyContainer
                     .getHorizontalScrollPosition()
                     + scrollingVelocity);
@@ -4750,6 +4755,7 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
             }
 
             sendSelectedRows();
+            return true;
         }
 
         // Page Down navigation
@@ -4771,6 +4777,7 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                     sendSelectedRows();
                 }
             }
+            return true;
         }
 
         // Page Up navigation
@@ -4792,6 +4799,7 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                     sendSelectedRows();
                 }
             }
+            return true;
         }
 
         // Goto start navigation
@@ -4808,6 +4816,7 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                 }
             }
             bodyContainer.setScrollPosition(0);
+            return true;
         }
 
         // Goto end navigation
@@ -4824,9 +4833,10 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                 }
             }
             bodyContainer.setScrollPosition(scrollBody.getOffsetHeight());
+            return true;
         }
 
-        return true;
+        return false;
     }
 
     /*