multiline breakpoint widget - implementing #56521

[YisraelV]

No description provided.

[msftclas]

All CLA requirements met.

[weinand]

Thanks for the PR.
What problem is this PR trying to solve?

[isidorn]

@weinand here's the issue for the PR #56521

[isidorn]

Use a proper name, not _ even though it is not used

[isidorn]

I prefer const to let if it will not be mutated, so can you please use const

[isidorn]

This is a listener, add it to the this.toDispose

[isidorn]

Same here, please use const

[isidorn]

@YisraelV great work, I really like it.
I commented directly in the code, so once you address that we can merge this in.

I checked for the css change, and I can no longer reproduce the original bug so we are good there. You do not have to worry about that.

[YisraelV]

@isidorn that's very nice to hear thank you very much.

I made the requested changes. Would you like me to squash my two commits into 1?

[isidorn]

@YisraelV no need for squashing, merging it in. Thanks a lot!