Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(503)

Side by Side Diff: chrome/browser/resources/settings/device_page/stylus.html

Issue 2617533003: MD Settings: change most checkboxes to toggles (leave dialogs alone) (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <link rel="import" href="chrome://resources/html/action_link.html"> 1 <link rel="import" href="chrome://resources/html/action_link.html">
2 <link rel="import" href="chrome://resources/html/polymer.html"> 2 <link rel="import" href="chrome://resources/html/polymer.html">
3 <link rel="import" href="/controls/settings_checkbox.html"> 3 <link rel="import" href="/controls/settings_toggle_button.html">
4 <link rel="import" href="/settings_shared_css.html"> 4 <link rel="import" href="/settings_shared_css.html">
5 5
6 <dom-module id="settings-stylus"> 6 <dom-module id="settings-stylus">
7 <template> 7 <template>
8 <style include="settings-shared"></style> 8 <style include="settings-shared"></style>
9 9
10 <div class="settings-box"> 10 <div class="settings-box">
11 <settings-checkbox 11 <settings-toggle-button
12 pref="{{prefs.settings.enable_stylus_tools}}" 12 pref="{{prefs.settings.enable_stylus_tools}}"
13 label="$i18n{stylusEnableStylusTools}"> 13 label="$i18n{stylusEnableStylusTools}">
14 </settings-checkbox> 14 </settings-toggle-button>
15 </div> 15 </div>
16 16
17 <div class="settings-box continuation"> 17 <div class="settings-box continuation">
18 <settings-checkbox 18 <settings-toggle-button
19 disabled="[[!prefs.settings.enable_stylus_tools.value]]"
20 pref="{{prefs.settings.launch_palette_on_eject_event}}" 19 pref="{{prefs.settings.launch_palette_on_eject_event}}"
21 label="$i18n{stylusAutoOpenStylusTools}"> 20 label="$i18n{stylusAutoOpenStylusTools}"
22 </settings-checkbox> 21 disabled="[[!prefs.settings.enable_stylus_tools.value]]">
dschuyler 2017/01/05 20:25:05 It's okay if this disabled line needs to move, but
Dan Beam 2017/01/06 02:45:29 so all <settings-toggle-button> are more consisten
22 </settings-toggle-button>
23 </div> 23 </div>
24 24
25 <div class="settings-box"> 25 <div class="settings-box">
26 <!-- TODO(jdufault): This should launch an intent (using the 26 <!-- TODO(jdufault): This should launch an intent (using the
27 enable-intent-picker flag) that opens up the play store to a curated 27 enable-intent-picker flag) that opens up the play store to a curated
28 list of apps. --> 28 list of apps. -->
29 <a is="action-link" target="_blank" 29 <a is="action-link" target="_blank"
30 href="https://play.google.com/store/apps/collection/promotion_30023cb_s tylus_apps"> 30 href="https://play.google.com/store/apps/collection/promotion_30023cb_s tylus_apps">
31 $i18n{stylusFindMoreApps} 31 $i18n{stylusFindMoreApps}
32 </a> 32 </a>
33 </div> 33 </div>
34 34
35 </template> 35 </template>
36 <script src="stylus.js"></script> 36 <script src="stylus.js"></script>
37 </dom-module> 37 </dom-module>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698