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

Side by Side Diff: chrome/browser/resources/chromeos/login/oobe_a11y_option.js

Issue 2688153004: ChromeOS OOBE: fix focus flow in MD-OOBE. (Closed)
Patch Set: Created 3 years, 10 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 Polymer({ 5 Polymer({
6 is: 'oobe-a11y-option', 6 is: 'oobe-a11y-option',
7 7
8 properties: { 8 properties: {
9 /** 9 /**
10 * If paper-toggle-button is checked. 10 * If paper-toggle-button is checked.
11 */ 11 */
12 checked: { 12 checked: {
13 type: Boolean, 13 type: Boolean,
14 }, 14 },
15 15
16 /** 16 /**
17 * Chrome message handling this option. 17 * Chrome message handling this option.
18 */ 18 */
19 chromeMessage: { 19 chromeMessage: {
20 type: String, 20 type: String,
21 }, 21 },
22 22
23 /** 23 /**
24 * ARIA-label for the button. 24 * ARIA-label for the button.
25 */ 25 */
26 ariaLabel: String, 26 label: String,
stevenjb 2017/02/14 01:21:32 Why does this need to be renamed here and elsewher
Alexander Alekseev 2017/02/14 11:59:27 Because otherwise it will become aria-label attrib
stevenjb 2017/02/14 17:35:34 I see. I'm not super fond of just 'label' for some
Alexander Alekseev 2017/02/14 23:41:17 Done.
27 },
28
29 focus: function() {
30 this.$.button.focus();
27 }, 31 },
28 }); 32 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698