Skip to content

Show breakpoints in overview ruler#82787

Merged
isidorn merged 3 commits intomicrosoft:masterfrom
anirudhrb:breakpoints_in_scrollbar
Oct 18, 2019
Merged

Show breakpoints in overview ruler#82787
isidorn merged 3 commits intomicrosoft:masterfrom
anirudhrb:breakpoints_in_scrollbar

Conversation

@anirudhrb
Copy link
Contributor

@anirudhrb anirudhrb commented Oct 17, 2019

This makes life easier when working with breakpoints in large files. Introduce a
new setting editor.showBreakpointsInOverviewRuler that controls whether this
is enabled or not. This setting is false by default. Use maroon for breakpoint
markers in the overview ruler to differentiate them from error markers.

Closes #15104.

@anirudhrb
Show breakpoints in overview ruler
2dbe34d 
This makes life easier when working with breakpoints in large files. Introduce a
new setting `editor.showBreakpointsInOverviewRuler` that controls whether this
is enabled or not. This setting is `false` by default. Use maroon for breakpoint
markers in the overview ruler to differentiate them from error markers.

Closes microsoft#15104.
@anirudhrb
Copy link
Contributor Author

anirudhrb commented Oct 17, 2019 •
edited
Loading

This is what it looks like:

Screenshot_20191017_220805

@anirudhrb anirudhrb mentioned this pull request Oct 17, 2019
Closed
@isidorn isidorn self-assigned this Oct 17, 2019
@isidorn isidorn added the debug Debug viewlet, configurations, breakpoints, adapter issues label Oct 17, 2019
@isidorn
Copy link
Collaborator

isidorn commented Oct 17, 2019

Great PR, thanks a lot!
Everything is great, except can you move the setting to the debug section. Not the editor section.
The setting should be called
debug.showBreakpointsInOverviewRuler and it should be defined somewhere here

Once you tackle this I would be happy merging in the PR

@anirudhrb
Move editor.showBreakpointsInOverviewRuler to
8c343e7 
`debug.showBreakpointsInOverviewRuler`.
@anirudhrb
Copy link
Contributor Author

anirudhrb commented Oct 18, 2019

@isidorn Moved the setting to the debug section now. Please have a look.

@isidorn
Copy link
Collaborator

isidorn commented Oct 18, 2019

@anirudhrb this i great work. Thanks a lot!
I tested it and it works nicely. There are just two more things:

  1. We decided to not expose and make the color configurable. Reason being this should always be aligned with the breakpoint color. So this will be tackled in the future when we make the breakpoint color customisable. So can you please remove the contribution of a new color
  2. BreakpointEditorContribution should update all breakpoint decorations on configuration change of debug.showBreakpointsInOverviewRuler . That way when the user changes this setting he will imediatly see the change and he will not need to close / reopen the edditor

Can you please tackle this and I will merge it in. THanks!

@isidorn isidorn added this to the October 2019 milestone Oct 18, 2019
@anirudhrb
Improvements suggested in code review
4f5a47b 
1. Added a configuration change listener in breakpointEditorContribution
   that resets the decorations if debug.showBreakpointsInOverviewRuler
   changes.
2. Removed the color registry contribution for the breakpoint marker
   since it is tied to the breakpoint color which is not configurable
   today.
@anirudhrb
Copy link
Contributor Author

anirudhrb commented Oct 18, 2019 •
edited
Loading

@isidorn Thanks for the feedback. I have implemented your suggestions. I am now able to see the effect of the setting change without re-opening the editor.

Please have a look and let me know if something can be done in a better way.

@isidorn
Copy link
Collaborator

isidorn commented Oct 18, 2019 •
edited
Loading

Looks great, thanks a lot for this PR
☀️ 🌈

@isidorn isidorn merged commit 0431149 into microsoft:master Oct 18, 2019
@anirudhrb anirudhrb deleted the breakpoints_in_scrollbar branch October 21, 2019 02:16
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@isidorn isidorn

Labels

debug Debug viewlet, configurations, breakpoints, adapter issues

Projects

None yet

Milestone

October 2019

Development

Successfully merging this pull request may close these issues.

Display breakpoints in scrollbar

2 participants

@anirudhrb @isidorn