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

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, 5 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..5bc41fd04bd9336da5aee9fbef1153c40805dc4a 100644
--- a/chrome/browser/resources/chromeos/login/oobe_welcome.js
+++ b/chrome/browser/resources/chromeos/login/oobe_welcome.js
@@ -11,15 +11,49 @@ 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,
+ value: null,
michaelpg 2016/08/02 00:06:38 same for these, can the value just be left out? se
Alexander Alekseev 2016/08/02 04:25:57 Done.
+ },
+
+ /**
+ * List of keyboards for keyboard selector dropdown.
+ * @type {!Array<OobeTypes.IMEDsc>}
+ */
+ keyboards: {
+ type: Array,
+ value: null,
+ },
+
+ /**
+ * 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,
@@ -28,6 +62,15 @@ Polymer({
},
/**
+ * Hides all screens to help switching from one screen to another.
+ */
+ hideAllScreens_: function() {
+ this.welcomeScreenShown = false;
+ this.networkSelectionScreenShown = false;
+ this.languageSelectionScreenShown = false;
+ },
+
+ /**
* GUID of the user-selected network. It is remembered after user taps on
* network entry. After we receive event "connected" on this network,
* OOBE will proceed.
@@ -85,10 +128,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 +234,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