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

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

Issue 2718133004: ChromeOS OOBE: More accessibility fixes. (Closed)
Patch Set: Update after review Created 3 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 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-text-button', 6 is: 'oobe-text-button',
7 7
8 properties: { 8 properties: {
9 disabled: {type: Boolean, value: false, reflectToAttribute: true}, 9 disabled: {type: Boolean, value: false, reflectToAttribute: true},
10 10
11 inverse: Boolean, 11 inverse: Boolean,
12 12
13 ariaLabel: String, 13 /* Note that we are not using "aria-label" property here, because
14 * we want to pass the label value but not actually declare it as an
15 * ARIA property anywhere but the actual target element.
16 */
17 labelForAria: String,
14 }, 18 },
15 19
16 focus: function() { 20 focus: function() {
17 this.$.textButton.focus(); 21 this.$.textButton.focus();
18 }, 22 },
19 23
20 onClick_: function(e) { 24 onClick_: function(e) {
21 if (this.disabled) 25 if (this.disabled)
22 e.stopPropagation(); 26 e.stopPropagation();
23 } 27 }
24 }); 28 });
25 29
26 Polymer({ 30 Polymer({
27 is: 'oobe-back-button', 31 is: 'oobe-back-button',
28 32
29 properties: { 33 properties: {
30 disabled: {type: Boolean, value: false, reflectToAttribute: true}, 34 disabled: {type: Boolean, value: false, reflectToAttribute: true},
31 35
32 ariaLabel: String, 36 /* Note that we are not using "aria-label" property here, because
37 * we want to pass the label value but not actually declare it as an
38 * ARIA property anywhere but the actual target element.
39 */
40 labelForAria: String,
33 }, 41 },
34 42
35 focus: function() { 43 focus: function() {
36 this.$.button.focus(); 44 this.$.button.focus();
37 }, 45 },
38 46
39 onClick_: function(e) { 47 onClick_: function(e) {
40 if (this.disabled) 48 if (this.disabled)
41 e.stopPropagation(); 49 e.stopPropagation();
42 } 50 }
(...skipping 15 matching lines...) Expand all
58 e.stopPropagation(); 66 e.stopPropagation();
59 } 67 }
60 }); 68 });
61 69
62 Polymer({ 70 Polymer({
63 is: 'oobe-welcome-secondary-button', 71 is: 'oobe-welcome-secondary-button',
64 72
65 properties: { 73 properties: {
66 icon: String, 74 icon: String,
67 75
68 ariaLabel: String 76 /* Note that we are not using "aria-label" property here, because
77 * we want to pass the label value but not actually declare it as an
78 * ARIA property anywhere but the actual target element.
79 */
80 labelForAria: String
69 }, 81 },
70 82
71 focus: function() { 83 focus: function() {
72 this.$.button.focus(); 84 this.$.button.focus();
73 }, 85 },
74 }); 86 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/chromeos/login/oobe_buttons.html ('k') | chrome/browser/resources/chromeos/login/oobe_eula.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698