search

Necktwi / crontab

4 stars 1 forks source link

Default key combination #2

Open Kwadz opened 7 years ago

Kwadz commented 7 years ago

As Safari use Alt + Tab by default to switch between windows, I would suggest using Ctrl + Tab to switch between tabs like many other app do.

tolgraven commented 7 years ago

Are you sure? Very sensible key binding but I'm pretty sure it's not the default for either Move focus to next window or Cycle through windows. But it is default (and the only realistic option really) in HyperSwitch, which I would guess is very common among the kind of users in the market for keyboard-centric safari extensions. Ctrl + Tab already means next/prev tab so then you're replacing functionality instead of extending it, no go. I guess my point is it needs to be configurable. The current code doesn't exactly look setup for that, but you can edit KeyHandler.js to swap it.

Kwadz commented 7 years ago

Are you sure? Very sensible key binding but I'm pretty sure it's not the default for either Move focus to next window or Cycle through windows

You're right, the default is Cmd ⌘ + ` I forgot that I changed it...

Do you think it would take long time to make it configurable?