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

Cannot open command palette in Theia when using Firefox #15145

Closed
3 tasks done
amisevsk opened this issue Nov 7, 2019 · 4 comments
Closed
3 tasks done

Cannot open command palette in Theia when using Firefox #15145

amisevsk opened this issue Nov 7, 2019 · 4 comments
Assignees
Labels
area/editor/theia Issues related to the che-theia IDE of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@amisevsk
Copy link
Contributor

amisevsk commented Nov 7, 2019

Describe the bug

Theia doesn't / can't bind the key combination ctrl + shift + p on firefox; attempting to use this shortcut to open the command palette just opens a new private browsing window.

Che version

  • latest 7.3.1

Steps to reproduce

Start a workspace in firefox and press ctrl+shift+p

Expected behavior

Command palette opens.

Runtime

  • Openshift 3.11

Installation method

  • che.openshift.io
@amisevsk amisevsk added kind/bug Outline of a bug - must adhere to the bug report template. area/editor/theia Issues related to the che-theia IDE of Che labels Nov 7, 2019
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Nov 7, 2019
@tsmaeder tsmaeder added severity/P2 Has a minor but important impact to the usage or development of the system. team/ide2 and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Nov 7, 2019
@tsmaeder
Copy link
Contributor

tsmaeder commented Nov 7, 2019

Added to epic #14015

@slemeur slemeur mentioned this issue Nov 8, 2019
7 tasks
@azatsarynnyy azatsarynnyy mentioned this issue Dec 18, 2019
8 tasks
@vzhukovs vzhukovs self-assigned this Dec 19, 2019
@vzhukovs vzhukovs added the status/in-progress This issue has been taken by an engineer and is under active development. label Dec 19, 2019
@vzhukovs
Copy link
Contributor

Actually there is a duplicated issue created in theia: eclipse-theia/theia#1696
Studying the mozilla sources shows that this shortcut can't be neither overwritten nor disabled.

https://hg.mozilla.org/mozilla-central/file/tip/browser/base/content/browser-sets.inc#l289

    <key id="key_privatebrowsing" command="Tools:PrivateBrowsing" key="&privateBrowsingCmd.commandkey;"
         modifiers="accel,shift" reserved="true"/>

As a workaround, which provided in eclipse-theia/theia#1696, we can rebind this shortcut to F1

@azatsarynnyy
Copy link
Member

azatsarynnyy commented Dec 20, 2019

Let me clarify.
Here're the commands for opening Command Palette in Theia and VSCode:

Theia
image

VSCode Online
image

VSCode Electron
image

As you can see, there are different hotkeys in all cases. But actually there's a common key for opening Command Palette everywhere - F1. It's supported by VSCode (electron+online) and Theia as well. It works well in all browsers. It comes from the underlying Monaco editor. VSCode just proposes alternative hotkeys - Alt+X and Ctrl+Shift+P.
BTW as I know, F1 is more know and more spread than others.

I'd say Ctrl+Shift+P it's rather a "hidden" feature in Theia since this is not declared anywhere.
I don't see a reason why a hotkey from VSCode Electron version has to be supported by Che Theia.

VSCode Electron uses Ctrl+Shift+P because it works well in Chrominum inside Electron.
But in FireFox, Ctrl+Shift+P is a reserved hotkey that can not be overridden. That's why VSCode Online introduced a completely different hotkey - Alt+X.

I believe it's not a bug since there is a key for opening Command Palette in FireFox - F1.
I'd close it. WDYT guys?
cc/ @slemeur

@vzhukovs vzhukovs removed the status/in-progress This issue has been taken by an engineer and is under active development. label Dec 23, 2019
@slemeur
Copy link
Contributor

slemeur commented Dec 23, 2019

Adding @beaumorley for awarness.

Thanks for the precise analyze and studying the other alternates. I'm +1 for closing the issue. Eventually, we could add the support for Alt+x, depending on user feedbacks. Otherwise, I like having F1 as being the hotkeys working for every browser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/editor/theia Issues related to the che-theia IDE of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

6 participants