Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Editor inputs have contributed commands in the context menu #89459

Closed
jrieken opened this issue Jan 28, 2020 · 7 comments · Fixed by #123130
Closed

Editor inputs have contributed commands in the context menu #89459

jrieken opened this issue Jan 28, 2020 · 7 comments · Fixed by #123130
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders ux User experience issues verified Verification succeeded
Milestone

Comments

@jrieken
Copy link
Member

jrieken commented Jan 28, 2020

re #89333

  • focus SCM input
  • show context menu
  • the is a entry "Command Palette..." which is awkward

image

@joaomoreno joaomoreno added bug Issue identified by VS Code Team member as probable bug scm General SCM compound issues labels Jan 28, 2020
@joaomoreno joaomoreno added this to the January 2020 milestone Jan 28, 2020
@gjsjohnmurray
Copy link
Contributor

Related to this, on my system that context menu also gives me entries that have been contributed by a couple of my extensions.

@deepak1556
Copy link
Collaborator

Same issue on macOS as well

@joaomoreno joaomoreno changed the title SCM input box context menu has command palette entry Editor inputs have contributed commands in the context menu Jan 29, 2020
@joaomoreno
Copy link
Member

This happens across all our inputs which use the Editor. IMO we should not include all those contributions and should just have Cut, Copy & Paste in that menu.

What's the simplest way towards this @alexdima?

@joaomoreno joaomoreno modified the milestones: January 2020, Backlog Jan 29, 2020
@joaomoreno joaomoreno added ux User experience issues and removed scm General SCM compound issues labels Jan 29, 2020
@alexdima
Copy link
Member

@joaomoreno The "Command Palette..." should potentially have a better context key for precondition, not undefined:

@alexdima alexdima assigned alexdima and unassigned joaomoreno Jan 29, 2020
@alexdima alexdima modified the milestones: Backlog, January 2020 Jan 29, 2020
@joaomoreno
Copy link
Member

joaomoreno commented Jan 29, 2020

@alexdima Thanks! But this is actually only half of the story. As @gjsjohnmurray mentions, extension contributed commands show up:

image

Should we file issues to these extensions, suggesting to change their when clauses?

@jrieken
Copy link
Member Author

jrieken commented Jan 29, 2020

It might be better/simpler to have another menu id, something like embeddedEditorContext or even something SCM specific

@alexdima alexdima reopened this Jan 29, 2020
@alexdima alexdima modified the milestones: January 2020, February 2020 Jan 29, 2020
@alexdima alexdima modified the milestones: February 2020, March 2020 Feb 26, 2020
@alexdima alexdima modified the milestones: March 2020, April 2020 Apr 2, 2020
@alexdima alexdima modified the milestones: April 2020, May 2020 Apr 27, 2020
@alexdima alexdima modified the milestones: May 2020, June 2020 Jun 3, 2020
@alexdima alexdima modified the milestones: June 2020, July 2020 Jun 29, 2020
@alexdima alexdima modified the milestones: July 2020, On Deck Aug 6, 2020
@alexdima
Copy link
Member

alexdima commented May 6, 2021

To reproduce, you can use:

"menus": [
    "editor/context/copy": [{
        "command": "workbench.action.toggleDevTools"
    }]
]

hediet added a commit to hediet/vscode that referenced this issue May 6, 2021
SimpleEditorContext is used instead of EditorContext for simple widget editors.
@jrieken jrieken modified the milestones: On Deck, May 2021 May 7, 2021
@lramos15 lramos15 added the verified Verification succeeded label Jun 3, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jun 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders ux User experience issues verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants
@joaomoreno @deepak1556 @jrieken @hediet @lramos15 @alexdima @gjsjohnmurray and others