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

Transparent Normal Mode #194

Merged
merged 8 commits into from
Sep 1, 2020
Merged

Transparent Normal Mode #194

merged 8 commits into from
Sep 1, 2020

Conversation

nbelzer
Copy link
Collaborator

@nbelzer nbelzer commented Aug 28, 2020

Resolves #188

This PR implements a new option transparentBindings (better name suggestion welcome) that is by default set to true. When enabled this makes the isolation between normal mode and insert mode more transparent as it allows all non-Vimari bound keys to be sent to the website (as if you were in insert mode). Keys bound to Vimari will however still be isolated from the website until the user enters insert mode.

As the isolation happens on a per-key basis all keys, even if they are part of a multi-key binding and do not occur alone in any other binding, are included. This means that even though by default g doesn't do anything in Vimari it is isolated because it is part of both g+i,shift+g, and g+g.

This is aimed at improving the usability and compatibility for people upgrading to Vimari 2.1 as mentioned in #188. If people prefer the full isolation they can disable the option.

@nbelzer nbelzer self-assigned this Aug 28, 2020
@danielcompton
Copy link
Member

Just for anyone else following along, you can also disable Vimium's keybindings entirely by pressing i to enter "Insert mode", and leave insert mode by pressing esc.

@nbelzer nbelzer merged commit 1b0f4fc into master Sep 1, 2020
@nbelzer nbelzer deleted the 188-normal-mode-non-bound-keys branch September 1, 2020 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make keys not bound to anything in normal mode be usable
2 participants