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

Unified Diff: third_party/WebKit/Source/devtools/front_end/ui/TabbedPane.js

Issue 2655393003: DevTools: Introduce ARIAUtils (Closed)
Patch Set: setAccessibleName in InspectorView Created 3 years, 11 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/WebKit/Source/devtools/front_end/ui/TabbedPane.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/TabbedPane.js b/third_party/WebKit/Source/devtools/front_end/ui/TabbedPane.js
index c8a3456c35b8e2c0b0e955021e91eaf1b5469d0d..e3dff50bbfe3d674745788a93514f9130649fc04 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/TabbedPane.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/TabbedPane.js
@@ -40,7 +40,6 @@ UI.TabbedPane = class extends UI.VBox {
this.contentElement.tabIndex = -1;
this._headerElement = this.contentElement.createChild('div', 'tabbed-pane-header');
this._headerContentsElement = this._headerElement.createChild('div', 'tabbed-pane-header-contents');
- this._headerContentsElement.setAttribute('aria-label', Common.UIString('Panels'));
this._tabSlider = createElementWithClass('div', 'tabbed-pane-tab-slider');
this._tabsElement = this._headerContentsElement.createChild('div', 'tabbed-pane-header-tabs');
this._tabsElement.setAttribute('role', 'tablist');
@@ -61,6 +60,13 @@ UI.TabbedPane = class extends UI.VBox {
}
/**
+ * @param {string} name
+ */
+ setAccessibleName(name) {
+ UI.ARIAUtils.setAccessibleName(this._tabsElement, name);
+ }
+
+ /**
* @param {boolean} locked
*/
setCurrentTabLocked(locked) {
@@ -753,7 +759,7 @@ UI.TabbedPane = class extends UI.VBox {
*/
_showTab(tab) {
tab.tabElement.classList.add('selected');
- tab.tabElement.setAttribute('aria-selected', 'true');
+ UI.ARIAUtils.setSelected(tab.tabElement, true);
tab.view.show(this.element);
this._updateTabSlider();
}
@@ -1023,8 +1029,8 @@ UI.TabbedPaneTab = class {
var tabElement = createElementWithClass('div', 'tabbed-pane-header-tab');
tabElement.id = 'tab-' + this._id;
tabElement.tabIndex = -1;
- tabElement.setAttribute('role', 'tab');
- tabElement.setAttribute('aria-selected', 'false');
+ UI.ARIAUtils.markAsTab(tabElement);
+ UI.ARIAUtils.setSelected(tabElement, false);
tabElement.selectTabForTest = this._tabbedPane.selectTab.bind(this._tabbedPane, this.id, true);
var titleElement = tabElement.createChild('span', 'tabbed-pane-header-tab-title');

Powered by Google App Engine
This is Rietveld 408576698