-
Notifications
You must be signed in to change notification settings - Fork 9
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
Minor mode support. #8
Comments
Sorry for the late response. Somehow I didn't receive notification for this. That's an interesting idea. I never thought about it and don't have a use-case for it. I'm also not sure how it's gonna work because for a given buffer, multiple minor modes might be enabled. How should it decide which "minor mode hydra" to open when you invoke the command? |
I never really ran into that issue since I typically used it for modes that don't conflict but I suppose there could be a priority list in the config that determines the precedence of each mode. |
I don't know if I'd implement exactly what you want at this point. But I do have something in mind that will make it easier to achieve what you want. Basically I plan to introduce a new command where you can specify how you'd want to dispatch to different hydras. |
@WolfeCub On It should allow you to do something like this: (major-mode-hydra-bind foobar-mode "Eval"
("b" eval-buffer "buffer")
("e" eval-defun "defun")
("r" eval-region "region"))
(major-mode-hydra-bind foobar-mode "Test"
("t" ert "prompt")
("T" (ert t) "all")
("F" (ert :failed) "failed"))
(major-mode-hydra-bind foobar-mode "Doc"
("d" describe-foo-at-point "thing-at-pt")
("f" describe-function)
("v" describe-variable))
(defun call-foobar-hydra ()
(interactive)
(major-mode-hydra-dispatch 'foobar-mode)) You can create commands on top of that function to do your own dispatching. |
Great! Thanks for the follow up. |
I know this goes against the package name but is there any reason you don't support minor modes?
I have similar functionality to this package in my dotfiles but when I switched over to this package I noticed it was missing minor mode support.
Is there any chance of this being added?
Thanks!
The text was updated successfully, but these errors were encountered: