Commit 5acb2185 authored by John Emau's avatar John Emau Committed by Commit Bot

DevTools: Add group role to console settings checkboxes

Marked the console settings checkboxes container as a group so screen
readers will announce entering and exiting the collection of settings.

Screenshot: https://i.imgur.com/4Vp5Ypn.png
Bug: 963183
Change-Id: I017121ecd0826dcd9bd067b30d749ba41b0a1858
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1628223Reviewed-by: default avatarErik Luo <luoe@chromium.org>
Commit-Queue: John Emau <johnemau@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#665294}
parent 33fe7900
...@@ -132,6 +132,8 @@ Console.ConsoleView = class extends UI.VBox { ...@@ -132,6 +132,8 @@ Console.ConsoleView = class extends UI.VBox {
settingsPane.show(this._contentsElement); settingsPane.show(this._contentsElement);
settingsPane.element.classList.add('console-settings-pane'); settingsPane.element.classList.add('console-settings-pane');
UI.ARIAUtils.setAccessibleName(settingsPane.element, ls`Console settings`);
UI.ARIAUtils.markAsGroup(settingsPane.element);
const settingsToolbarLeft = new UI.Toolbar('', settingsPane.element); const settingsToolbarLeft = new UI.Toolbar('', settingsPane.element);
settingsToolbarLeft.makeVertical(); settingsToolbarLeft.makeVertical();
settingsToolbarLeft.appendToolbarItem(this._hideNetworkMessagesCheckbox); settingsToolbarLeft.appendToolbarItem(this._hideNetworkMessagesCheckbox);
......
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