use affectsConfiguration (#169919)

related to #169806
microsoft · Dec 23, 2022 · 27a911a · 27a911a
1 parent 7f65f02
commit 27a911a
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/vs/platform/list/browser/listService.ts b/src/vs/platform/list/browser/listService.ts
@@ -683,8 +683,10 @@ abstract class ResourceNavigator<T> extends Disposable {
 
  if (typeof options?.openOnSingleClick !== 'boolean' && options?.configurationService) {
  this.openOnSingleClick = options?.configurationService!.getValue(openModeSettingKey) !== 'doubleClick';
- this._register(options?.configurationService.onDidChangeConfiguration(() => {
- this.openOnSingleClick = options?.configurationService!.getValue(openModeSettingKey) !== 'doubleClick';
+ this._register(options?.configurationService.onDidChangeConfiguration(e => {
+ if (e.affectsConfiguration(openModeSettingKey)) {
+ this.openOnSingleClick = options?.configurationService!.getValue(openModeSettingKey) !== 'doubleClick';
+ }
  }));
  } else {
  this.openOnSingleClick = options?.openOnSingleClick ?? true;