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

Cannot move down with C-n in Magit #2277

Closed
tuhdo opened this issue Jul 7, 2015 · 6 comments
Closed

Cannot move down with C-n in Magit #2277

tuhdo opened this issue Jul 7, 2015 · 6 comments

Comments

@tuhdo
Copy link
Contributor

tuhdo commented Jul 7, 2015

magit-section-forward--evilified-magit-mode-map-C-n is bound to C-n, but pressing it in Magit does not go to next line with Emacs style, while it's supposed to be.

@syl20bnr
Copy link
Owner

syl20bnr commented Jul 9, 2015

Can you confirm that it works inside sections ? Is this limited to part of the buffer with no section ?

@syl20bnr
Copy link
Owner

Actually this is the expected behavior, you don't need C-n in holy-mode, it is just n. In emacs state the stock magit key bindings just works.
Now I realize that the remapping creates a lot of noise with SPC ? so I should find better function names.

Right now the function names have 2 possible forms:

  • x-or-y--evilified... where x is for emacs state (or any other state which is not evilified) and y is for evilified.
  • y-evilified... where y is for evilified and y does nothing in other state.

Not easy to find some naming rules to make it clear.

Now I'm aware of this I'll be able to think about a better way to handle the remapping. I think I will store the stock unmodified maps in order to use/restore them in emacs state, and put some function to convert them on the fly at editing style switching time and initialization time.
May be quite some work though.

@tuhdo
Copy link
Contributor Author

tuhdo commented Jul 10, 2015

But C-p works normal. Without it, I cannot move down a line inside the hunk, since n skips to the next hunk.

@ozanmakes
Copy link

Not being able to use C-n is quite serious since it is used to mark and stage individual lines, which is a fundamental part of magit workflow.

Btw, C-v is also broken.

@sixpi
Copy link

sixpi commented Aug 11, 2015

This is broken for me too.

@justbur
Copy link
Contributor

justbur commented Oct 28, 2015

After #3578, magit will no longer be evilified and this should be resolved

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

7 participants