Jed, I have time now to review your work and of course Flam's. I'm -1 on
renaming shortcuts, because Spyder 2 has been around for almost two
years and that's probably going to break some people's workflow. Besides
that, this is really a great addition to the project.
Let's continue the discussion on issue 554.
Cheers,
Carlos
El 06/06/12 10:15, Jed Ludlow escribió:
I created a clone last week that has all the recent debugging features merged:
http://code.google.com/r/jedludlow-spyderlib-default-merged/
On 6/6/12, Steve<[email protected]> wrote:
The functionality seems to be working great. Any chance you can merge the
breakpoint widget into this same bookmark? Or what's the easiest way for
me to get it and merge it in?
On Friday, June 1, 2012 2:03:07 PM UTC-5, Jed Ludlow wrote:
It's common to use function keys as hot keys for the basic debugging
commands as Anatoly pointed out. If there really is enough demand for
debugging hot keys to be somewhat inline with the model of other IDEs I
guess we could reassign some of the less commonly used function keys to
debugging commands, maybe in the F10-F12 range. Thoughts?
As a quick experiment, I decided to adopt a hot key mapping convention
similar to Visual Studio, and I've pushed a prototype implementation to my
clone here:
http://code.google.com/r/jedludlow-spyderlib-default/source/browse?name=jedludlow-issue-554
Here are the new hot key mappings:
F10: Debug step over
F11: Debug step into
Shift+F11: Debug step return
F12: Debug continue
Shift+F12: Debug exit
In the process, these got new mappings:
Ctrl+F10: Run profiler
Ctrl+F11: Full screen mode
Ctrl+F12: Add breakpoint
Ctrl+Shift+12: Add conditional breakpoint
It's not really straightforward to reuse F5 for continue since we are
modeless and F5 already has an assignment to run the currently active
editor script. It may be possible to get fancier with this, but for now
the
simplest approach was to use F12 for continue.
I'm publishing these changes to my clone simply for testing purposes, and
I would recommend broader discussion on the implications before it gets
pulled to the default repo. In that light, if there are strong opinions on
this, please dive in, test, and make recommendations.
--
You received this message because you are subscribed to the Google Groups
"spyder" group.
To view this discussion on the web visit
https://groups.google.com/d/msg/spyderlib/-/Y-5szqbItwcJ.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/spyderlib?hl=en.
--
You received this message because you are subscribed to the Google Groups
"spyder" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/spyderlib?hl=en.
