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

Unified Diff: chrome/browser/resources/chromeos/login/oobe_welcome.js

Issue 2189733006: ChromeOS: Implement Language Selection screen of material design OOBE. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update after review. Created 4 years, 4 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/chromeos/login/oobe_welcome.js
diff --git a/chrome/browser/resources/chromeos/login/oobe_welcome.js b/chrome/browser/resources/chromeos/login/oobe_welcome.js
index 5b489319198265e3f739778f0a5b0a38a18673de..52cb4f9aeec03d47362eb603e2b8b1461f4c265b 100644
--- a/chrome/browser/resources/chromeos/login/oobe_welcome.js
+++ b/chrome/browser/resources/chromeos/login/oobe_welcome.js
@@ -11,20 +11,69 @@ Polymer({
properties: {
/**
- * Currently selected system language.
+ * Currently selected system language (display name).
*/
currentLanguage: {
type: String,
- value: 'English (US)',
+ value: '',
},
/**
- * Flag that switches Welcome screen to Network Selection screen.
+ * List of languages for language selector dropdown.
+ * @type {!Array<OobeTypes.LanguageDsc>}
+ */
+ languages: {
+ type: Array,
+ },
+
+ /**
+ * List of keyboards for keyboard selector dropdown.
+ * @type {!Array<OobeTypes.IMEDsc>}
+ */
+ keyboards: {
+ type: Array,
+ },
+
+ /**
+ * Flag that shows Welcome screen.
+ */
+ welcomeScreenShown: {
+ type: Boolean,
+ value: true,
+ },
+
+ /**
+ * Flag that shows Language Selection screen.
+ */
+ languageSelectionScreenShown: {
+ type: Boolean,
+ value: false,
+ },
+
+ /**
+ * Flag that shows Network Selection screen.
*/
networkSelectionScreenShown: {
type: Boolean,
value: false,
},
+
+ /**
+ * Flag that enables MD-OOBE.
michaelpg 2016/08/04 07:56:45 Can you explain more what this does? If this is th
+ */
+ enabled: {
+ type: Boolean,
+ value: false,
+ },
+ },
+
+ /**
+ * Hides all screens to help switching from one screen to another.
+ */
+ hideAllScreens_: function() {
+ this.welcomeScreenShown = false;
+ this.networkSelectionScreenShown = false;
+ this.languageSelectionScreenShown = false;
},
/**
@@ -85,10 +134,21 @@ Polymer({
* @private
*/
onWelcomeNextButtonClicked_: function() {
+ this.hideAllScreens_();
this.networkSelectionScreenShown = true;
},
/**
+ * Handle "Language" button for "Welcome" screen.
+ *
+ * @private
+ */
+ onWelcomeSelectLanguageButtonClicked_: function() {
+ this.hideAllScreens_();
+ this.languageSelectionScreenShown = true;
+ },
+
+ /**
* Handle Networwork Setup screen "Proxy settings" button.
*
* @private
@@ -180,9 +240,44 @@ Polymer({
/**
* @param {!Event} event
+ * @private
*/
onNetworkListCustomItemSelected_: function(e) {
var itemState = e.detail;
itemState.customData.onTap();
},
+
+ /**
+ * Handle language selection.
+ *
+ * @param {!{detail: {!OobeTypes.LanguageDsc}}} event
+ * @private
+ */
+ onLanguageSelected_: function(event) {
+ var item = event.detail;
+ var languageId = item.value;
+ this.screen.onLanguageSelected_(languageId);
+ },
+
+ /**
+ * Handle keyboard layout selection.
+ *
+ * @param {!{detail: {!OobeTypes.IMEDsc}}} event
+ * @private
+ */
+ onKeyboardSelected_: function(event) {
+ var item = event.detail;
+ var inputMethodId = item.value;
+ this.screen.onKeyboardSelected_(inputMethodId);
+ },
+
+ /**
+ * Handle "OK" button for "LanguageSelection" screen.
+ *
+ * @private
+ */
+ closeLanguageSection_: function() {
+ this.hideAllScreens_();
+ this.welcomeScreenShown = true;
+ },
});

Powered by Google App Engine
This is Rietveld 408576698