Skip to content

Commit aa2068d

Browse files
authored
signals.chatResponsePending -> singals.progress (microsoft#210659)
fix microsoft#210590
1 parent e20f630 commit aa2068d

File tree

4 files changed

+27
-15
lines changed

4 files changed

+27
-15
lines changed

src/vs/platform/accessibilitySignal/browser/accessibilitySignalService.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,6 @@ export class Sound {
311311
public static readonly diffLineDeleted = Sound.register({ fileName: 'diffLineDeleted.mp3' });
312312
public static readonly diffLineModified = Sound.register({ fileName: 'diffLineModified.mp3' });
313313
public static readonly chatRequestSent = Sound.register({ fileName: 'chatRequestSent.mp3' });
314-
public static readonly chatResponsePending = Sound.register({ fileName: 'chatResponsePending.mp3' });
315314
public static readonly chatResponseReceived1 = Sound.register({ fileName: 'chatResponseReceived1.mp3' });
316315
public static readonly chatResponseReceived2 = Sound.register({ fileName: 'chatResponseReceived2.mp3' });
317316
public static readonly chatResponseReceived3 = Sound.register({ fileName: 'chatResponseReceived3.mp3' });
@@ -321,6 +320,7 @@ export class Sound {
321320
public static readonly format = Sound.register({ fileName: 'format.mp3' });
322321
public static readonly voiceRecordingStarted = Sound.register({ fileName: 'voiceRecordingStarted.mp3' });
323322
public static readonly voiceRecordingStopped = Sound.register({ fileName: 'voiceRecordingStopped.mp3' });
323+
public static readonly progress = Sound.register({ fileName: 'progress.mp3' });
324324

325325
private constructor(public readonly fileName: string) { }
326326
}
@@ -359,7 +359,7 @@ export const enum AccessibilityAlertSettingId {
359359
OnDebugBreak = 'accessibility.alert.onDebugBreak',
360360
NoInlayHints = 'accessibility.alert.noInlayHints',
361361
LineHasBreakpoint = 'accessibility.alert.lineHasBreakpoint',
362-
ChatResponsePending = 'accessibility.alert.chatResponsePending'
362+
Progress = 'accessibility.alert.chatResponseProgress'
363363
}
364364

365365

@@ -553,13 +553,13 @@ export class AccessibilitySignal {
553553
settingsKey: 'accessibility.signals.chatResponseReceived'
554554
});
555555

556-
public static readonly chatResponsePending = AccessibilitySignal.register({
557-
name: localize('accessibilitySignals.chatResponsePending', 'Chat Response Pending'),
558-
sound: Sound.chatResponsePending,
556+
public static readonly progress = AccessibilitySignal.register({
557+
name: localize('accessibilitySignals.progress', 'Progress'),
558+
sound: Sound.progress,
559559
legacySoundSettingsKey: 'audioCues.chatResponsePending',
560-
legacyAnnouncementSettingsKey: AccessibilityAlertSettingId.ChatResponsePending,
561-
announcementMessage: localize('accessibility.signals.chatResponsePending', 'Chat Response Pending'),
562-
settingsKey: 'accessibility.signals.chatResponsePending'
560+
legacyAnnouncementSettingsKey: AccessibilityAlertSettingId.Progress,
561+
announcementMessage: localize('accessibility.signals.progress', 'Progress'),
562+
settingsKey: 'accessibility.signals.progress'
563563
});
564564

565565
public static readonly clear = AccessibilitySignal.register({

src/vs/platform/accessibilitySignal/browser/progressAccessibilitySignalScheduler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ export class AccessibilityProgressSignalScheduler extends Disposable {
1818
constructor(msDelayTime: number, msLoopTime: number | undefined, @IAccessibilitySignalService private readonly _accessibilitySignalService: IAccessibilitySignalService) {
1919
super();
2020
this._scheduler = new RunOnceScheduler(() => {
21-
this._signalLoop = this._accessibilitySignalService.playSignalLoop(AccessibilitySignal.chatResponsePending, msLoopTime ?? PROGRESS_SIGNAL_LOOP_DELAY);
21+
this._signalLoop = this._accessibilitySignalService.playSignalLoop(AccessibilitySignal.progress, msLoopTime ?? PROGRESS_SIGNAL_LOOP_DELAY);
2222
}, msDelayTime);
2323
this._scheduler.schedule();
2424
}

src/vs/workbench/contrib/accessibility/browser/accessibilityConfiguration.ts

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -257,8 +257,8 @@ const configuration: IConfigurationNode = {
257257
'markdownDescription': localize('announcement.chatRequestSent', "Indicates when a chat request is sent. Also see {0}.", '`#audioCues.chatRequestSent#`'),
258258
...baseAlertProperty
259259
},
260-
[AccessibilityAlertSettingId.ChatResponsePending]: {
261-
'markdownDescription': localize('announcement.chatResponsePending', "Indicates when a chat response is pending. Also see {0}.", '`#audioCues.chatResponsePending#`'),
260+
[AccessibilityAlertSettingId.Progress]: {
261+
'markdownDescription': localize('announcement.progress', "Indicates when a chat response is pending. Also see {0}.", '`#audioCues.chatResponsePending#`'),
262262
...baseAlertProperty
263263
},
264264
[AccessibilityAlertSettingId.NoInlayHints]: {
@@ -540,16 +540,16 @@ const configuration: IConfigurationNode = {
540540
},
541541
}
542542
},
543-
'accessibility.signals.chatResponsePending': {
543+
'accessibility.signals.progress': {
544544
...signalFeatureBase,
545-
'description': localize('accessibility.signals.chatResponsePending', "Plays a signal on loop while the response is pending."),
545+
'description': localize('accessibility.signals.progress', "Plays a signal on loop while progress is occurring."),
546546
'properties': {
547547
'sound': {
548-
'description': localize('accessibility.signals.chatResponsePending.sound', "Plays a sound on loop while the response is pending."),
548+
'description': localize('accessibility.signals.progress.sound', "Plays a sound on loop while progress is occurring."),
549549
...soundFeatureBase
550550
},
551551
'announcement': {
552-
'description': localize('accessibility.signals.chatResponsePending.announcement', "Alerts on loop while the response is pending."),
552+
'description': localize('accessibility.signals.progress.announcement', "Alerts on loop while progress is occurring."),
553553
...announcementFeatureBase
554554
},
555555
},
@@ -790,6 +790,18 @@ Registry.as<IConfigurationMigrationRegistry>(WorkbenchExtensions.ConfigurationMi
790790
}
791791
}]);
792792

793+
794+
Registry.as<IConfigurationMigrationRegistry>(WorkbenchExtensions.ConfigurationMigration)
795+
.registerConfigurationMigrations([{
796+
key: 'accessibility.signals.chatResponsePending',
797+
migrateFn: (value, accessor) => {
798+
return [
799+
['accessibility.signals.progress', { value }],
800+
['accessibility.signals.chatResponsePending', { value: undefined }],
801+
];
802+
}
803+
}]);
804+
793805
Registry.as<IConfigurationMigrationRegistry>(WorkbenchExtensions.ConfigurationMigration)
794806
.registerConfigurationMigrations(AccessibilitySignal.allAccessibilitySignals.map(item => ({
795807
key: item.legacySoundSettingsKey,

0 commit comments

Comments
 (0)