Skip to content

Commit ad758a4

Browse files
Fix unintended hiding of inline actions on SCM Changes view (fix microsoft#272716) (microsoft#272717)
1 parent b8f5add commit ad758a4

File tree

1 file changed

+3
-2
lines changed
  • src/vs/workbench/contrib/scm/browser

1 file changed

+3
-2
lines changed

src/vs/workbench/contrib/scm/browser/menus.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -348,10 +348,11 @@ export class SCMMenus implements ISCMMenus, IDisposable {
348348
// In order to keep the Repositories view clean, we hide the
349349
// primary actions by default. Users can always promote actions
350350
// from the `...` menu to inline actions.
351+
let itemToAppend = menuItem;
351352
if (isIMenuItem(menuItem) && menuItem.group === 'navigation') {
352-
menuItem.isHiddenByDefault = true;
353+
itemToAppend = { ...menuItem, isHiddenByDefault: true };
353354
}
354-
this.repositoryMenuDisposables.add(MenuRegistry.appendMenuItem(MenuId.SCMSourceControlInline, menuItem));
355+
this.repositoryMenuDisposables.add(MenuRegistry.appendMenuItem(MenuId.SCMSourceControlInline, itemToAppend));
355356
}
356357
}));
357358
}

0 commit comments

Comments
 (0)