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

Improve keyboard navigation #56

Closed
drtebi opened this issue Mar 28, 2018 · 7 comments
Closed

Improve keyboard navigation #56

drtebi opened this issue Mar 28, 2018 · 7 comments
Projects

Comments

@drtebi
Copy link

drtebi commented Mar 28, 2018

This is a very nice add-on, I just wish a couple of keyboard actions could be improved:

  • When I use the up-arrow to go through the list and reach the search box, another up-arrow key press should go to the bottom of the tab list (the last item). There is no use for the up-arrow in the search box anyway, so I believe this is worth the effort. That way one can easily navigate the list backwards.
  • The end-key scrolls the list to the bottom, but doesn't change the selection. In my opinion, it should also select the last item in the list.
  • The same applies to the home-key
  • The page-up and page-down key scroll the list, but don't change the selection. In most applications (e.g. your file browser), the selection would also change.
  • Shift-tab should go backwards in the list (this is already filed in another issue).

These are just my improvement suggestions. The add-on itself is already very good, a bit of polishing of the keyboard navigation would make it perfect :)

@reblws reblws added this to Doing in 0.4 Mar 30, 2018
@reblws reblws moved this from Doing to Done in 0.4 Mar 30, 2018
@reblws
Copy link
Owner

reblws commented Mar 30, 2018

Hey @drtebi, thanks for your feedback, I've implemented most of your suggestions.

The only one I'm not sure about is one aspect of the Home/End changes.

Right now the way I have it set up is if either of those keys are pressed, the focus jumps to the end of the tab list regardless of whether the search input is active or not, but Home/End keys are also used to jump to the end of a line.

If the search input is active I'm thinking maybe it shouldn't skip to the end of the tablist, but should jump the cursor to the end of the search input instead.

Curious to hear your thoughts on this.

@reblws reblws mentioned this issue Mar 30, 2018
reblws added a commit that referenced this issue Mar 30, 2018
Closes #55, #56
- Changed previous tab command behavior: When the search input is active, going to the previous tab selects the last tab instead
- Improved key-handling for common system shortcuts:
  + <kbd>Shift</kbd>-<kbd>Tab</kbd> now goes to the previous item in the tab list
  + <kbd>Home</kbd> and <kbd>End</kbd> jump to the beginning or the ending of the tab list
  + <kbd>PgUp</kbd> and <kbd>PgDown</kbd> go to the previous or next tab in the tab list
@drtebi
Copy link
Author

drtebi commented Mar 31, 2018

Hi,

you are completely right, if the focus is on the search box, then Home and End should work with the text (e.g. move the cursor within the text box accordingly).

If focus is on the list, then Home and End would be nice to work as a "jump" to the top and bottom of the list.

Basically... it should be easy and quick to use the plugin without a mouse... :)

@ciufudean
Copy link

+1 for "no mouse"
what I am missing is the ability to ctrl+shift+f and then start typing in the search box
I am looking to switch to FF - I am using Quick Tabs in Chrome

@reblws
Copy link
Owner

reblws commented Apr 3, 2018

@drtebi

Thanks, I should have made it like that from the start. I'm having some issues with my PC at the moment. I'll push this change as soon as the issues are resolved.

@ciufudean

Did this break for you recently? If not, this sounds like #13 and will be fixed in a near-future Firefox update.

@ciufudean
Copy link

Nope, it is the first time I've installed the extension

@reblws
Copy link
Owner

reblws commented Apr 4, 2018

Ah ok @ciufudean, it should be fixed by Firefox 60. In the meantime, try pressing Tab as soon as you open the popup window. That should focus the search input immediately.

@reblws reblws mentioned this issue Apr 4, 2018
@reblws reblws closed this as completed in #58 Apr 4, 2018
@ciufudean
Copy link

ciufudean commented Apr 5, 2018

pressing Tab twice will do the trick ;), once for the popup window, once for the input

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

No branches or pull requests

3 participants