Skip to content

Commit 85e1bb0

Browse files
authored
Fix code action list showing disabled code action (microsoft#166418)
Looks like this was introduced when splitting out the action widget
1 parent 7d6c575 commit 85e1bb0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/vs/platform/actionWidget/browser/actionWidget.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,6 @@ export class ActionWidgetService extends Disposable implements IActionWidgetServ
203203
async show(user: string, toMenuItems: (inputQuickFixes: readonly IActionItem[], showHeaders: boolean) => IListMenuItem<IActionItem>[], delegate: IRenderDelegate<any>, actions: ActionSet<any>, anchor: IAnchor, container: HTMLElement | undefined, options: IActionShowOptions, resolver?: IActionKeybindingResolver): Promise<void> {
204204
this._currentShowingContext = undefined;
205205
const visibleContext = ActionWidgetContextKeys.Visible.bindTo(this._contextKeyService);
206-
const list = this._instantiationService.createInstance(ActionList, user, actions.allActions, true, delegate, resolver, toMenuItems);
207206

208207
const actionsToShow = options.includeDisabledActions && (this._showDisabled || actions.validActions.length === 0) ? actions.allActions : actions.validActions;
209208
if (!actionsToShow.length) {
@@ -213,6 +212,7 @@ export class ActionWidgetService extends Disposable implements IActionWidgetServ
213212

214213
this._currentShowingContext = { user, toMenuItems, delegate, actions, anchor, container, options, resolver };
215214

215+
const list = this._instantiationService.createInstance(ActionList, user, actionsToShow, true, delegate, resolver, toMenuItems);
216216
this.contextViewService.showContextView({
217217
getAnchor: () => anchor,
218218
render: (container: HTMLElement) => {

0 commit comments

Comments
 (0)