MD Settings: make custom UIs look "actionable"

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

 <link rel="import" href="chrome://resources/html/i18n_behavior.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/classes/iron-flex-layout.html">
 <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-radio-button/paper-radio-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-toggle-button/paper-toggle-button.html">
 <link rel="import" href="/appearance_page/appearance_browser_proxy.html">
 <link rel="import" href="/controls/settings_dropdown_menu.html">
@@ skipping 12 matching lines @@
     <style include="settings-shared">
       .settings-box iron-icon {
         -webkit-margin-end: var(--iron-icon-spacing);
       }
     </style>
     <settings-animated-pages id="pages" current-route="{{currentRoute}}"
         section="appearance">
       <neon-animatable id="main">
 <if expr="chromeos">
         <div class="settings-box first two-line" id="wallpaperButton"
-            on-tap="openWallpaperManager_">
+            on-tap="openWallpaperManager_" actionable>
           <iron-icon icon="settings:wallpaper"></iron-icon>
           <div class="start">
             <div>$i18n{setWallpaper}</div>
             <div class="secondary">$i18n{openWallpaperApp}</div>
           </div>
         </div>
         <div class="settings-box two-line">
 </if>
 <if expr="not chromeos">
         <div class="settings-box two-line first">
 </if>
           <div class="layout horizontal center flex"
-              on-tap="openThemesGallery_">
+              on-tap="openThemesGallery_" actionable>
             <iron-icon icon="settings:palette"></iron-icon>
             <div>
               <div>$i18n{getThemes}</div>
               <div class="secondary">[[themeSublabel_]]</div>
             </div>
           </div>
           <template is="dom-if" if="[[allowResetTheme_]]">
             <div class="secondary-action">
-              <paper-button id="resetTheme" on-tap="resetTheme_"
+              <paper-button id="resetTheme" on-tap="resetTheme_" actionable

[dpapad, 2016/06/23 20:51:45]

Given that paper-buttons are by default having the correct cursor, do we need
this? Or are you adding it just for consistency? Personally I would be fine with
only adding it in divs with on-tap (or in general elements that don't already
have the correct cursor).

[Dan Beam, 2016/06/23 21:28:03]

no, this is wrong.  Reverted.

                   class="secondary-button">
                 $i18n{resetToDefaultTheme}
               </paper-button>
             </div>
           </template>
         </div>
         <div class="settings-box two-line">
           <settings-checkbox class="start" label="$i18n{showHomeButton}"
               sub-label="[[getShowHomeSubLabel_(prefs.homepage_is_newtabpage.value, prefs.homepage.value)]]"
               pref="{{prefs.browser.show_home_button}}">
@@ skipping 19 matching lines @@
               pref="{{prefs.bookmark_bar.show_on_all_tabs}}">
           </settings-checkbox>
         </div>
         <div class="settings-box">
           <div class="start">$i18n{fontSize}</div>
           <settings-dropdown-menu id="defaultFontSize"
               pref="{{prefs.webkit.webprefs.default_font_size}}"
               menu-options="[[fontSizeOptions_]]" no-label-float>
           </settings-dropdown-menu>
         </div>
-        <div class="settings-box two-line" on-tap="onCustomizeFontsTap_">
+        <div class="settings-box two-line" on-tap="onCustomizeFontsTap_"
+            actionable>
           <div class="start">
             <div>$i18n{customizeFonts}</div>
           <div class="secondary">$i18n{chooseFontsAndEncoding}</div>
           </div>
         </div>
         <div class="settings-box">
           <div class="start">$i18n{pageZoom}</div>
           <settings-dropdown-menu id="pageZoom" pref="{{defaultZoomLevel_}}"
               menu-options="[[pageZoomOptions_]]" no-label-float>
           </settings-dropdown-menu>
         </div>
       </neon-animatable>
       <template is="dom-if" name="appearance-fonts">
         <settings-subpage page-title="$i18n{customizeFonts}">
           <settings-appearance-fonts-page prefs="{{prefs}}">
           </settings-appearance-fonts-page>
         </settings-subpage>
       </template>
     </settings-animated-pages>
   </template>
   <script src="appearance_page.js"></script>
 </dom-module>