From 40fb74cf1802c5a77f907170c892d23596588e4a Mon Sep 17 00:00:00 2001 From: Rob Lourens Date: Tue, 19 Jun 2018 17:35:19 -0700 Subject: [PATCH] Fix #52314 - add aria label for replace toggle state --- src/vs/workbench/parts/search/browser/searchWidget.ts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/vs/workbench/parts/search/browser/searchWidget.ts b/src/vs/workbench/parts/search/browser/searchWidget.ts index becbaf33d09..8af51ede386 100644 --- a/src/vs/workbench/parts/search/browser/searchWidget.ts +++ b/src/vs/workbench/parts/search/browser/searchWidget.ts @@ -241,6 +241,7 @@ export class SearchWidget extends Widget { buttonHoverBackground: null }; this.toggleReplaceButton = this._register(new Button(parent, opts)); + this.toggleReplaceButton.element.setAttribute('aria-expanded', 'false'); this.toggleReplaceButton.icon = 'toggle-replace-button collapse'; // TODO@joh need to dispose this listener eventually this.toggleReplaceButton.onDidClick(() => this.onToggleReplaceButton()); @@ -330,6 +331,7 @@ export class SearchWidget extends Widget { dom.toggleClass(this.replaceContainer, 'disabled'); dom.toggleClass(this.toggleReplaceButton.element, 'collapse'); dom.toggleClass(this.toggleReplaceButton.element, 'expand'); + this.toggleReplaceButton.element.setAttribute('aria-expanded', this.isReplaceShown() ? 'true' : 'false'); this.updateReplaceActiveState(); this._onReplaceToggled.fire(); } -- GitLab