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

Allow to delete breakpoint #25

Open
sidorares opened this issue Aug 31, 2014 · 3 comments
Open

Allow to delete breakpoint #25

sidorares opened this issue Aug 31, 2014 · 3 comments

Comments

@sidorares
Copy link
Owner

If "set breakpoint" is clicked on a line with breakpoint, toggle it.

Note that it currently possible to set multiple breakpoints on one line in different columns.

Should we toggle breakpoint only is cursor is "near enough" to existing bp? ( this might be tricky, because "near enough" to me is "if set here, actual_position would be exactly the same" so it requires extra test "setBreakpoint").
Pseudo algorithm:

get breakpoints
set at current cursor position
if actual_position overlaps with existing breakpoint, delete both + delete overlapping breakpoint marker.
otherwise, add breakpoint marker
@Pieter-Jan
Copy link

I would really like to see this feature added as well!

@sidorares
Copy link
Owner Author

sidorares commented Sep 13, 2016

@Pieter-Jan haven't looked at this for a long time, maybe will try to do over weekend

Also would be good to move to chrome remote debugging protocol since it's part of soon to become LTS node v6 - that way it would be possible to debug node & browser simultaneously

@xahon
Copy link

xahon commented May 13, 2017

Any news?

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

No branches or pull requests

3 participants