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

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

Issue 12087105: Use drop-down UI for "add language". (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebasing Created 7 years, 10 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.js
diff --git a/chrome/browser/resources/options/language_add_language_overlay.js b/chrome/browser/resources/options/language_add_language_overlay.js
index d5ebf93554ebc669ecb22d4790b391333da2a91b..c564c2d488b26f29b0d22e43153f3304862cde2d 100644
--- a/chrome/browser/resources/options/language_add_language_overlay.js
+++ b/chrome/browser/resources/options/language_add_language_overlay.js
@@ -44,25 +44,13 @@ cr.define('options', function() {
var language = languageListData[i];
var displayText = language.displayName;
// If the native name is different, add it.
- if (language.displayName != language.nativeDisplayName) {
+ if (language.displayName != language.nativeDisplayName)
displayText += ' - ' + language.nativeDisplayName;
- }
- if (cr.isChromeOS) {
- var button = document.createElement('button');
- button.className = 'link-button';
- button.textContent = displayText;
- button.languageCode = language.code;
- var li = document.createElement('li');
- li.languageCode = language.code;
- li.appendChild(button);
- addLanguageList.appendChild(li);
- } else {
- var option = document.createElement('option');
- option.value = language.code;
- option.textContent = displayText;
- addLanguageList.appendChild(option);
- }
+ var option = cr.doc.createElement('option');
+ option.value = language.code;
+ option.textContent = displayText;
+ addLanguageList.appendChild(option);
}
},
};

Powered by Google App Engine
This is Rietveld 408576698