Commit 1149aaa9 authored by Martin Robinson's avatar Martin Robinson Committed by Commit Bot

Replace aria-active-attribute in WebUI

With the switch to cr-button from paper-button, the component no longer
has support for aria-active-attribute. Instead we simply set the
expanded value via an element attribute binding.

Bug: 967793
Change-Id: I033c025aa25160ede9271cd5e8725e61630183f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1798342Reviewed-by: default avatarEsmael Elmoslimany <aee@chromium.org>
Commit-Queue: Martin Robinson <mrobinson@igalia.com>
Cr-Commit-Position: refs/heads/master@{#695723}
parent 2dca660c
...@@ -225,7 +225,8 @@ ...@@ -225,7 +225,8 @@
$i18n{pluginVmPageTitle} $i18n{pluginVmPageTitle}
</div> </div>
</a> </a>
<cr-button id="advancedButton" aria-active-attribute="aria-expanded" <cr-button id="advancedButton"
aria-expanded$="[[boolToString_(advancedOpened)]]"
on-click="onAdvancedButtonToggle_"> on-click="onAdvancedButtonToggle_">
<span>$i18n{advancedPageTitle}</span> <span>$i18n{advancedPageTitle}</span>
<iron-icon icon="[[arrowState_(advancedOpened)]]"> <iron-icon icon="[[arrowState_(advancedOpened)]]">
......
...@@ -108,4 +108,13 @@ Polymer({ ...@@ -108,4 +108,13 @@ Polymer({
const submenu = /** @type {IronCollapseElement} */ (this.$.advancedSubmenu); const submenu = /** @type {IronCollapseElement} */ (this.$.advancedSubmenu);
return submenu.opened; return submenu.opened;
}, },
/**
* @param {boolean} bool
* @return {string}
* @private
*/
boolToString_: function(bool) {
return bool.toString();
},
}); });
...@@ -176,7 +176,8 @@ ...@@ -176,7 +176,8 @@
<iron-icon icon="settings:power-settings-new"></iron-icon> <iron-icon icon="settings:power-settings-new"></iron-icon>
$i18n{onStartup} $i18n{onStartup}
</a> </a>
<cr-button id="advancedButton" aria-active-attribute="aria-expanded" <cr-button id="advancedButton"
aria-expanded$="[[boolToString_(advancedOpened)]]"
on-click="onAdvancedButtonToggle_" on-click="onAdvancedButtonToggle_"
hidden="[[!pageVisibility.advancedSettings]]"> hidden="[[!pageVisibility.advancedSettings]]">
<span>$i18n{advancedPageTitle}</span> <span>$i18n{advancedPageTitle}</span>
......
...@@ -95,4 +95,13 @@ Polymer({ ...@@ -95,4 +95,13 @@ Polymer({
chrome.metricsPrivate.recordUserAction( chrome.metricsPrivate.recordUserAction(
'SettingsMenu_ExtensionsLinkClicked'); 'SettingsMenu_ExtensionsLinkClicked');
}, },
/**
* @param {boolean} bool
* @return {string}
* @private
*/
boolToString_: function(bool) {
return bool.toString();
},
}); });
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment