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

Buffers micro-state vim like bindings #4778

Closed
Dietr1ch opened this issue Jan 24, 2016 · 6 comments
Closed

Buffers micro-state vim like bindings #4778

Dietr1ch opened this issue Jan 24, 2016 · 6 comments

Comments

@Dietr1ch
Copy link
Contributor

I feel that mnemonics for [n]next and [p]previous should be dropped for [j]next [p]previous. Even while it goes a bit against an easy way to remember, it uses your vim muscle memory for movement keys (bonus j/k are next to each other on qwerty and dvorak).

Maybe this change should be applied only if the user is using evil-mode.
There are probably a few more microstates that go through lists, they should be modified too to keep consistency.

@Andre0991
Copy link
Contributor

j and k are vertical movements. I think it would be weird to use them to go to, for example, the next and previous layouts, as it's a kind of horizontal movement. I think n and p are fine. Perhaps h and l, for that matter.

@justbur
Copy link
Contributor

justbur commented Feb 1, 2016

The convention used in some other transient states is to bind p and N to "previous" while keeping n for next

@StreakyCobra
Copy link
Contributor

I think n and p are used a lot through Emacs and Spacemacs, and not only in micro/transient-state. For me we should keep these ones, but if you want to make (and maintain after) a layer that change these bindings, feel free to contribute 😃

@Dietr1ch
Copy link
Contributor Author

You are right, it seems that n/p for iterating is way too deep into emacs, so changing it may break many workflows (relevant xkcd).

I'll review the bépo layer and start a new one. I might be in need of an automated way to find n/p usages, theoretically most of the changes can be done automatically, only requiring review on bind collisions, but an explicit remap might be good enough.

@StreakyCobra
Copy link
Contributor

I don't know if bepo will be really useful for this, as usages are listed manually and it doesn't deal with transient states for now.

Also, maybe before going too far into making it a layer, ask @syl20bnr if he is ready to merge this kind of changes.

@Dietr1ch
Copy link
Contributor Author

It seems that n/N/p is actually better. I'll get used to it eventually.

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

No branches or pull requests

4 participants