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

github layer: magit keys missing from the magit help #2086

Closed
tko opened this issue Jun 23, 2015 · 4 comments
Closed

github layer: magit keys missing from the magit help #2086

tko opened this issue Jun 23, 2015 · 4 comments
Labels
Enhancement ☺ Git Key Bindings Ready for work stale marked as a stale issue/pr (usually by a bot)

Comments

@tko
Copy link
Contributor

tko commented Jun 23, 2015

github layer adds a few keybindings in magit-status-mode. However, those keys are not behaving like magit keys otherwise; they don't show up in the list of keys/commands you see with ?, when you hit the new # command/prefix key you still don't get any further guidance, neither from magit nor from guide-key.

@tko
Copy link
Contributor Author

tko commented Jun 23, 2015

Something along the lines of

(magit-key-mode-insert-action 'dispatch (kbd "#") "GitHub-PR" 'magit-key-mode-popup-github)

might work (ref: emacsorphanage/magit-gerrit#17) .. might need to implement the popup as well, I guess.

@TheBB
Copy link
Collaborator

TheBB commented Oct 26, 2015

The proposed code doesn't work in the new version of Magit, but you can use magit-define-popup-key and magit-change-popup-key to get the same effect.

Unfortunately it's not so clear how to do this in a way that works with both evilified and emacs state. Simplest would be to go with one of them on startup and say that if you switch while Spacemacs is running, that's your own problem.

@tarsius
Copy link
Contributor

tarsius commented Aug 5, 2016

those keys [...] don't show up in the list of keys/commands you see with ?

Since this doesn't affect only Spacemacs, the respective bindings should be established in the upstream packages.

For magit-gh-pulls I have done so at sigma/magit-gh-pulls#96. Someone(tm) please do that for the other extensions. See that pr for how to do it.

@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
Enhancement ☺ Git Key Bindings Ready for work stale marked as a stale issue/pr (usually by a bot)
Projects
None yet
Development

No branches or pull requests

6 participants