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

Unified Diff: chrome/browser/resources/settings/languages_page/languages_page.html

Issue 2523403003: Language settings: fix focus issues (Closed)
Patch Set: Created 4 years, 1 month 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/languages_page/languages_page.html
diff --git a/chrome/browser/resources/settings/languages_page/languages_page.html b/chrome/browser/resources/settings/languages_page/languages_page.html
index c664a0dca741579cf0d9c993c39da0470a06464d..e7456189f0e5e04813e75278eefa80a92fd8ff9a 100644
--- a/chrome/browser/resources/settings/languages_page/languages_page.html
+++ b/chrome/browser/resources/settings/languages_page/languages_page.html
@@ -1,3 +1,4 @@
+<link rel="import" href="chrome://resources/html/action_link.html">
<link rel="import" href="chrome://resources/html/assert.html">
<link rel="import" href="chrome://resources/html/cr.html">
<link rel="import" href="chrome://resources/html/polymer.html">
@@ -70,7 +71,8 @@
</settings-languages>
<settings-animated-pages id="pages" section="languages">
<neon-animatable route-path="default">
- <div class="settings-box first two-line">
+ <div class="settings-box first two-line" actionable
+ on-tap="toggleExpandButton_">
<div class="start">
<div>$i18n{languagesListTitle}</div>
<div class="secondary">
@@ -121,13 +123,14 @@
</div>
</template>
<div class="list-item list-button" on-tap="onAddLanguagesTap_">
- $i18n{addLanguages}
+ <a is="action-link">$i18n{addLanguages}</a>
</div>
</div>
</iron-collapse>
<if expr="chromeos">
<div id="manage-input-methods-subpage-trigger"
- class="settings-box two-line">
+ class="settings-box two-line" actionable
+ on-tap="toggleExpandButton_">
<div class="start">
<div>$i18n{inputMethodsListTitle}</div>
<div class="secondary">
@@ -145,7 +148,8 @@
items="[[languages.inputMethods.enabled]]">
<div class$="list-item [[getInputMethodItemClass_(
item.id, languages.inputMethods.currentId)]]"
- on-tap="onInputMethodTap_" actionable>
+ on-tap="onInputMethodTap_" on-keypress="onInputMethodTap_"
+ actionable tabindex="0">
<div class="start">
<div>[[item.displayName]]</div>
<div class="explain-selected"
@@ -162,13 +166,14 @@
</template>
<div class="list-item list-button"
on-tap="onManageInputMethodsTap_">
- $i18n{manageInputMethods}
+ <a is="action-link">$i18n{manageInputMethods}</a>
</div>
</div>
</iron-collapse>
</if>
<if expr="not is_macosx">
- <div id="spellcheck-subpage-trigger" class="settings-box two-line">
+ <div id="spellcheck-subpage-trigger" class="settings-box two-line"
+ actionable on-tap="toggleExpandButton_">
<div class="start">
<div>$i18n{spellCheckListTitle}</div>
<div class="secondary">
@@ -192,7 +197,7 @@
</label>
</template>
<div class="list-item list-button" on-tap="onEditDictionaryTap_">
- $i18n{manageSpellCheck}
+ <a is="action-link">$i18n{manageSpellCheck}</a>
</div>
</div>
</iron-collapse>

Powered by Google App Engine
This is Rietveld 408576698