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

Expose buttons as `interactive' commands #261

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

nbfalcon
Copy link
Contributor

I often found it useful to view a symbol in the manual, jump to its source etc.,
but navigating to the corresponding buttons was cumbersome. As such, expose them
as interactive commands, with mnemoic keybindings beginning with C-c:

  • helpful-view-in-manual (C-c m)
  • helpful-goto-source (C-c s)
  • helpful-toggle-edebug (C-c e)
  • helpful-toggle-tracing (C-c t)

I often found it useful to view a symbol in the manual, jump to its source etc.,
but navigating to the corresponding buttons was cumbersome. As such, expose them
as `interactive' commands: `helpful-view-in-manual', ....

Refactor the "toggle tracing" code into a separate function, since it used by
the toggle trace button and its corresponding `interactive' command.
`nil' is also a symbol.
@nbfalcon nbfalcon force-pushed the feature/button-commands branch 2 times, most recently from f32a9d8 to 6a50593 Compare March 12, 2021 07:41
Store the source sexp in helpful buffers, since it is needed by
show-callees.

This should now be all buttons.
@nbfalcon
Copy link
Contributor Author

This PR should now cover all buttons. There is one question though: helpful--show-callees (now helpful--show-callees-1), enables helpful-mode on the output buffer. However, this enables all the command keybindings (toggle edebug, ...) which don't work because the buffer doesn't set helpful--sym etc. Do you think that function should active a different major-mode, or to rather set helpful--sym, etc so that the commands can work even in show-callees buffers?

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

Successfully merging this pull request may close these issues.

None yet

1 participant