From a6652b5df25afec4f43a4ff1c845a5e1f1f77b91 Mon Sep 17 00:00:00 2001 From: isidor Date: Wed, 19 Jun 2019 14:56:13 +0200 Subject: [PATCH] gotoErrorWidget: do not focus the error widget, but use alert to read out message fixes #41356 --- src/vs/editor/contrib/gotoError/gotoErrorWidget.ts | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts b/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts index 8f2bef9143554..968fa5d44cbf3 100644 --- a/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts +++ b/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts @@ -25,7 +25,7 @@ import { basename } from 'vs/base/common/resources'; import { IAction } from 'vs/base/common/actions'; import { IActionBarOptions, ActionsOrientation } from 'vs/base/browser/ui/actionbar/actionbar'; import { peekViewTitleForeground, peekViewTitleInfoForeground } from 'vs/editor/contrib/referenceSearch/referencesWidget'; -import { AccessibilitySupport } from 'vs/platform/accessibility/common/accessibility'; +import * as aria from 'vs/base/browser/ui/aria/aria'; import { SeverityIcon } from 'vs/platform/severityIcon/common/severityIcon'; class MessageWidget { @@ -287,9 +287,7 @@ export class MarkerNavigationWidget extends PeekViewWidget { this.editor.revealPositionInCenter(position, ScrollType.Smooth); - if (this.editor.getConfiguration().accessibilitySupport !== AccessibilitySupport.Disabled) { - this.focus(); - } + aria.alert(marker.message); } updateMarker(marker: IMarker): void {