Moved base theme constants (v-widget and v-layout) from BaseTheme to

StyleConstants

client/src/com/vaadin/client/StyleConstants.java

@@ -1,4 +1,4 @@
-/* 
+/*
  * Copyright 2011 Vaadin Ltd.
  * 
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
@@ -26,4 +26,13 @@ public class StyleConstants {
     public static final String VERTICAL_SPACING = "vspacing";
     public static final String HORIZONTAL_SPACING = "hspacing";
 
+    /**
+     * Added to all UI components
+     */
+    public static final String UI_WIDGET = "v-widget";
+
+    /**
+     * Added to all layouts to denote they are layouts
+     */
+    public static final String UI_LAYOUT = "v-layout";
 }

client/src/com/vaadin/client/ui/AbstractComponentConnector.java

@@ -29,6 +29,7 @@ import com.vaadin.client.ComponentContainerConnector;
 import com.vaadin.client.ConnectorMap;
 import com.vaadin.client.LayoutManager;
 import com.vaadin.client.ServerConnector;
+import com.vaadin.client.StyleConstants;
 import com.vaadin.client.TooltipInfo;
 import com.vaadin.client.UIDL;
 import com.vaadin.client.Util;
@@ -44,7 +45,6 @@ import com.vaadin.shared.ComponentState;
 import com.vaadin.shared.Connector;
 import com.vaadin.shared.ui.ComponentStateUtil;
 import com.vaadin.shared.ui.TabIndexState;
-import com.vaadin.ui.themes.BaseTheme;
 
 public abstract class AbstractComponentConnector extends AbstractConnector
         implements ComponentConnector {
@@ -73,7 +73,7 @@ public abstract class AbstractComponentConnector extends AbstractConnector
         getConnection().getVTooltip().connectHandlersToWidget(getWidget());
 
         // Set the core 'v' style name for the widget
-        getWidget().setStyleName(BaseTheme.UI_WIDGET, true);
+        getWidget().setStyleName(StyleConstants.UI_WIDGET, true);
     }
 
     /**

client/src/com/vaadin/client/ui/absolutelayout/VAbsoluteLayout.java

@@ -24,8 +24,8 @@ import com.google.gwt.user.client.ui.ComplexPanel;
 import com.google.gwt.user.client.ui.SimplePanel;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.client.ApplicationConnection;
+import com.vaadin.client.StyleConstants;
 import com.vaadin.client.VCaption;
-import com.vaadin.ui.themes.BaseTheme;
 
 public class VAbsoluteLayout extends ComplexPanel {
 
@@ -45,7 +45,7 @@ public class VAbsoluteLayout extends ComplexPanel {
 
     public VAbsoluteLayout() {
         setElement(Document.get().createDivElement());
-        setStyleName(BaseTheme.UI_LAYOUT);
+        setStyleName(StyleConstants.UI_LAYOUT);
         addStyleName(CLASSNAME);
         marginElement = Document.get().createDivElement();
         canvas.getStyle().setProperty("position", "relative");

client/src/com/vaadin/client/ui/csslayout/VCssLayout.java

@@ -18,7 +18,7 @@ package com.vaadin.client.ui.csslayout;
 
 import com.google.gwt.user.client.ui.FlowPanel;
 import com.google.gwt.user.client.ui.Widget;
-import com.vaadin.ui.themes.BaseTheme;
+import com.vaadin.client.StyleConstants;
 
 /**
  * VCCSlayout is a layout which supports configuring it's children with CSS
@@ -33,7 +33,7 @@ public class VCssLayout extends FlowPanel {
      */
     public VCssLayout() {
         super();
-        setStyleName(BaseTheme.UI_LAYOUT);
+        setStyleName(StyleConstants.UI_LAYOUT);
         addStyleName(CLASSNAME);
     }
 

client/src/com/vaadin/client/ui/customlayout/VCustomLayout.java

@@ -29,10 +29,10 @@ import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.client.ApplicationConnection;
 import com.vaadin.client.BrowserInfo;
 import com.vaadin.client.ComponentConnector;
+import com.vaadin.client.StyleConstants;
 import com.vaadin.client.Util;
 import com.vaadin.client.VCaption;
 import com.vaadin.client.VCaptionWrapper;
-import com.vaadin.ui.themes.BaseTheme;
 
 /**
  * Custom Layout implements complex layout defined with HTML template.
@@ -83,7 +83,7 @@ public class VCustomLayout extends ComplexPanel {
             DOM.setStyleAttribute(getElement(), "position", "relative");
         }
 
-        setStyleName(BaseTheme.UI_LAYOUT);
+        setStyleName(StyleConstants.UI_LAYOUT);
         addStyleName(CLASSNAME);
     }
 

client/src/com/vaadin/client/ui/formlayout/VFormLayout.java

@@ -40,7 +40,6 @@ import com.vaadin.shared.ComponentConstants;
 import com.vaadin.shared.ComponentState;
 import com.vaadin.shared.ui.ComponentStateUtil;
 import com.vaadin.shared.ui.MarginInfo;
-import com.vaadin.ui.themes.BaseTheme;
 
 /**
  * Two col Layout that places caption on left col and field on right col
@@ -53,7 +52,7 @@ public class VFormLayout extends SimplePanel {
 
     public VFormLayout() {
         super();
-        setStyleName(BaseTheme.UI_LAYOUT);
+        setStyleName(StyleConstants.UI_LAYOUT);
         addStyleName(CLASSNAME);
         table = new VFormLayoutTable();
         setWidget(table);

client/src/com/vaadin/client/ui/gridlayout/VGridLayout.java

@@ -32,6 +32,7 @@ import com.vaadin.client.ApplicationConnection;
 import com.vaadin.client.ComponentConnector;
 import com.vaadin.client.ConnectorMap;
 import com.vaadin.client.LayoutManager;
+import com.vaadin.client.StyleConstants;
 import com.vaadin.client.UIDL;
 import com.vaadin.client.Util;
 import com.vaadin.client.VCaption;
@@ -39,7 +40,6 @@ import com.vaadin.client.ui.layout.ComponentConnectorLayoutSlot;
 import com.vaadin.client.ui.layout.VLayoutSlot;
 import com.vaadin.shared.ui.AlignmentInfo;
 import com.vaadin.shared.ui.MarginInfo;
-import com.vaadin.ui.themes.BaseTheme;
 
 public class VGridLayout extends ComplexPanel {
 
@@ -71,7 +71,7 @@ public class VGridLayout extends ComplexPanel {
         spacingStyle.setPosition(Position.ABSOLUTE);
         getElement().appendChild(spacingMeasureElement);
 
-        setStyleName(BaseTheme.UI_LAYOUT);
+        setStyleName(StyleConstants.UI_LAYOUT);
         addStyleName(CLASSNAME);
     }
 

client/src/com/vaadin/client/ui/orderedlayout/VOrderedLayout.java

@@ -32,10 +32,10 @@ import com.google.gwt.user.client.ui.SimplePanel;
 import com.google.gwt.user.client.ui.UIObject;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.client.LayoutManager;
+import com.vaadin.client.StyleConstants;
 import com.vaadin.client.Util;
 import com.vaadin.shared.ui.AlignmentInfo;
 import com.vaadin.shared.ui.MarginInfo;
-import com.vaadin.ui.themes.BaseTheme;
 
 /**
  * Base class for ordered layouts
@@ -60,7 +60,7 @@ public class VOrderedLayout extends FlowPanel {
      * Constructor
      */
     public VOrderedLayout() {
-        setStyleName(BaseTheme.UI_LAYOUT);
+        setStyleName(StyleConstants.UI_LAYOUT);
         setVertical(true);
     }
 
@@ -387,7 +387,7 @@ public class VOrderedLayout extends FlowPanel {
                 if (caption == null) {
                     caption = DOM.createDiv();
                     captionWrap = DOM.createDiv();
-                    captionWrap.addClassName(BaseTheme.UI_WIDGET);
+                    captionWrap.addClassName(StyleConstants.UI_WIDGET);
                     captionWrap.addClassName("v-has-caption");
                     getElement().appendChild(captionWrap);
                     captionWrap.appendChild(getWidget().getElement());

client/src/com/vaadin/client/ui/splitpanel/VAbstractSplitPanel.java

@@ -41,13 +41,13 @@ import com.vaadin.client.BrowserInfo;
 import com.vaadin.client.ComponentConnector;
 import com.vaadin.client.ConnectorMap;
 import com.vaadin.client.LayoutManager;
+import com.vaadin.client.StyleConstants;
 import com.vaadin.client.Util;
 import com.vaadin.client.VConsole;
 import com.vaadin.client.ui.TouchScrollDelegate;
 import com.vaadin.client.ui.TouchScrollDelegate.TouchScrollHandler;
 import com.vaadin.client.ui.VOverlay;
 import com.vaadin.client.ui.splitpanel.VAbstractSplitPanel.SplitterMoveHandler.SplitterMoveEvent;
-import com.vaadin.ui.themes.BaseTheme;
 
 public class VAbstractSplitPanel extends ComplexPanel {
 
@@ -118,7 +118,7 @@ public class VAbstractSplitPanel extends ComplexPanel {
 
     public VAbstractSplitPanel(int orientation) {
         setElement(DOM.createDiv());
-        setStyleName(BaseTheme.UI_LAYOUT);
+        setStyleName(StyleConstants.UI_LAYOUT);
         switch (orientation) {
         case ORIENTATION_HORIZONTAL:
             addStyleName(CLASSNAME + "-horizontal");

server/src/com/vaadin/ui/themes/BaseTheme.java

@@ -55,15 +55,4 @@ public class BaseTheme {
      * overflow.
      */
     public static final String CLIP = "v-clip";
-
-    /**
-     * Added to all UI components
-     */
-    public static final String UI_WIDGET = "v-widget";
-
-    /**
-     * Added to all layouts to denote they are layouts
-     */
-    public static final String UI_LAYOUT = "v-layout";
-
 }