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

Tree-style tab overrides Ctrl+shift+⬅️ shortcut, makes it difficult to change #1995

Closed
MikeFHay opened this issue Aug 17, 2018 · 15 comments
Labels

Comments

@MikeFHay
Copy link

Ctrl+shift+⬅️ and Ctrl+shift+➡️ are commonly-used keybindings on Windows and Linux to select a word in a textbox. A recent version of Tree-Style-Tabs overrode those keybindings resulting in surprising behaviour (changing tabs) when I tried to select some text.

Further, it is quite difficult to change or remove the keybindings. In the Keyboard Shortcuts section of TST's settings, clearing a textbox then clicking away results in no change. Clicking the reset button, clearing the textbox, then clicking away results in "undefined" in the textbox, but doesn't change the keybinding - leaving and returning to the page shows the original keybinding.

I eventually worked around this by reassigning the unwanted keybindings to infrequently-used keys.

Ideally commonly-used shortcuts would not be overridden by Tree Style Tabs, and it would be easy to unbind those keybindings which do exist.

Thanks for all your work on this extension :)

Environment

  • Platform (OS): Ubuntu
  • Version of Firefox: 61.0.1
  • Version (or revision) of Tree Style Tab: 2.5.0
@Rukt
Copy link

Rukt commented Aug 17, 2018

I'm experiencing much the same issue on Windows 10 / FF 61.0.2 / TFT 2.5.0, with the exception that I can only get "undefined" to appear against the shortcuts for "Focus to previous tab" and "...next tab" (although the shortcut continues to work anyway, as MikeFHay describes).

I would welcome the option to disable (or unbind) shortcuts globally or locally.

Otherwise I echo MikeFHay again: keep up the great work!

@elonderin
Copy link

elonderin commented Aug 17, 2018

great work with this plugin but this regression is a real showstopper for those of us that need to do some real writing of text/editing in webapps

on win 7 with FF 6.1.02

@piroor
Copy link
Owner

piroor commented Aug 17, 2018

I'm very sorry, I completely forgot those shortcuts because I don't use them. (Those shortcuts don't work well for languages without whitespaces to separate words.) By the last commit default shortcuts became Ctrl-Alt-Arrow keys instead of Ctrl-Shift-Arrow.

@piroor piroor added the fixed label Aug 17, 2018
@EricCousineau-TRI
Copy link

EricCousineau-TRI commented Aug 17, 2018

Just curious: Can I ask if the issue about changing the keyboard shortcuts will be fixed as well? (should a separate issue be filed?)
It is rather confusing / slightly confusing to change them (e.g. I can't press ESC to disable a keybinding, and I can't use something silly like Ctrl + Shift + /).

BTW Thanks for fixing this so quickly!

@dubrowgn
Copy link

I'm not sure if you are aware, but ctrl+alt+left/right are also very common linux key bindings (default on pretty much every distro I am aware of).

+1 On the ability to disable all key bindings

@piroor
Copy link
Owner

piroor commented Aug 18, 2018

Hmm, then I give up to define default shortcut keys for those features. Due to Firefox's restriction https://bugzilla.mozilla.org/show_bug.cgi?id=1475043 default shortcuts defined in manifest.json cannot be disabled.

@ibrahima
Copy link

ibrahima commented Aug 19, 2018

Yeah, I don't think default shortcuts for these functions (at least with arrow keys) are a good idea, they're going to break a lot of people's workflows. I was confused because it only happened in certain tabs, I never realized it would have been due to the parent/child relationships. But I did assume that it was due to TST because it was related to switching tabs.

For now I've just added "alt" to the hotkey to make it less likely to be unintentionally pressed. Can't unset it :(.

Edit: That still doesn't seem to work 😢

@Kissaki
Copy link

Kissaki commented Aug 19, 2018

My god this is so annoying. I see it is fixed in master. When can we expect a bugfix release?

I regularly use the shortcut to select words, and this is driving me crazy.

piroor added a commit that referenced this issue Aug 19, 2018
Due to https://bugzilla.mozilla.org/show_bug.cgi?id=1475043 we cannot clear predefined default shortcuts, and as a workaround I decided to undefine them and define them as user-defined shortcuts on the initial startup. "Restore to default shortcut" feature became unavailable anymore, but this is necessary to allow clearing default shortcuts by the user.
@piroor
Copy link
Owner

piroor commented Aug 19, 2018

By the commit a7d1dde now all of keyboard shortcuts provided by TST are became user-defined one. This means that you can disable them easily by the "Reset" button. However it also means that you cannot restore the predefined shortcut by the "Reset" button anymore. The button always clears the shortcut to blank.

@piroor
Copy link
Owner

piroor commented Aug 19, 2018

Updated: by the commit 6251feb the "Reset" button correctly restores "default" shortcut defined by me. On the other hand the "Escape" key clears the current shortcut.

@josteink
Copy link

Just forced and update and can confirm I can remove keybindings with "Esc". And with that, my Firefox is useful again.

Thanks!

@ibrahima
Copy link

Thanks for the prompt fix!

@dromer
Copy link

dromer commented Aug 20, 2018

I was going to post an issue aboutt his exact problem :D (on 2.5.0 here).

Note that on linux ctrl + alt + left/right is commonly used to switch desktops.
Of course it's nearly impossible to take in to account all possible 'common' shortcuts, but some of these 'simple' ones are typically used in various UI circumstances.

In my case the text-selection was really breaking, that and not being able to override/disable. Thnx for fixing!

@piroor
Copy link
Owner

piroor commented Oct 31, 2018

I close this issue because the default shortcut was already changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

11 participants