-
-
Notifications
You must be signed in to change notification settings - Fork 643
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
More debugger #1176
More debugger #1176
Conversation
Reworded. |
@@ -917,10 +917,13 @@ Keyboard shortcut | Description | |||
<kbd>c</kbd> | Continue without stopping | |||
<kbd>o</kbd> | Move out of the current sexp (like `up-list`) | |||
<kbd>i</kbd> | Inject a value into running code | |||
<kbd>e</kbd> | Eval code in current context |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Btw, I believe this is still handy as a shorter alias for C-c M-:
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just verified that in edebug both e
and M-:
behave the same way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I probably shouldn't have removed it from the readme.
The key still exists, it's just not advertised on the prompt because I wanted to reduce it a little.
Anyway, I'll add this back, but I can't do it for at least a couple of hours.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem. Regarding the prompt itself - edebug
doesn't even show it by default. Maybe we should add an option to hide it - after you've seen the keys a few times you generally don't need them as hints (and a menu would help as well, as previously discussed).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All true.
After the next batch of commits, I'll focus entirely on the interface. Then I'll add a help key, a menu, and maybe some toolbar buttons while debugging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like a plan!
Readme Fixed |
Thanks! |
See clojure-emacs/cider-nrepl#222