Fix various small issues discovered while reviewing API

Change-Id: I196e490d5c5ae77ba895e0fca1b0d9160b6a7855

5 files changed, 6 insertions, 8 deletions

diff --git a/client/src/com/vaadin/client/widget/grid/AutoScroller.java b/client/src/com/vaadin/client/widget/grid/AutoScroller.java
index f7c80df623..773dc012c8 100644
--- a/client/src/com/vaadin/client/widget/grid/AutoScroller.java
+++ b/client/src/com/vaadin/client/widget/grid/AutoScroller.java
@@ -75,7 +75,7 @@ public class AutoScroller {
 
     public enum ScrollAxis {
         VERTICAL, HORIZONTAL
-    };
+    }
 
     /** The maximum number of pixels per second to autoscroll. */
     private static final int SCROLL_TOP_SPEED_PX_SEC = 500;
@@ -508,10 +508,10 @@ public class AutoScroller {
      * Defaults to 100px.
      * 
      * @param px
-     *            the height/width for the auto scroll area depending on
+     *            the pixel height/width for the auto scroll area depending on
      *            direction
      */
-    public void setScrollAreaPX(int px) {
+    public void setScrollArea(int px) {
         scrollAreaPX = px;
     }
 
@@ -522,7 +522,7 @@ public class AutoScroller {
      * 
      * @return size in pixels
      */
-    public int getScrollAreaPX() {
+    public int getScrollArea() {
         return scrollAreaPX;
     }
 
diff --git a/client/src/com/vaadin/client/widget/grid/DetailsGenerator.java b/client/src/com/vaadin/client/widget/grid/DetailsGenerator.java
index 309e3f1ea3..103bf96291 100644
--- a/client/src/com/vaadin/client/widget/grid/DetailsGenerator.java
+++ b/client/src/com/vaadin/client/widget/grid/DetailsGenerator.java
@@ -42,6 +42,5 @@ public interface DetailsGenerator {
      * @return the details for the given row, or <code>null</code> to leave the
      *         details empty.
      */
-    // TODO: provide a row object instead of index (maybe, needs discussion?)
     Widget getDetails(int rowIndex);
 }
diff --git a/client/src/com/vaadin/client/widget/grid/events/ColumnReorderHandler.java b/client/src/com/vaadin/client/widget/grid/events/ColumnReorderHandler.java
index e4f258088f..4733ed8bc0 100644
--- a/client/src/com/vaadin/client/widget/grid/events/ColumnReorderHandler.java
+++ b/client/src/com/vaadin/client/widget/grid/events/ColumnReorderHandler.java
@@ -33,7 +33,6 @@ public interface ColumnReorderHandler<T> extends EventHandler {
      * A column reorder event, fired by Grid when the columns of the Grid have
      * been reordered.
      * 
-     * @since
      * @param event
      *            column reorder event
      */
diff --git a/client/src/com/vaadin/client/widgets/Escalator.java b/client/src/com/vaadin/client/widgets/Escalator.java
index 0d65dda611..75b797eb1f 100644
--- a/client/src/com/vaadin/client/widgets/Escalator.java
+++ b/client/src/com/vaadin/client/widgets/Escalator.java
@@ -4494,7 +4494,7 @@ public class Escalator extends Widget implements RequiresResize,
      * The meaning of each value may differ depending on the context it is being
      * used in. Check that particular method's JavaDoc.
      */
-    public enum SpacerInclusionStrategy {
+    private enum SpacerInclusionStrategy {
         /** A representation of "the entire spacer". */
         COMPLETE,
 
diff --git a/client/src/com/vaadin/client/widgets/Grid.java b/client/src/com/vaadin/client/widgets/Grid.java
index e9288a7ece..9821591b91 100644
--- a/client/src/com/vaadin/client/widgets/Grid.java
+++ b/client/src/com/vaadin/client/widgets/Grid.java
@@ -3385,7 +3385,7 @@ public class Grid<T> extends ResizeComposite implements
             dragElement.addClassName("dragged-column-header");
 
             // start the auto scroll handler
-            autoScroller.setScrollAreaPX(60);
+            autoScroller.setScrollArea(60);
             autoScroller.start(startingEvent, ScrollAxis.HORIZONTAL,
                     autoScrollerCallback);
             return true;