commit f6557f3109afefb253f4b1cc0dcd77a0cb74a1fa
author jlabanca@google.com <jlabanca@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Fri Jan 21 13:38:10 2011 +0000
committer jlabanca@google.com <jlabanca@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Fri Jan 21 13:38:10 2011 +0000
tree 1ed5192b6675fb0e6a3558b3c78aa886015de0af
parent ced4cde9e9584f84c68517e48aebc175ab808b56

Fixing the overflow-x and overflow-y properties in Style to use camelCase instead of hyphenated notation.

Review at http://gwt-code-reviews.appspot.com/1315801

Review by: jgw@google.com

git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@9587 8db76d5a-ed1c-0410-87a9-c151d255dfc7

user/src/com/google/gwt/dom/client/Style.java

diff --git a/user/src/com/google/gwt/dom/client/Style.java b/user/src/com/google/gwt/dom/client/Style.java
index 30bce70..11a1261 100644
--- a/user/src/com/google/gwt/dom/client/Style.java
+++ b/user/src/com/google/gwt/dom/client/Style.java
@@ -682,8 +682,8 @@
   private static final String STYLE_PADDING_BOTTOM = "paddingBottom";
   private static final String STYLE_PADDING = "padding";
   private static final String STYLE_OVERFLOW = "overflow";
-  private static final String STYLE_OVERFLOW_X = "overflow-x";
-  private static final String STYLE_OVERFLOW_Y = "overflow-y";
+  private static final String STYLE_OVERFLOW_X = "overflowX";
+  private static final String STYLE_OVERFLOW_Y = "overflowY";
   private static final String STYLE_OPACITY = "opacity";
   private static final String STYLE_MARGIN_TOP = "marginTop";
   private static final String STYLE_MARGIN_RIGHT = "marginRight";

user/test/com/google/gwt/dom/client/StyleTest.java

diff --git a/user/test/com/google/gwt/dom/client/StyleTest.java b/user/test/com/google/gwt/dom/client/StyleTest.java
index a3d64c0..686690a 100644
--- a/user/test/com/google/gwt/dom/client/StyleTest.java
+++ b/user/test/com/google/gwt/dom/client/StyleTest.java
@@ -186,6 +186,34 @@
     assertEquals(Overflow.AUTO, style.getOverflow());
   }
 
+  public void testOverflowX() {
+    DivElement div = Document.get().createDivElement();
+    Style style = div.getStyle();
+
+    style.setOverflowX(Overflow.VISIBLE);
+    assertEquals(Overflow.VISIBLE, style.getOverflowX());
+    style.setOverflowX(Overflow.HIDDEN);
+    assertEquals(Overflow.HIDDEN, style.getOverflowX());
+    style.setOverflowX(Overflow.SCROLL);
+    assertEquals(Overflow.SCROLL, style.getOverflowX());
+    style.setOverflowX(Overflow.AUTO);
+    assertEquals(Overflow.AUTO, style.getOverflowX());
+  }
+
+  public void testOverflowY() {
+    DivElement div = Document.get().createDivElement();
+    Style style = div.getStyle();
+
+    style.setOverflowY(Overflow.VISIBLE);
+    assertEquals(Overflow.VISIBLE, style.getOverflowY());
+    style.setOverflowY(Overflow.HIDDEN);
+    assertEquals(Overflow.HIDDEN, style.getOverflowY());
+    style.setOverflowY(Overflow.SCROLL);
+    assertEquals(Overflow.SCROLL, style.getOverflowY());
+    style.setOverflowY(Overflow.AUTO);
+    assertEquals(Overflow.AUTO, style.getOverflowY());
+  }
+
   public void testPosition() {
     DivElement div = Document.get().createDivElement();
     Style style = div.getStyle();