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

Implement conditional function breakpoints #9106

Closed
edumunoz opened this issue Jul 12, 2016 · 7 comments
Closed

Implement conditional function breakpoints #9106

edumunoz opened this issue Jul 12, 2016 · 7 comments
Assignees
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues *duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality
Milestone

Comments

@edumunoz
Copy link
Contributor

  • VSCode Version: 1.3.0
  • OS Version: All

The debugging protocol for VSCode supports conditional function breakpoints.

However, I don't see how this can be achieved through the VSCode UI, since adding a condition is done through the glyph in the editor, but function breakpoints don't have a glyph:

screen shot 2016-07-11 at 5 28 19 pm

A solution to match the current experience for line breakpoints would be to implement #7262.

Is there a reason why there is no action to add a condition to a breakpoint from the Breakpoints panel?

@weinand @isidorn

@f111fei
Copy link
Contributor

f111fei commented Jul 12, 2016

4

Right click on line number. Edit BreakPoint is conditional function breakpoints. this is similar to chrome

@weinand
Copy link
Contributor

weinand commented Jul 12, 2016

The '+' action adds function breakpoints:

2016-07-12_11-44-11

@weinand weinand closed this as completed Jul 12, 2016
@weinand weinand added the debug Debug viewlet, configurations, breakpoints, adapter issues label Jul 12, 2016
@weinand weinand self-assigned this Jul 12, 2016
@weinand weinand added this to the July 2016 milestone Jul 12, 2016
@edumunoz
Copy link
Contributor Author

@f111fei, function breakpoints don't show a glyph on the line number (and hence referencing issue #7262).

@weinand, how do you add a condition to function breakpoints? I don't see a way.

@weinand
Copy link
Contributor

weinand commented Jul 12, 2016

@edumunoz oh I see, the question was about how to add the 'condition' and not how to add the 'function breakpoint'. Sorry, I missed that.

@weinand weinand reopened this Jul 12, 2016
@weinand weinand removed this from the July 2016 milestone Jul 12, 2016
@weinand weinand assigned isidorn and unassigned weinand Jul 12, 2016
@weinand
Copy link
Contributor

weinand commented Jul 12, 2016

@isidorn I don't see a reason why a context action 'Edit Condition' in the breakpoints view wouldn't work, right?

@weinand weinand added the feature-request Request for new features or functionality label Jul 12, 2016
@isidorn
Copy link
Contributor

isidorn commented Jul 19, 2016

The UI for editing breakpoints is bound to the editor, thus if a breakpoint is in file a.txt, then editing that breakpoint would open that file and show the breakpoint widget on top of the line where the breakpoint is. That is the reason why we did not include 'edit breakpoint' in the viewlet, but only in the editor.

The issue here is that function breakpoints are not tied to an editor or a line, so I am not sure where to show the breakpoint widget in that case. We could just show it on top of the active editor, though then it is not in pair with the regualr edit breakpoint experience.

Suggestions welcome

@isidorn isidorn added this to the Backlog milestone Jul 21, 2016
@weinand
Copy link
Contributor

weinand commented Apr 5, 2017

duplicate of #3646

@weinand weinand closed this as completed Apr 5, 2017
@weinand weinand added the *duplicate Issue identified as a duplicate of another issue(s) label Apr 5, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues *duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality
Projects
None yet
Development

No branches or pull requests

4 participants