Skip to content
This repository has been archived by the owner on Apr 11, 2023. It is now read-only.

Most recently used tab selector #137

Merged
merged 13 commits into from
Jan 14, 2018
Merged

Most recently used tab selector #137

merged 13 commits into from
Jan 14, 2018

Conversation

alx741
Copy link
Contributor

@alx741 alx741 commented Nov 5, 2017

Default mapping after vimperator

<C-^> === <C-6>: Select the most recently used tab

Mimics the behavior of vimperator

@alx741
Copy link
Contributor Author

alx741 commented Nov 6, 2017

Alright, the merge conflicts are now solved

@alx741 alx741 changed the title Add Last-selected tab selector Most recently used tab selector Nov 16, 2017
@alx741
Copy link
Contributor Author

alx741 commented Nov 16, 2017

Solves #164

@ueokande
Copy link
Owner

@alx741 That's nice enhanvecent, but your changes have a problem: It does not work for tabs switched by a mouse or key bindings in Firefox.

@alx741
Copy link
Contributor Author

alx741 commented Nov 19, 2017

mmh! didn't thought about that, will see if there's a way to work that out

@alx741
Copy link
Contributor Author

alx741 commented Nov 26, 2017

@ueokande Now it works as expected whenever the active tab changes, whether it is using vixen's commands/mappings, the mouse, or firefox's key bindings

@ueokande ueokande added this to the 0.9 milestone Nov 26, 2017
README.md Outdated
@@ -36,6 +36,7 @@ The default mappings are as follows:
- <kbd>u</kbd>: reopen close tab
- <kbd>K</kbd>, <kbd>J</kbd>: select prev or next tab
- <kbd>g0</kbd>, <kbd>g$</kbd>: select first or last tab
- <kbd>Ctrl</kbd>+<kbd>^</kbd>: select previous selected tab
Copy link
Owner

Choose a reason for hiding this comment

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

It differs from actual keys. Use kbd>Ctrl</kbd>+<kbd>6</kbd>.

Copy link
Owner

@ueokande ueokande left a comment

Choose a reason for hiding this comment

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

lgtm!

@ueokande ueokande merged commit 423d17a into ueokande:master Jan 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants