Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 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 | 6 * @fileoverview |
| 7 * `cr-checkbox` is a button that can be either checked or unchecked. User | 7 * `cr-checkbox` is a button that can be either checked or unchecked. User |
| 8 * can tap the checkbox to check or uncheck it. Usually you use checkboxes | 8 * can tap the checkbox to check or uncheck it. Usually you use checkboxes |
| 9 * to allow user to select multiple options from a set. If you have a single | 9 * to allow user to select multiple options from a set. If you have a single |
| 10 * ON/OFF option, avoid using a single checkbox and use `cr-toggle-button` | 10 * ON/OFF option, avoid using a single checkbox and use `cr-toggle-button` |
| (...skipping 22 matching lines...) Expand all Loading... | |
| 33 /** | 33 /** |
| 34 * If true, the user cannot interact with this element. | 34 * If true, the user cannot interact with this element. |
| 35 */ | 35 */ |
| 36 disabled: { | 36 disabled: { |
| 37 type: Boolean, | 37 type: Boolean, |
| 38 value: false, | 38 value: false, |
| 39 reflectToAttribute: true | 39 reflectToAttribute: true |
| 40 }, | 40 }, |
| 41 }, | 41 }, |
| 42 | 42 |
| 43 /** @override */ | |
| 44 attached: function() { | |
| 45 // HACK(dschuyler): paper-checkbox 1.0.6 will hide the label | |
| 46 // if the content is empty. | |
| 47 // TODO(dschuyler): rework settings checkbox to use content | |
| 48 // rather than spans. | |
| 49 this.$.checkbox.$.checkboxLabel.hidden = false; | |
| 50 }, | |
| 51 | |
| 52 toggle: function() { | 43 toggle: function() { |
| 53 this.checked = !this.checked; | 44 this.checked = !this.checked; |
| 54 }, | 45 }, |
| 55 | 46 |
| 56 /** @override */ | 47 /** @override */ |
| 57 ready: function() { | 48 ready: function() { |
| 49 this.ariaHidden = true; | |
|
Jeremy Klein
2015/09/10 21:19:51
Why this? Do we intentionally want to hide this ch
dschuyler
2015/09/10 21:23:05
That should not have been checked in. Thanks for
| |
| 58 this.$.events.forward(this.$.checkbox, ['change']); | 50 this.$.events.forward(this.$.checkbox, ['change']); |
| 59 }, | 51 }, |
| 60 }); | 52 }); |
| OLD | NEW |