Skip to content

editor: include lane in GUTTER_GLYPH_MARGIN hover events#202194

Merged
connor4312 merged 2 commits intomainfrom
connor4312/test-coverage-decorations-3
Jan 10, 2024
Merged

editor: include lane in GUTTER_GLYPH_MARGIN hover events#202194
connor4312 merged 2 commits intomainfrom
connor4312/test-coverage-decorations-3

Conversation

@connor4312
Copy link
Copy Markdown
Member

Also fixes handling of testing clicks and margin hovers which were not
aware of multiple lanes. Hover messages are now shown correctly and
tests no longer run if e.g. a breakpoint decoration on the same line is
clicked.

Fixes handling of decorations when word wrap is on as well.

Note sure if this was the best way to do this, let me know.

Also fixes handling of testing clicks and margin hovers which were not
aware of multiple lanes. Hover messages are now shown correctly and
tests no longer run if e.g. a breakpoint decoration on the same line is
clicked.

Note sure if this was the best way to do this, let me know.
@connor4312 connor4312 self-assigned this Jan 10, 2024
@vscodenpa vscodenpa added this to the December / January 2024 milestone Jan 10, 2024
@connor4312 connor4312 merged commit 709db78 into main Jan 10, 2024
@connor4312 connor4312 deleted the connor4312/test-coverage-decorations-3 branch January 10, 2024 23:39
@microsoft microsoft locked and limited conversation to collaborators Jun 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants