Chromium Code Reviews| 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> |