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

Unified Diff: chrome/browser/resources/chromeos/login/oobe_i18n_dropdown.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_i18n_dropdown.js
diff --git a/chrome/browser/resources/chromeos/login/oobe_i18n_dropdown.js b/chrome/browser/resources/chromeos/login/oobe_i18n_dropdown.js
new file mode 100644
index 0000000000000000000000000000000000000000..0d3d1c7ab2f8c7e7bee5969e6297c92efa906882
--- /dev/null
+++ b/chrome/browser/resources/chromeos/login/oobe_i18n_dropdown.js
@@ -0,0 +1,97 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/**
+ * Polymer class definition for 'oobe-i18n-dropdown'.
+ */
+(function() {
+
+
+/**
+ * Languages/keyboard descriptor to display
+ * @type {!OobeTypes.LanguageDsc|!OobeTypes.IMEDsc}
+ */
+var I18nMenuItem;
+
+Polymer({
+ is: 'oobe-i18n-dropdown',
+
+ properties: {
+ /**
+ * List of languages/keyboards to display
+ * @type {!Array<I18nMenuItem>}
+ */
+ items: {
+ type: Array,
+ observer: 'onItemsChanged_',
+ },
+
+ /**
+ * Accessibility label.
+ * @type {!string}
michaelpg 2016/08/04 07:56:45 type not needed when it's the same as below
+ */
+ label: {
+ type: String,
+ },
+ },
+
+ /**
+ * This flag prevents recursive calls of observers and callbacks.
+ */
+ observersDisabled_: false,
+
+ /**
+ * @param {!{detail: !{item: { item: {!I18nMenuItem}}}}} event
+ * @private
+ */
+ onSelect_: function(event) {
+ if (this.observersDisabled_)
+ return;
+
+ var selectedModel = this.$.domRepeat.modelForElement(event.detail.item);
+ if (!selectedModel.item)
+ return;
+
+ selectedModel.set('item.selected', true);
+ this.fire('select-item', selectedModel.item);
+ },
+
+ onDeselect_: function(event) {
+ if (this.observersDisabled_)
+ return;
+
+ var deSelectedModel = this.$.domRepeat.modelForElement(event.detail.item);
+ if (!deSelectedModel.item)
+ return;
+
+ deSelectedModel.set('item.selected', false);
+ },
+
+ onItemsChanged_: function(items) {
+ if (this.observersDisabled_)
+ return;
+
+ if (!items)
+ return;
+
+ this.observersDisabled_ = true;
+
+ var index = items.findIndex(function(item) { return item.selected; });
+ if (index != -1) {
+ // This is needed for selectIndex() to pick up values from updated
+ // this.items (for example, translated into other language).
+ Polymer.dom.flush();
+
+ if (this.$.listboxDropdown.selected == index) {
+ // This is to force update real <input> element value even if selection
+ // index has not changed.
+ this.$.listboxDropdown.selectIndex(null);
+ }
+ this.$.listboxDropdown.selectIndex(index);
+ }
+
+ this.observersDisabled_ = false;
+ },
+});
+})();

Powered by Google App Engine
This is Rietveld 408576698