MD User Manager: Supervised user checkbox is always displaying

chrome/browser/resources/md_user_manager/create_profile.html

Index: chrome/browser/resources/md_user_manager/create_profile.html
diff --git a/chrome/browser/resources/md_user_manager/create_profile.html b/chrome/browser/resources/md_user_manager/create_profile.html
index 19c76b97312d74503a51fdce3a31bf7beddeec11..a3ec1aadc8ddaa3a6298fcfcbf696f1ff6d4e8a2 100644
--- a/chrome/browser/resources/md_user_manager/create_profile.html
+++ b/chrome/browser/resources/md_user_manager/create_profile.html
@@ -60,9 +60,6 @@
 
       #nameInput,
       paper-dropdown-menu {
-        --paper-input-container: {
-          padding: 0;
-        };
         --paper-input-container-input: {
           color: inherit;
           font-size: inherit;
@@ -75,26 +72,6 @@
         width: 300px;
       }
 
-      paper-dropdown-menu {
-        margin-top: 12px;
-        width: 200px;
-      }
-
-      paper-menu {
-        --paper-menu: {
-          color: inherit;
-          padding: 0;
-        };
-        --paper-menu-selected-item: {
-          font-weight: normal;
-        }
-      }
-
-      paper-menu paper-item {
-        font-size: inherit;
-        min-height: 40px;
-      }
-
       #icons #wrapper {
         display: flex;
         flex-wrap: wrap;
@@ -117,28 +94,49 @@
         outline: 1px dotted #666;
       }
 
-      #supervised-user-container {
-        margin-top: 24px;
-      }
-
-      #supervised-user-container paper-checkbox {
+      paper-checkbox {
         --paper-checkbox-label-spacing: 16px;
         --paper-checkbox-size: 20px;
+        --primary-text-color: var(--paper-grey-800);
+        margin-top: 24px;
       }
 
-      #supervised-user-container #learn-more-accounts {
-        -webkit-margin-start: 36px;
+      #supervised-user-container {
+        -webkit-padding-start: 36px;
       }
 
       #supervised-user-container #learn-more {
-        display: block;
         line-height: 20px;
       }
 
+      #supervised-user-container paper-dropdown-menu {
+        margin-top: 4px;
+        width: 200px;
+      }
+
+      #supervised-user-container paper-menu {
+        --paper-menu: {
+          color: inherit;
+          padding: 0;
+        };
+        --paper-menu-selected-item: {
+          font-weight: normal;
+        };
+      }
+
+      #supervised-user-container paper-menu paper-item {
+        font-size: inherit;
+        min-height: 40px;
+      }
+
       #supervised-user-container #import-user {
         -webkit-margin-start: 16px;
       }
 
+      #supervised-user-container #sign-in-howto-message {
+        margin-top: 12px;
+      }
+
       #actions {
         bottom: 16px;
         display: flex;
@@ -199,39 +197,41 @@
         </template>
         </div>
       </div>
-      <template is="dom-if" if="[[isSignedIn_(signedInUsers_)]]">
-        <div id="supervised-user-container">
-          <paper-checkbox checked="{{isSupervised_}}">
-            [[i18n('manageProfilesSupervisedSignedInLabel')]]
-          </paper-checkbox>
-          <div id="learn-more-accounts">
-            <a id="learn-more" is="action-link" on-tap="onLearnMoreTap_">
-              [[i18n('learnMore')]]
-            </a>
-            <template is="dom-if" if="[[isSupervised_]]">
-              <paper-dropdown-menu no-label-float>
-                <paper-menu class="dropdown-content"
-                    selected="{{signedInUserIndex_}}"
-                    attr-for-selected="data-user-index">
-                  <paper-item data-user-index$="[[NO_USER_SELECTED]]">
-                    [[i18n('selectAnAccount')]]
+      <paper-checkbox checked="{{isSupervised_}}">
+        [[i18n('manageProfilesSupervisedSignedInLabel')]]
+      </paper-checkbox>
+      <div id="supervised-user-container">
+        <div id="learn-more" >
+          <a is="action-link" on-tap="onLearnMoreTap_">[[i18n('learnMore')]]</a>
+        <div>
+        <template is="dom-if" if="[[isSupervised_]]">
+          <template is="dom-if" if="[[isSignedIn_(signedInUsers_)]]">
+            <paper-dropdown-menu no-label-float>
+              <paper-menu class="dropdown-content"

[dpapad, 2016/05/03 17:30:26]

Could this paper-menu be swapped with the lighter weight paper-listbox? See
related discussion at crbug.com/601214

[Moe, 2016/05/03 20:30:10]

yep. Done!

+                  selected="{{signedInUserIndex_}}"
+                  attr-for-selected="data-user-index">
+                <paper-item data-user-index$="[[NO_USER_SELECTED]]">
+                  [[i18n('selectAnAccount')]]
+                </paper-item>
+                <template is="dom-repeat" items="[[signedInUsers_]]">
+                  <paper-item data-user-index$="[[index]]">
+                    [[item.username]]
                   </paper-item>
-                  <template is="dom-repeat" items="[[signedInUsers_]]">
-                    <paper-item data-user-index$="[[index]]">
-                      [[item.username]]
-                    </paper-item>
-                  </template>
-                </paper-menu>
-              </paper-dropdown-menu>
-              <a id="import-user" is="action-link" on-tap="onImportUserTap_"
-                  hidden="[[isImportUserLinkHidden_(createInProgress_,
-                                                    signedInUserIndex_)]]">
-                [[i18n('importExistingSupervisedUserLink')]]
-              </a>
-            </template>
+                </template>
+              </paper-menu>
+            </paper-dropdown-menu>
+            <a id="import-user" is="action-link" on-tap="onImportUserTap_"
+                hidden="[[isImportUserLinkHidden_(createInProgress_,
+                                                  signedInUserIndex_)]]">
+              [[i18n('importExistingSupervisedUserLink')]]
+            </a>
+          </template>
+          <div id="sign-in-howto-message"
+              hidden="[[isSignedIn_(signedInUsers_)]]"
+              inner-h-t-m-l="[[i18n('noSignedInUserMessage')]]">
           </div>
-        </div>
-      </template>
+        </template>
+      </div>
       <div id="actions">
         <paper-spinner active="[[createInProgress_]]"></paper-spinner>
         <paper-button id="cancel" on-tap="onCancelTap_" i18n-content="cancel">