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

Spacemacs losing key binding for discard region in magit with Vim/Hybrid mode #6668

Closed
tuhdo opened this issue Jul 25, 2016 · 7 comments
Closed
Labels
- Bug tracker - Git stale marked as a stale issue/pr (usually by a bot) To be reproduced
Projects

Comments

@tuhdo
Copy link
Contributor

tuhdo commented Jul 25, 2016

One useful feature of Magit is that you can mark a region in your hunk and discard it. The current key bindings prevent me from doing so, because it is assigned to other magit commands.

@TheBB
Copy link
Collaborator

TheBB commented Jul 25, 2016

Which editing style? (If only you had used SPC h I...)

@nixmaniack
Copy link
Contributor

In hybrid I use x for discarding the marked region and I think it should be same for vim as we use evil-magit by default.

@nixmaniack
Copy link
Contributor

Bindings for evil-magit were discussed and finalized at emacs-evil/evil-magit#2 (comment)

@tuhdo
Copy link
Contributor Author

tuhdo commented Jul 26, 2016

But I could not move forward/backward using j and k or w and b to mark a region.

@nixmaniack
Copy link
Contributor

j and k are rightly bound to up/down movement for me which I think is the evil-magit default. But for horizontal movement, you need to set evil-magit-want-horizontal-movement to t which enables h/l for horizontal movement.

If j/k are not working for down/up movement(which should btw), then it might be the case spacemacs is doing something and I would suspect hybrid-mode-enable-hjkl-bindings from my spacemacs config for hybrid style.

Since selection in magit is only linewise there wasn't any need of having horizontal movement functionality but with evil-magit-want-horizontal-movement you can get minimal characterwise horizontal movement.

@TheBB
Copy link
Collaborator

TheBB commented Aug 3, 2016

I think we have answered everything here. Can this be closed @tuhdo?

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please let us know if this issue is still valid!

@github-actions github-actions bot added the stale marked as a stale issue/pr (usually by a bot) label Feb 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- Bug tracker - Git stale marked as a stale issue/pr (usually by a bot) To be reproduced
Projects
Bugs
Not reproduced
Development

No branches or pull requests

3 participants