MD Settings: fix toggle button spacing

chrome/browser/resources/settings/controls/settings_toggle_button.html

 <link rel="import" href="chrome://resources/cr_elements/policy/cr_policy_pref_indicator.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-toggle-button/paper-toggle-button.html">
 <link rel="import" href="settings_boolean_control_behavior.html">
 <link rel="import" href="../settings_shared_css.html">
 
 <dom-module id="settings-toggle-button">
   <template>
     <style include="settings-shared">
       :host([elide-label]),
@@ skipping 10 matching lines @@
         align-items: center;
         display: flex;
         min-height: var(--settings-row-two-line-min-height);
         width: 100%;
       }
 
       #outerRow[noSubLabel] {
         min-height: var(--settings-row-min-height);
       }
 
-      paper-toggle-button,
+      #labelWrapper,
+      ::content .more-actions,

[scottchen, 2017/03/10 19:05:46]

that's cool, didn't know about ::content.

[Dan Beam, 2017/03/10 19:14:10]

it's probably going away soon, don't get used to it ;)

       cr-policy-pref-indicator {
-        -webkit-margin-start: var(--checkbox-spacing);
-      }
-
-      ::content .more-actions {
-        -webkit-margin-end: 10px;
+        -webkit-margin-end: var(--checkbox-spacing);
       }
     </style>
     <div id="outerRow" noSubLabel$="[[!subLabel]]">
-      <div class="flex" on-tap="onLabelWrapperTap_"
-          actionable$="[[!controlDisabled_(disabled, pref.*)]]">
+      <div class="flex" id="labelWrapper" on-tap="onLabelWrapperTap_"
+          actionable$="[[!controlDisabled_(disabled, pref.*)]]"
+          hidden="[[!label]]">

[scottchen, 2017/03/10 19:05:46]

would dom-if be better here?

[Dan Beam, 2017/03/10 19:14:10]

for a couple <div>s, probably not

         <div id="label" class="label">[[label]]</div>
         <div class="secondary label">[[subLabel]]</div>
       </div>
       <content select=".more-actions"></content>
       <template is="dom-if" if="[[hasPrefPolicyIndicator(pref.*)]]">
         <cr-policy-pref-indicator pref="[[pref]]" icon-aria-label="[[label]]">
         </cr-policy-pref-indicator>
       </template>
       <paper-toggle-button id="control" checked="{{checked}}"
           on-change="notifyChangedByUserInteraction" aria-labelledby="label"
           disabled="[[controlDisabled_(disabled, pref)]]">
       </paper-toggle-button>
     </div>
   </template>
   <script src="settings_toggle_button.js"></script>
 </dom-module>