From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Tue, 24 Jun 2008 13:27:03 +0000 (+0000)
Subject: fixes #1834. Tested only a bit with FF & SA.
X-Git-Tag: 6.7.0.beta1~4557
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=095c8b6bbe04753595845a09ea80fda4f5c66490;p=vaadin-framework.git

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 5dabc86e83..312c8ca5cb 100644
--- 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);
 
         p = new Panel("TestPanel");
+        p.setScrollable(true);
 
         for (int i = 0; i < ROWS; i++) {
             p
diff --git a/src/com/itmill/toolkit/ui/Panel.java b/src/com/itmill/toolkit/ui/Panel.java
index 162e59a3fe..6359687c3c 100644
--- 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();