Only add new separator when SyncSettingAction is present

This commit is contained in:
Rob Lourens
2020-02-26 17:36:35 -08:00
parent d9507c90f6
commit 6f15da251f

View File

@@ -1205,7 +1205,6 @@ export class SettingTreeRenderers {
new Separator(),
this._instantiationService.createInstance(CopySettingIdAction),
this._instantiationService.createInstance(CopySettingAsJSONAction),
new Separator(),
];
const actionFactory = (setting: ISetting) => this.getActionsForSetting(setting);
@@ -1239,7 +1238,10 @@ export class SettingTreeRenderers {
private getActionsForSetting(setting: ISetting): IAction[] {
const enableSync = this._userDataSyncEnablementService.isEnabled();
return enableSync && !setting.disallowSyncIgnore ?
[this._instantiationService.createInstance(SyncSettingAction, setting)] :
[
new Separator(),
this._instantiationService.createInstance(SyncSettingAction, setting)
] :
[];
}