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

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

Issue 2108813002: Added strings of i18n and made the pin-keyboard work for rtl lang. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkgr
Patch Set: Fixed patch set 3 errors. Created 4 years, 6 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/people_page/people_page.html
diff --git a/chrome/browser/resources/settings/people_page/people_page.html b/chrome/browser/resources/settings/people_page/people_page.html
index 56dda11a553edd831dc59d6842002eacb1d847c4..49f8db4f220f5dd7886318a8bd06e9cf921d0c55 100644
--- a/chrome/browser/resources/settings/people_page/people_page.html
+++ b/chrome/browser/resources/settings/people_page/people_page.html
@@ -162,7 +162,8 @@
<div class="settings-box">
<!-- TODO(jdufault): i18n -->
<p>Quick Unlock PIN</p>
- <pin-keyboard></pin-keyboard>
+ <pin-keyboard placeholder="$i18n{pinKeyboardPlaceholderPin}">
jdufault 2016/06/28 23:06:10 Drop this change; placeholder should be driven by
sammiequon 2016/06/29 19:44:08 Done.
+ </pin-keyboard>
</div>
</template>

Powered by Google App Engine
This is Rietveld 408576698