fixes #1834. Tested only a bit with FF & SA.

svn changeset:4948/svn branch:trunk

diff --git a/src/com/itmill/toolkit/tests/tickets/Ticket1834PanelScrolling.java b/src/com/itmill/toolkit/tests/tickets/Ticket1834PanelScrolling.java
index 5dabc86e8304ca1b43a7e5daedbf60749f95b20f..312c8ca5cbce8e018bcb22811cea66878bda1913 100644 (file)
--- a/src/com/itmill/toolkit/tests/tickets/Ticket1834PanelScrolling.java
+++ b/src/com/itmill/toolkit/tests/tickets/Ticket1834PanelScrolling.java
@@ -68,6 +68,7 @@ public class Ticket1834PanelScrolling extends com.itmill.toolkit.Application {
         main.addComponent(b);\r
 \r
         p = new Panel("TestPanel");\r
+        p.setScrollable(true);\r
 \r
         for (int i = 0; i < ROWS; i++) {\r
             p\r

diff --git a/src/com/itmill/toolkit/ui/Panel.java b/src/com/itmill/toolkit/ui/Panel.java
index 162e59a3fe14faa95fb18d8ca09796447960ec1a..6359687c3ccf47f8e445e7a8490dce716afb690e 100644 (file)
--- a/src/com/itmill/toolkit/ui/Panel.java
+++ b/src/com/itmill/toolkit/ui/Panel.java
@@ -176,8 +176,8 @@ public class Panel extends AbstractComponentContainer implements Scrollable,
         }
 
         if (isScrollable()) {
-            target.addVariable(this, "scrollleft", getScrollLeft());
-            target.addVariable(this, "scrolltop", getScrollTop());
+            target.addVariable(this, "scrollLeft", getScrollLeft());
+            target.addVariable(this, "scrollTop", getScrollTop());
         }
 
         if (actionHandlers != null && !actionHandlers.isEmpty()) {
@@ -292,8 +292,8 @@ public class Panel extends AbstractComponentContainer implements Scrollable,
         }
 
         // Scrolling
-        final Integer newScrollX = (Integer) variables.get("scrollleft");
-        final Integer newScrollY = (Integer) variables.get("scrolltop");
+        final Integer newScrollX = (Integer) variables.get("scrollLeft");
+        final Integer newScrollY = (Integer) variables.get("scrollTop");
         if (newScrollX != null && newScrollX.intValue() != getScrollLeft()) {
             // set internally, not to fire request repaint
             scrollOffsetX = newScrollX.intValue();