Skip to content

Commit

Permalink
gotoErrorWidget: do not focus the error widget, but use alert to read…
Browse files Browse the repository at this point in the history
… out message

fixes #41356
  • Loading branch information
isidorn committed Jun 19, 2019
1 parent a2a9aeb commit a6652b5
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/vs/editor/contrib/gotoError/gotoErrorWidget.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down Expand Up @@ -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 {
Expand Down

0 comments on commit a6652b5

Please sign in to comment.