[MD settings] change button padding and placement

chrome/browser/resources/settings/settings_shared_css.html

 <!-- Common styles for Material Design settings. -->
 <dom-module id="settings-shared">
   <template>
     <style>
       :root {
         --checkbox-margin-start: 2px;
         --checkbox-size: 16px;
         --checkbox-spacing: 18px;
         --iron-icon-fill-color: var(--paper-grey-600);
         --iron-icon-height: 20px;
@@ skipping 41 matching lines @@
         margin: 0;
         min-height: var(--settings-row-min-height);
       }
 
       iron-icon[icon='cr:check'],
       iron-icon[icon='settings:done'] {
        --iron-icon-fill-color: var(--google-green-500);
       }
 
       paper-button {
+        @apply(--layout-center);
+        display: flex;
         margin: 0;
+        min-height: 32px;
+        padding: 0 12px;
       }
 
       paper-button[toggles][active] {
         background-color: var(--paper-grey-300);
       }
 
       span ~ a {
         -webkit-margin-start: 4px;
       }
 
@@ skipping 10 matching lines @@
         /* TODO(dbeam): check with bettes@ on this one, but I'm pretty sure we
          * shouldn't be showing much :hover stuff on Polymer pages. */
         text-decoration: none;
       }
 
       paper-button.primary-button {
         --paper-button: {
           font-weight: 500;
           text-align: start;
         };
-        margin: 0 -0.57em;  /* Offsets default paper-button padding. */
       }
 
       paper-button.secondary-button {
         --paper-button: {
           color: var(--paper-grey-600);
           text-decoration: none;
           font-weight: 500;
         };
       }
 
@@ skipping 25 matching lines @@
       paper-radio-group {
         width: 100%;
       }
 
       .text-elide {
         overflow: hidden;
         text-overflow: ellipsis;
         white-space: nowrap;
       }
 
-      .button-row {
-        display: flex;
-        margin-top: 25px;
-      }
-
       .button-strip {
         text-align: end;
       }
 
       .action-button {
         background: var(--google-blue-500);
         color: white;
       }
 
       .action-button[disabled] {
@@ skipping 63 matching lines @@
       }
 
       .settings-box {
         @apply(--layout-center);
         border-top: var(--settings-separator-line);
         display: flex;
         min-height: var(--settings-row-min-height);
         padding: 0 20px;
       }
 
+      /* The button-strip inside of a settings-box is moved over a small amount
+       * to line up the text of the button label with the 20px settings-box
+       * margin. See also: .secondary-action for buttons at the end of the row.
+       */
+      .settings-box.button-strip {
+        /* Plus 12px for the button equals 20px total. */
+        -webkit-padding-start: 8px;
+      }
+
       .settings-box.first,
       .settings-box.continuation {
         border-top: none;
       }
 
       .settings-box.block {
         display: block;
       }
 
       .settings-box.two-line {
@@ skipping 17 matching lines @@
 
       /* The secondary-action wraps a clickable sub-area of a .settings-box.
        * An example is the |sign out| button on the People settings.
        * Here is an example with and without a secondary action box:
        *
        *  +-------------------------------------------------------+
        *  | Main action area .settings-box    | .secondary-action |
        *  +-------------------------------------------------------+
        *  | Another setting-box without a secondary-action        |
        *  +-------------------------------------------------------+
+       *
+       * See also: .button-strip for buttons at the start of the row.
        */
       .settings-box .secondary-action {
+        @apply(--layout-center);
         -webkit-border-start: var(--settings-separator-line);
-        -webkit-padding-start: 20px;
+        -webkit-padding-start: 8px;
+        display: flex;
+        left: 12px;  /* Equal to the paper-button padding. */

[Dan Beam, 2016/05/18 00:32:47]

wont work in RTL

[dschuyler, 2016/05/18 20:29:04]

Done.

+        min-height: 32px;
+        position: relative;
       }
 
       .settings-box paper-item iron-icon {
         /* Same padding as paper-icon-button. */
         padding: 8px;
       }
 
       .vertical-list > *:not(:first-of-type) {
         border-top: var(--settings-separator-line);
       }
@@ skipping 14 matching lines @@
 
       .favicon-image {
         background-repeat: no-repeat;
         background-size: contain;
         height: 16px;
         width: 16px;
       }
     </style>
   </template>
 </dom-module>