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

Unified Diff: chrome/browser/resources/settings/device_page/stylus.html

Issue 2630623002: cros: Allow user to configure note taking app in md-settings. (Closed)
Patch Set: Address comments 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/device_page/stylus.html
diff --git a/chrome/browser/resources/settings/device_page/stylus.html b/chrome/browser/resources/settings/device_page/stylus.html
index 2d4a4737370bbbb3357c04dfd5064bde03621a6d..cf7b757083c949769de8baa419d7d46fae56c1fb 100644
--- a/chrome/browser/resources/settings/device_page/stylus.html
+++ b/chrome/browser/resources/settings/device_page/stylus.html
@@ -1,20 +1,37 @@
<link rel="import" href="chrome://resources/html/action_link.html">
<link rel="import" href="chrome://resources/html/polymer.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button-light.html">
<link rel="import" href="/controls/settings_toggle_button.html">
<link rel="import" href="/settings_shared_css.html">
<dom-module id="settings-stylus">
<template>
- <style include="settings-shared"></style>
+ <style include="settings-shared md-select">
+ .settings-box > .secondary {
+ align-items: center;
+ display: flex;
+ /* 'secondary' class applies a top margin. This breaks centering. */
+ margin-top: 0;
+ }
- <div class="settings-box">
+ settings-toggle-button {
+ width: 100%;
+ }
+
+ paper-spinner {
+ margin-left: 12px;
+ @apply(--cr-icon-height-width);
+ }
+ </style>
+
+ <div class="settings-box continuation">
<settings-toggle-button class="start"
pref="{{prefs.settings.enable_stylus_tools}}"
label="$i18n{stylusEnableStylusTools}">
</settings-toggle-button>
</div>
- <div class="settings-box continuation">
+ <div class="settings-box">
<settings-toggle-button class="start"
pref="{{prefs.settings.launch_palette_on_eject_event}}"
label="$i18n{stylusAutoOpenStylusTools}"
@@ -23,13 +40,41 @@
</div>
<div class="settings-box">
- <!-- TODO(jdufault): This should launch an intent (using the
- enable-intent-picker flag) that opens up the play store to a curated
- list of apps. -->
- <a is="action-link" target="_blank"
- href="https://play.google.com/store/apps/collection/promotion_30023cb_stylus_apps">
- $i18n{stylusFindMoreApps}
- </a>
+ <div class="start">$i18n{stylusNoteTakingApp}</div>
+
+ <div id="no-apps" class="secondary"
+ hidden$="[[!showNoApps_(appChoices_, waitingForAndroid_)]]">
+ $i18n{stylusNoteTakingAppNoneAvailable}
+ </div>
+
+ <div id="waiting" class="secondary" hidden$="[[!waitingForAndroid_]]">
+ $i18n{stylusNoteTakingAppWaitingForAndroid}
+ <paper-spinner active></paper-spinner>
+ </div>
+
+ <div id="select-app" class="md-select-wrapper"
+ hidden$="[[!showApps_(appChoices_, waitingForAndroid_)]]">
+ <select id="menu" class="md-select"
+ disabled="[[!prefs.settings.enable_stylus_tools.value]]"
+ on-change="onSelectedAppChanged_">
+ <template is="dom-repeat" items="[[appChoices_]]">
+ <option value="[[item.value]]" selected$="[[item.preferred]]">
+ [[item.name]]
+ </option>
+ </template>
+ </select>
+ <span class="md-select-underline"></span>
+ </div>
+ </div>
+
+ <div class="settings-box two-line" on-tap="onFindAppsTap_" actionable>
+ <div class="start">
+ $i18n{stylusFindMoreAppsPrimary}
+ <div class="secondary">
+ $i18n{stylusFindMoreAppsSecondary}
+ </div>
+ </div>
+ <button class="icon-external" is="paper-icon-button-light"></button>
</div>
</template>

Powered by Google App Engine
This is Rietveld 408576698