[MD settings] adjust button layout

ui/webui/resources/cr_elements/shared_style_css.html

 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/cr_elements/shared_vars_css.html">
 <link rel="import" href="hidden_style_css.html">
 
 <!-- Common styles for Material Design WebUI. Included directly in
   settings_shared_css.html. -->
 <dom-module id="cr-shared-style">
   <template>
     <style include="cr-hidden-style">
       :host-context([dir=rtl]) button[is='paper-icon-button-light'] {
@@ skipping 31 matching lines @@
         font-weight: 500;
         padding-left: 16px;
         padding-right: 16px;
       }
 
       [actionable] {
         @apply(--cr-actionable);
       }
 
       button[is='paper-icon-button-light'] {
-        @apply(--cr-paper-icon-button-margin);
+        -webkit-margin-end: calc(var(--cr-icon-ripple-padding) * -1);

[hcarmona, 2017/05/25 23:42:58]

This is used in 4 places, can we make a variable for
var(--cr-icon-ripple-padding) * -1 ?
possibly --cr-icon-ripple-margin

[dschuyler, 2017/05/26 00:07:03]

Done.

+        -webkit-margin-start: 16px;
         background-position: center;
         background-repeat: no-repeat;
         background-size: var(--cr-icon-size);
         flex-shrink: 0;
         height: var(--cr-icon-ripple-size);
         width: var(--cr-icon-ripple-size);
       }
 
+      .separator + button[is='paper-icon-button-light'] {
+        -webkit-margin-start: calc(var(--cr-icon-ripple-padding) * -1);
+      }
+
       button[is='paper-icon-button-light'].subpage-arrow {
         background-image: url(chrome://resources/images/arrow_right.svg);
       }
 
       button[is='paper-icon-button-light'].icon-cancel {
         background-image: url(chrome://resources/images/icon_cancel.svg);
       }
 
       button[is='paper-icon-button-light'].icon-cancel-toolbar {
         background-image:
             url(chrome://resources/images/icon_cancel_toolbar.svg);
       }
 
       button[is='paper-icon-button-light'].icon-external {
         background-image: url(chrome://resources/images/open_in_new.svg);
       }
 
       .subpage-arrow,
       .icon-external {
         display: none;
       }
 
+      paper-icon-button {

[hcarmona, 2017/05/25 23:42:58]

Can this

[dschuyler, 2017/05/26 00:07:03]

Done.

+        color: var(--paper-grey-600);
+        flex-shrink: 0;
+      }
+
+      paper-icon-button {

[hcarmona, 2017/05/25 23:42:58]

and this can be combined b/c they use the same selector?

[dschuyler, 2017/05/26 00:07:03]

Done.

+        -webkit-margin-end: calc(var(--cr-icon-ripple-padding) * -1);
+        -webkit-margin-start: 16px;
+      }
+
+      .separator + paper-icon-button {
+        -webkit-margin-start: calc(var(--cr-icon-ripple-padding) * -1);
+      }
+
       paper-icon-button.subpage-arrow {
         background-image: url(../images/arrow_right.svg);
       }
 
       [actionable] :-webkit-any(.subpage-arrow, .icon-external),
       [actionable]:-webkit-any(.subpage-arrow, .icon-external) {
         display: block;
       }
 
       [scrollable] {
@@ skipping 25 matching lines @@
       [selectable]:focus,
       [selectable] > :focus {
         @apply(--cr-selectable-focus);
       }
       [selectable] > * {
         @apply(--cr-actionable);
       }
     </style>
   </template>
 </dom-module>