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

Unified Diff: ui/login/account_picker/user_pod_row.js

Issue 2516903002: PS - Adjusting Public Session login pod for expanded whitelisting (Closed)
Patch Set: Fixed monitoring-dialog position in advanced view Created 4 years, 1 month 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: ui/login/account_picker/user_pod_row.js
diff --git a/ui/login/account_picker/user_pod_row.js b/ui/login/account_picker/user_pod_row.js
index 3fb9aa0169baa32a2a950262864b153e3c5d25ce..fba1c21dc65ffaf02f7ae68aca2ff41299075394 100644
--- a/ui/login/account_picker/user_pod_row.js
+++ b/ui/login/account_picker/user_pod_row.js
@@ -2092,6 +2092,16 @@ cr.define('login', function() {
languageAndInput.addEventListener('click',
this.transitionToAdvanced_.bind(this));
+ var monitoringLearnMore = this.querySelector('.monitoring-learn-more');
+ monitoringLearnMore.tabIndex = UserPodTabOrder.POD_INPUT;
+ monitoringLearnMore.addEventListener(
+ 'click', this.onMonitoringLearnMoreClicked_.bind(this));
+
+ var monitoringDialog = this.querySelector('.monitoring-dialog');
+ monitoringDialog.addEventListener(
+ 'iron-overlay-closed',
+ this.onMonitoringDialogOverlayClosed_.bind(this));
+
this.enterButtonElement.addEventListener('click', (function(e) {
this.enterButtonElement.disabled = true;
var locale = this.querySelector('.language-select').value;
@@ -2243,6 +2253,25 @@ cr.define('login', function() {
},
/**
+ * Show a dialog when user clicks on learn more (monitoring) button.
+ */
+ onMonitoringLearnMoreClicked_: function() {
+ monitoringDialog = this.querySelector('.monitoring-dialog');
+ monitoringDialog.fitInto = this;
+ monitoringDialog.open();
+ monitoringDialog.focus();
+ this.querySelector('.monitoring-overlay').hidden = false;
+
+ },
+
+ /**
+ * Cleanup after the (monitoring) dialog overlay is closed.
+ */
+ onMonitoringDialogOverlayClosed_: function() {
+ this.querySelector('.monitoring-overlay').hidden = true;
+ },
+
+ /**
* Retrieves the list of keyboard layouts available for the currently
* selected locale.
*/

Powered by Google App Engine
This is Rietveld 408576698