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

Shortcuts for Next/Prev Tab Navigation #3599

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

normaltaro
Copy link

@normaltaro normaltaro commented Dec 11, 2024

Added 2 shortcuts for tab navigation.

  1. Switching to Next Tab (Unbounded by default)
  2. Switching to Previous Tab (Unbounded by default)

Using extensions such as Tridactyl or Vimium to do this stops working on protected pages such as New Tab, Settings, etc and breaks the flow. But this keyboard shortcut will work on any opened tab. It is a better user experience to those who really like the keyboard.
Relevant Issue: #1678

I know English. I have submitted the relevant PR to the l10n repo: zen-browser/l10n-packs#110

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. Feature labels Dec 11, 2024
@normaltaro normaltaro changed the title Shortcuts for Tab Navigation Prev/Next Shortcuts for Next/Prev Tab Navigation Dec 11, 2024
@mauro-balades
Copy link
Member

Great!

But we'll need to disable the other shortcuts.. Ill do that when I get home and ill merge

Copy link
Member

@mauro-balades mauro-balades left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you update LATEST_KBS_VERSION?

@normaltaro
Copy link
Author

done

@normaltaro
Copy link
Author

normaltaro commented Dec 16, 2024

do i move LATEST_KBS_VERSION to 6?

@goosethedev
Copy link

Any updates on this? 👀

@mauro-balades
Copy link
Member

Any updates on this? 👀

This ended up being not as easy as it looks... Im still investigating

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants