UI stylename handling (#9832)

* Change VUI class name from v-view to v-ui
* Use state.primaryStyleName so it can be changed
* Remove useless v-app-VaadinServiceSession from bootstrap root div

Change-Id: I67d3cb3efb5b4033416dcfcf7f39cbc71ac70cbc

diff --git a/WebContent/VAADIN/themes/base/common/common.scss b/WebContent/VAADIN/themes/base/common/common.scss
index 7fd4d2cc3291fe76e0a2a29e5c3ff96ecd527a55..b1e5ae167440bf445afa2bfcb45395fe1283039f 100644 (file)
--- a/WebContent/VAADIN/themes/base/common/common.scss
+++ b/WebContent/VAADIN/themes/base/common/common.scss
@@ -28,7 +28,7 @@ div.v-app-loading {
        width: 100%;
        height: 100%;
 }
-.v-view {
+.v-ui {
        height: 100%;
        width: 100%;
        /* avoid scrollbars with margins in root layout */
@@ -36,7 +36,7 @@ div.v-app-loading {
        position: relative;
 }
 /* Prevent margin collapse */
-.v-view.v-view-embedded {
+.v-ui.v-ui-embedded {
        margin-top: -1px;
        border-top: 1px solid transparent;
 }
@@ -54,15 +54,15 @@ div.v-app-loading {
                height: auto;
                min-height: 20cm;
        }
-       .v-view {
+       .v-ui {
                overflow: visible;
        }
        .v-gridlayout {
                overflow: visible !important;
        }
 }
-.v-view:active,
-.v-view:focus {
+.v-ui:active,
+.v-ui:focus {
        outline: none;
 }
 .v-app select,

diff --git a/WebContent/VAADIN/themes/chameleon/common/common.scss b/WebContent/VAADIN/themes/chameleon/common/common.scss
index a7878b38ec830d738450cfec074cfc4352d335db..10247032ac9e57af25813aecc67b568b52b9014c 100644 (file)
--- a/WebContent/VAADIN/themes/chameleon/common/common.scss
+++ b/WebContent/VAADIN/themes/chameleon/common/common.scss
@@ -15,7 +15,7 @@ $chameleon-line-height: 1.4;
                overflow: hidden;
                }
        
-       .v-view {
+       .v-ui {
                margin-top: 0;
                border-top: none;
                }

diff --git a/WebContent/VAADIN/themes/tests-tickets/styles.css b/WebContent/VAADIN/themes/tests-tickets/styles.css
index f088e0dd35990100625e1b8406912cd05f8930fc..fd4db4e6b5530c9f878fc8b426540177ee9fb111 100644 (file)
--- a/WebContent/VAADIN/themes/tests-tickets/styles.css
+++ b/WebContent/VAADIN/themes/tests-tickets/styles.css
@@ -320,7 +320,7 @@
 /*****************************************************************************/
 /* The reference screenshots all have a white background                                        */
 /*****************************************************************************/
-.v-view { 
+.v-ui { 
        background: #fff;
 }
 

diff --git a/client/src/com/vaadin/client/ui/VUI.java b/client/src/com/vaadin/client/ui/VUI.java
index 7f067c605da38395d9416aeb3ddd98a785be6456..a4d1df94235d089e745b40f33d3acec63008e223 100644 (file)
--- a/client/src/com/vaadin/client/ui/VUI.java
+++ b/client/src/com/vaadin/client/ui/VUI.java
@@ -51,8 +51,6 @@ public class VUI extends SimplePanel implements ResizeHandler,
         Window.ClosingHandler, ShortcutActionHandlerOwner, Focusable,
         HasResizeHandlers {
 
-    private static final String CLASSNAME = "v-view";
-
     private static int MONITOR_PARENT_TIMER_INTERVAL = 1000;
 
     /** For internal use only. May be removed or replaced in the future. */
@@ -157,8 +155,6 @@ public class VUI extends SimplePanel implements ResizeHandler,
 
     public VUI() {
         super();
-        setStyleName(CLASSNAME);
-
         // Allow focusing the view by using the focus() method, the view
         // should not be in the document focus flow
         getElement().setTabIndex(-1);

diff --git a/client/src/com/vaadin/client/ui/ui/UIConnector.java b/client/src/com/vaadin/client/ui/ui/UIConnector.java
index a0898b32813695d2c18545e3af59156b2b025bd6..a51cccf35502c27881b32a570dc98234927e77e5 100644 (file)
--- a/client/src/com/vaadin/client/ui/ui/UIConnector.java
+++ b/client/src/com/vaadin/client/ui/ui/UIConnector.java
@@ -355,7 +355,7 @@ public class UIConnector extends AbstractComponentContainerConnector implements
          * Must set absolute position if the child has relative height and
          * there's a chance of horizontal scrolling as some browsers will
          * otherwise not take the scrollbar into account when calculating the
-         * height. Assuming v-view does not have an undefined width for now, see
+         * height. Assuming v-ui does not have an undefined width for now, see
          * #8460.
          */
         if (child.isRelativeHeight() && !BrowserInfo.get().isIE9()) {

diff --git a/server/src/com/vaadin/server/BootstrapHandler.java b/server/src/com/vaadin/server/BootstrapHandler.java
index 15aaab313b047f79f5bbd37c752217d23fd3d5e4..df0ace78db6dd556775a83f2dacb5b9fffa81d77 100644 (file)
--- a/server/src/com/vaadin/server/BootstrapHandler.java
+++ b/server/src/com/vaadin/server/BootstrapHandler.java
@@ -317,16 +317,12 @@ public abstract class BootstrapHandler implements RequestHandler {
          *      .v-theme-<themeName, remove non-alphanum>
          */
 
-        String appClass = "v-app-"
-                + context.getSession().getClass().getSimpleName();
-
-        String classNames = "v-app " + appClass;
         List<Node> fragmentNodes = context.getBootstrapResponse()
                 .getFragmentNodes();
 
         Element mainDiv = new Element(Tag.valueOf("div"), "");
         mainDiv.attr("id", context.getAppId());
-        mainDiv.addClass(classNames);
+        mainDiv.addClass("v-app");
         if (style != null && style.length() != 0) {
             mainDiv.attr("style", style);
         }

diff --git a/shared/src/com/vaadin/shared/ui/ui/UIState.java b/shared/src/com/vaadin/shared/ui/ui/UIState.java
index c734ee5d0bcc41d1a631ff1771516680706d1af0..028218409f693b4b31f703b8060f9362de672185 100644 (file)
--- a/shared/src/com/vaadin/shared/ui/ui/UIState.java
+++ b/shared/src/com/vaadin/shared/ui/ui/UIState.java
@@ -18,4 +18,7 @@ package com.vaadin.shared.ui.ui;
 import com.vaadin.shared.ComponentState;
 
 public class UIState extends ComponentState {
+    {
+        primaryStyleName = "v-ui";
+    }
 }
\ No newline at end of file