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

Disable Alt-key for toggle menu when another keybinding is provided #41070

Closed
AmitJoki opened this issue Jan 3, 2018 · 7 comments
Closed

Disable Alt-key for toggle menu when another keybinding is provided #41070

AmitJoki opened this issue Jan 3, 2018 · 7 comments
Assignees
Labels
electron Issues and items related to Electron feature-request Request for new features or functionality info-needed Issue requires more information from poster upstream Issue identified as 'upstream' component related (exists outside of VS Code)

Comments

@AmitJoki
Copy link

AmitJoki commented Jan 3, 2018

  • VSCode Version: The latest one as of writing this
  • OS Version: Windows 10 Enterprise

Steps to Reproduce:

  1. Have some keybindings with Alt
  2. Put workspace.editor.menuBarVisibility: "hidden"
  3. Assign some other key to toggle menu bar in case you need it. I assigned [Pause Break] key
  4. Use [Pause Break] key or whatever that is you've assigned to toggle menu bar.
  5. A pop up shows [Info: You can also use Alt to toggle menu bar]

That is annoying because Alt is still being considered for toggling the menu bar when I want it to overwrite with some other key.

This also causing annoyance when I press Alt + Tab to go to other programs. If I don't press Alt and Tab fast enough, the menu bar toggles itself into view. The same is with the keybindings with Alt I said in the first step of the reproduction.

Reproduces without extensions: Yes

@bpasero bpasero added feature-request Request for new features or functionality and removed workbench labels Jan 3, 2018
@bpasero bpasero removed their assignment Jan 5, 2018
@lyallh
Copy link

lyallh commented Mar 7, 2018

Discussed in #35010

@bpasero
Copy link
Member

bpasero commented Mar 19, 2018

@AmitJoki @lyallh I think you will have to report this against Electron, there is no control (as far as I know) from VSCode to control this.

@bpasero bpasero added upstream Issue identified as 'upstream' component related (exists outside of VS Code) electron Issues and items related to Electron labels Mar 19, 2018
@bpasero
Copy link
Member

bpasero commented Mar 24, 2018

@AmitJoki @lyallh can you please check this build (https://az764295.vo.msecnd.net/exploration/b1513227ace6929e278c5d531250bc1d0cf89e2f/VSCode-win32-x64-1.22.0-exploration.zip) to see if the issue persists. It includes the fix for #35010

@lyallh
Copy link

lyallh commented Mar 24, 2018

@bpasero I'm runing linux-x64 not win. Is it just my upstream electron fix?

@bpasero
Copy link
Member

bpasero commented Mar 24, 2018

@bpasero
Copy link
Member

bpasero commented Aug 6, 2018

@AmitJoki @lyallh can you please try out from our latest insider builds if you still think this bug is valid? We updated to Electron 2.0.x meanwhile.

You can give our preview releases a try from: https://code.visualstudio.com/insiders/

@bpasero bpasero self-assigned this Aug 6, 2018
@bpasero bpasero added the info-needed Issue requires more information from poster label Aug 6, 2018
@vscodebot vscodebot bot closed this as completed Aug 13, 2018
@vscodebot
Copy link

vscodebot bot commented Aug 13, 2018

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot locked and limited conversation to collaborators Sep 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
electron Issues and items related to Electron feature-request Request for new features or functionality info-needed Issue requires more information from poster upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests

3 participants