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

Unified Diff: third_party/polymer/v1_0/components-chromium/paper-radio-button/paper-radio-button-extracted.js

Issue 1336623003: [MD settings] updating polymer to 1.1.13 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: changed Polymer.IronCheckedElementBehavior name Created 5 years, 3 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: third_party/polymer/v1_0/components-chromium/paper-radio-button/paper-radio-button-extracted.js
diff --git a/third_party/polymer/v1_0/components-chromium/paper-radio-button/paper-radio-button-extracted.js b/third_party/polymer/v1_0/components-chromium/paper-radio-button/paper-radio-button-extracted.js
index 9fffb2f13dee0ecf615e733e92629d3f00d768ac..e88bcbbd801d07a4df92bd4948b21e4a7d5a0c20 100644
--- a/third_party/polymer/v1_0/components-chromium/paper-radio-button/paper-radio-button-extracted.js
+++ b/third_party/polymer/v1_0/components-chromium/paper-radio-button/paper-radio-button-extracted.js
@@ -2,7 +2,8 @@ Polymer({
is: 'paper-radio-button',
behaviors: [
- Polymer.PaperInkyFocusBehavior
+ Polymer.PaperInkyFocusBehavior,
+ Polymer.IronCheckedElementBehavior
],
hostAttributes: {
@@ -24,38 +25,30 @@ Polymer({
* @event iron-change
*/
- /**
- * Gets or sets the state, `true` is checked and `false` is unchecked.
- */
- checked: {
- type: Boolean,
- value: false,
- reflectToAttribute: true,
- notify: true,
- observer: '_checkedChanged'
- },
-
- /**
- * If true, the button toggles the active state with each tap or press
- * of the spacebar.
- */
- toggles: {
- type: Boolean,
- value: true,
- reflectToAttribute: true
- }
+ ariaActiveAttribute: {
+ value: 'aria-checked'
+ }
},
attached: function() {
- var trimmedText = Polymer.dom(this).textContent.trim();
- if (trimmedText === '') {
- this.$.radioLabel.hidden = true;
- }
+ this._isReady = true;
+
// Don't stomp over a user-set aria-label.
- if (trimmedText !== '' && !this.getAttribute('aria-label')) {
- this.setAttribute('aria-label', trimmedText);
+ if (!this.getAttribute('aria-label')) {
+ this.updateAriaLabel();
}
- this._isReady = true;
+ },
+
+ /**
+ * Update the checkbox aria-label. This is a temporary workaround not
+ * being able to observe changes in <content>
+ * (see: https://github.com/Polymer/polymer/issues/1773)
+ *
+ * Call this if you manually change the contents of the checkbox
+ * and want the aria-label to match the new contents.
+ */
+ updateAriaLabel: function() {
+ this.setAttribute('aria-label', Polymer.dom(this).textContent.trim());
},
_buttonStateChanged: function() {
@@ -65,11 +58,5 @@ Polymer({
if (this._isReady) {
this.checked = this.active;
}
- },
-
- _checkedChanged: function() {
- this.setAttribute('aria-checked', this.checked ? 'true' : 'false');
- this.active = this.checked;
- this.fire('iron-change');
}
});

Powered by Google App Engine
This is Rietveld 408576698