Skip to content

Fix missing horizontal bracket guides when editor.wrappingIndent == "same"#152529

Merged
hediet merged 1 commit intomicrosoft:mainfrom
MonadChains:wrapping-indent-bugfix
Jun 23, 2022
Merged

Fix missing horizontal bracket guides when editor.wrappingIndent == "same"#152529
hediet merged 1 commit intomicrosoft:mainfrom
MonadChains:wrapping-indent-bugfix

Conversation

@MonadChains
Copy link
Contributor

This PR fixes #149074

The if statement removed prevents any guides in case of wrapped line with no wrapping-indentation (i.e. when the starting column of the new line is the first).
This prevented the rendering of horizontal guides.
Removing the statement also does not lead to wrongfully diplayed vertical guides (see example below).
example-guides

…same" (microsoft#149074)

The if statement removed prevents any guides in case of wrapped line with no wrapping-indentation (i.e. when the starting column of the new line is the first).
This prevented the rendering of horizontal guides.
Removing the statement also does not lead to wrongfully diplayed vertical lines.
@ghost
Copy link

ghost commented Jun 17, 2022

CLA assistant check
All CLA requirements met.

@hediet hediet added this to the June 2022 milestone Jun 23, 2022
@hediet hediet enabled auto-merge June 23, 2022 16:24
@hediet hediet merged commit 080c9a2 into microsoft:main Jun 23, 2022
@MonadChains MonadChains deleted the wrapping-indent-bugfix branch July 19, 2022 00:37
@github-actions github-actions bot locked and limited conversation to collaborators Aug 7, 2022
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.

Horizontal bracket guides break on wrapped lines in first column, if editor.wrappingIndent == "same"

3 participants