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

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

Issue 1965913005: ChromeOS: Implement minumal material design OOBE. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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
new file mode 100644
index 0000000000000000000000000000000000000000..bbacefc85f5cada95cd4201f88dcd652b10392e5
--- /dev/null
+++ b/chrome/browser/resources/chromeos/login/oobe_welcome.js
@@ -0,0 +1,59 @@
+// 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({
+is: 'oobe-poly-welcome',
jdufault 2016/05/11 22:07:44 nit: indent
Alexander Alekseev 2016/05/13 20:52:35 Done.
+
+ properties: {
+ disabled: {
+ type: Boolean,
+ value: false,
+ },
+ },
+
+ ready:
+ function() {
+ /**
+ * Workaround for
+ * https://github.com/PolymerElements/neon-animation/issues/32
+ * TODO(dzhioev): Remove when fixed in Polymer.
jdufault 2016/05/11 22:07:44 Is this still broken? The bug is almost a year old
Alexander Alekseev 2016/05/13 20:52:35 THhs is actually copy-paste from offline gaia scre
+ */
+ var pages = this.$.animatedPages;
+ delete pages._squelchNextFinishEvent;
+ Object.defineProperty(
+ pages, '_squelchNextFinishEvent',
+ {get: function() { return false; }});
+ },
+
+ focus:
jdufault 2016/05/11 22:07:44 nit: Place function() on the same line as the prop
Alexander Alekseev 2016/05/13 20:52:35 Done.
+ function() {
+ if (this.isWelcomeSectionActive_())
+ this.$.oobeWelcomeNextButton.focus();
+ else
+ this.$.oobeNetworkSettings.focus();
+ },
+
+ onAnimationFinish_:
jdufault 2016/05/11 22:07:44 nit: as above.
Alexander Alekseev 2016/05/13 20:52:35 Done.
+ function() {
+ this.focus();
+ },
+
+ isRTL_:
jdufault 2016/05/11 22:07:44 nit: as above.
Alexander Alekseev 2016/05/13 20:52:35 Done.
+ function() {
+ return !!document.querySelector('html[dir=rtl]');
+ },
+
+ isWelcomeSectionActive_:
jdufault 2016/05/11 22:07:44 nit: as above.
Alexander Alekseev 2016/05/13 20:52:35 Done.
+ function() {
+ return this.$.animatedPages.selected == 'welcomeSection';
+ },
+
+ setUpPageTransitions_: function() {
+ var isForward = true;
+ var isRTL = this.isRTL_();
+ this.$.animatedPages.entryAnimation =
+ 'slide-from-' + (isRTL ? 'left' : 'right') +
+ '-animation';
+ this.$.animatedPages.exitAnimation =
+ 'slide-' + (isRTL ? 'right' : 'left') + '-animation';
jdufault 2016/05/11 22:07:44 Add in the trailing }s?
Alexander Alekseev 2016/05/13 20:52:35 Done.

Powered by Google App Engine
This is Rietveld 408576698