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

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

Issue 1013613002: Three dot menu is shown now on WebView start screen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Button used instead of div for 'Add SU user'. Created 5 years, 9 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/header_bar.js
diff --git a/chrome/browser/resources/chromeos/login/header_bar.js b/chrome/browser/resources/chromeos/login/header_bar.js
index 2a3b622f9dd61e34ae5da98a4bbd181a815fab94..6c6a75911e21697b371ad6ed14e8f25c27e9678c 100644
--- a/chrome/browser/resources/chromeos/login/header_bar.js
+++ b/chrome/browser/resources/chromeos/login/header_bar.js
@@ -147,6 +147,7 @@ cr.define('login', function() {
handleMoreSettingsClick_: function(e) {
this.isMoreSettingsActive = !this.isMoreSettingsActive;
+ this.addSupervisedUserMenu.focus();
e.stopPropagation();
},
@@ -158,6 +159,7 @@ cr.define('login', function() {
chrome.send('showSupervisedUserCreationScreen');
e.preventDefault();
},
+
/**
* Cancel add user button click handler.
*
@@ -329,8 +331,8 @@ cr.define('login', function() {
$('add-user-button').hidden =
!accountPickerIsActive || isMultiProfilesUI || isLockScreen;
$('more-settings-header-bar-item').hidden = !this.isMinuteMaid_ ||
- !gaiaIsActive ||
- !this.showCreateSupervised_;
+ !this.showCreateSupervised_ ||
+ !accountPickerIsActive;
$('cancel-add-user-button').hidden =
(gaiaIsActive && this.isMinuteMaid_) ||
accountPickerIsActive ||
« no previous file with comments | « chrome/browser/resources/chromeos/login/header_bar.html ('k') | chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698