Skip to content

Commit 9fea8e2

Browse files
authored
Focusing previously focused notification does not announce their content (fix microsoft#188762) (microsoft#188936)
1 parent f7bffd7 commit 9fea8e2

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

src/vs/workbench/browser/parts/notifications/notificationsCenter.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,10 @@ export class NotificationsCenter extends Themable implements INotificationsCente
7878
show(): void {
7979
if (this._isVisible) {
8080
const notificationsList = assertIsDefined(this.notificationsList);
81+
82+
// Make visible and focus first
8183
notificationsList.show(true /* focus */);
84+
notificationsList.focusFirst();
8285

8386
return; // already visible
8487
}

src/vs/workbench/browser/parts/notifications/notificationsCommands.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ export function registerNotificationCommands(center: INotificationsCenterControl
104104
});
105105

106106
// Toggle Notifications Center
107-
CommandsRegistry.registerCommand(TOGGLE_NOTIFICATIONS_CENTER, accessor => {
107+
CommandsRegistry.registerCommand(TOGGLE_NOTIFICATIONS_CENTER, () => {
108108
if (center.isVisible) {
109109
center.hide();
110110
} else {

0 commit comments

Comments
 (0)