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

Side by Side Diff: ui/login/account_picker/user_pod_row.js

Issue 2644903002: Action box area should get focus back after closing user menu (Closed)
Patch Set: Created 3 years, 11 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview User pod row implementation. 6 * @fileoverview User pod row implementation.
7 */ 7 */
8 8
9 cr.define('login', function() { 9 cr.define('login', function() {
10 /** 10 /**
(...skipping 1516 matching lines...) Expand 10 before | Expand all | Expand 10 after
1527 this.actionBoxMenuRemoveElement.focus(); 1527 this.actionBoxMenuRemoveElement.focus();
1528 } 1528 }
1529 e.stopPropagation(); 1529 e.stopPropagation();
1530 break; 1530 break;
1531 // Ignore these two, so ChromeVox hotkeys don't close the menu before 1531 // Ignore these two, so ChromeVox hotkeys don't close the menu before
1532 // they can navigate through it. 1532 // they can navigate through it.
1533 case 'Shift': 1533 case 'Shift':
1534 case 'Meta': 1534 case 'Meta':
1535 break; 1535 break;
1536 case 'Escape': 1536 case 'Escape':
1537 this.actionBoxAreaElement.focus();
1537 this.isActionBoxMenuActive = false; 1538 this.isActionBoxMenuActive = false;
1538 e.stopPropagation(); 1539 e.stopPropagation();
1539 break; 1540 break;
1540 case 'Tab': 1541 case 'Tab':
1541 if (!this.parentNode.alwaysFocusSinglePod) 1542 if (!this.parentNode.alwaysFocusSinglePod)
1542 this.parentNode.focusPod(); 1543 this.parentNode.focusPod();
1543 default: 1544 default:
1544 this.isActionBoxMenuActive = false; 1545 this.isActionBoxMenuActive = false;
1545 break; 1546 break;
1546 } 1547 }
(...skipping 2174 matching lines...) Expand 10 before | Expand all | Expand 10 after
3721 if (pod && pod.multiProfilesPolicyApplied) { 3722 if (pod && pod.multiProfilesPolicyApplied) {
3722 pod.userTypeBubbleElement.classList.remove('bubble-shown'); 3723 pod.userTypeBubbleElement.classList.remove('bubble-shown');
3723 } 3724 }
3724 } 3725 }
3725 }; 3726 };
3726 3727
3727 return { 3728 return {
3728 PodRow: PodRow 3729 PodRow: PodRow
3729 }; 3730 };
3730 }); 3731 });
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698