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

Unified Diff: chrome/browser/resources/options/language_add_language_overlay.html

Issue 6296017: dom-ui settings: enable languages & spell checker settings for all platforms.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: chromeos namespace fix, part 4 Created 9 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/options/language_add_language_overlay.html
===================================================================
--- chrome/browser/resources/options/language_add_language_overlay.html (revision 71884)
+++ chrome/browser/resources/options/language_add_language_overlay.html (working copy)
@@ -1,6 +1,24 @@
<div class="page hidden" id="add-language-overlay-page">
+ <h1 i18n-content="add_language_title"></h1>
satorux1 2011/01/21 01:38:46 Please move this to inside of <if expr="not pp_ifd
+<if expr="pp_ifdef('chromeos')">
<ul id="add-language-overlay-language-list">
</ul>
<button id="add-language-overlay-cancel-button"
i18n-content="cancel"></button>
+</if>
+<if expr="not pp_ifdef('chromeos')">
satorux1 2011/01/20 10:40:33 Why cannot we share the same UI? The Chrome OS ver
csilv 2011/01/20 20:09:55 The two column UI works well on ChromeOS, but on t
satorux1 2011/01/21 01:38:46 Thank you for the explanation. Didn't know that we
+ <label>
+ <span i18n-content="add_language_select_label"></span>
+ <select id="add-language-overlay-language-list">
+ </select>
James Hawkins 2011/01/20 03:11:08 Fits on the line above.
csilv 2011/01/20 20:09:55 Done.
+ </label>
+ <div class="action-area">
+ <div class="button-strip">
+ <button id="add-language-overlay-cancel-button" i18n-content="cancel">
+ </button>
+ <button id="add-language-overlay-ok-button" i18n-content="ok">
+ </button>
James Hawkins 2011/01/20 03:11:08 Probably fits on the line above.
csilv 2011/01/20 20:09:55 Done.
+ </div>
+ </div>
+</if>
</div>

Powered by Google App Engine
This is Rietveld 408576698