[MD settings] icons for clickable rows

chrome/browser/resources/settings/appearance_page/appearance_page.html

Index: chrome/browser/resources/settings/appearance_page/appearance_page.html
diff --git a/chrome/browser/resources/settings/appearance_page/appearance_page.html b/chrome/browser/resources/settings/appearance_page/appearance_page.html
index b7bc08ff803d9d74428eb24bccccbc1ed1830dbb..82151c59c396008749e6db9e7ffee50bc24b1459 100644
--- a/chrome/browser/resources/settings/appearance_page/appearance_page.html
+++ b/chrome/browser/resources/settings/appearance_page/appearance_page.html
@@ -4,6 +4,7 @@
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/neon-animation/neon-animatable.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button-light.html">
 <link rel="import" href="/appearance_page/appearance_browser_proxy.html">
 <link rel="import" href="/controls/controlled_radio_button.html">
 <link rel="import" href="/controls/settings_dropdown_menu.html">
@@ -17,13 +18,8 @@
 <link rel="import" href="appearance_fonts_page.html">
 
 <dom-module id="settings-appearance-page">
-  <link rel="import" type="css" href="chrome://resources/css/widgets.css">
   <template>
-    <style include="settings-shared">
-      .settings-box iron-icon {
-        -webkit-margin-end: var(--iron-icon-spacing);

[Dan Beam, 2016/09/28 21:56:27]

why are you removing this?

[dschuyler, 2016/09/29 00:40:57]

It's not needed. Is it needed and I'm missing
where that is?

[Dan Beam, 2016/09/29 03:19:34]

so you're saying this rule didn't have any affect?  it looks like it's matching
<iron-icon> tags inside of class="settings-box" in this file

[dschuyler, 2016/09/29 21:22:50]

I see where that would look wrong. There was an earlier
mistake in that a .start class was used after the iron-icon
rather than a .middle class. The .middle has space before it.
So the icon doesn't need space after it. This change removes
a minor local hack of having a one-off custom css spacing.

[Dan Beam, 2016/09/29 21:45:45]

i see now, thanks for 'splainin

-      }
-    </style>
+    <style include="settings-shared"></style>
     <settings-animated-pages id="pages" section="appearance">
       <neon-animatable route-path="default">
 <if expr="chromeos">
@@ -31,26 +27,29 @@
             on-tap="openWallpaperManager_" actionable
             hidden="[[!pageVisibility.setWallpaper]]">
           <iron-icon icon="settings:wallpaper"></iron-icon>

[Dan Beam, 2016/09/29 03:19:34]

like this

[dschuyler, 2016/09/29 21:22:50]

Acknowledged.

-          <div class="start">
+          <div class="middle">
             <div>$i18n{setWallpaper}</div>
             <div class="secondary">$i18n{openWallpaperApp}</div>
           </div>
+          <button class="icon-external" is="paper-icon-button-light">
+          </button>
         </div>
         <div class="settings-box two-line"
-            hidden="[[!pageVisibility.setTheme]]">
+            hidden="[[!pageVisibility.setTheme]]"
+            on-tap="openThemesGallery_" actionable>
 </if>
 <if expr="not chromeos">
         <div class="settings-box two-line first"
-            hidden="[[!pageVisibility.setTheme]]">
+            hidden="[[!pageVisibility.setTheme]]"
+            on-tap="openThemesGallery_" actionable>
 </if>
-          <div class="layout horizontal center flex"
-              on-tap="openThemesGallery_" actionable>
-            <iron-icon icon="settings:palette"></iron-icon>
-            <div>
-              <div>$i18n{theme}</div>
-              <div class="secondary">[[themeSublabel_]]</div>
-            </div>
+          <iron-icon icon="settings:palette"></iron-icon>

[Dan Beam, 2016/09/28 21:56:26]

are we dropping this now or later?

[dschuyler, 2016/09/29 00:40:57]

I'd like to avoiding adding more to this CL.

+          <div class="middle">
+            <div>$i18n{theme}</div>
+            <div class="secondary">[[themeSublabel_]]</div>
           </div>
+          <button class="icon-external" is="paper-icon-button-light">
+          </button>
           <template is="dom-if" if="[[allowResetTheme_]]">
             <div class="secondary-action">
               <paper-button id="resetTheme" on-tap="resetTheme_"
@@ -95,8 +94,7 @@
               pref="{{prefs.bookmark_bar.show_on_all_tabs}}">
           </settings-checkbox>
         </div>
-        <div class$="settings-box [[getFirst_(
-            pageVisibility.bookmarksBar)]]">
+        <div class$="settings-box [[getFirst_(pageVisibility.bookmarksBar)]]">
 <if expr="is_linux and not chromeos">
           <settings-checkbox class="start" label="$i18n{showWindowDecorations}"
               pref="{{prefs.browser.custom_chrome_frame}}" inverted>
@@ -110,11 +108,14 @@
               menu-options="[[fontSizeOptions_]]">
           </settings-dropdown-menu>
         </div>
-        <div id="customize-fonts-subpage-trigger"
-            class="settings-box two-line single-column"
+        <div id="customize-fonts-subpage-trigger" class="settings-box two-line"
             on-tap="onCustomizeFontsTap_" actionable>
-          <div>$i18n{customizeFonts}</div>
-          <div class="secondary">$i18n{chooseFonts}</div>
+          <div class="start">
+            <div>$i18n{customizeFonts}</div>
+            <div class="secondary">$i18n{chooseFonts}</div>
+          </div>
+          <button class="icon-arrow-right" is="paper-icon-button-light">
+          </button>
         </div>
         <div class="settings-box"
             hidden="[[!pageVisibility.pageZoom]]">