Skip to content

Fixes #46027 by using box-shadow css to render guides#46029

Merged
alexdima merged 2 commits intomicrosoft:masterfrom
gizm0bill:46027_fix_guides_display
Mar 26, 2018
Merged

Fixes #46027 by using box-shadow css to render guides#46029
alexdima merged 2 commits intomicrosoft:masterfrom
gizm0bill:46027_fix_guides_display

Conversation

@gizm0bill
Copy link

@gizm0bill gizm0bill commented Mar 17, 2018

…rather than background color and computed width
Should also be faster without these computations
vscode-guides-fixed

…r than background color and computed width

...should also be faster without these computations
@msftclas
Copy link

msftclas commented Mar 17, 2018

CLA assistant check
All CLA requirements met.

@alexdima
Copy link
Member

alexdima commented Mar 26, 2018

This is very cool! Thank you! I will just push a follow-up commit to remove the usage of CSS variables, as this is not supported in IE11 (supported by the monaco editor)

@alexdima alexdima added this to the March 2018 milestone Mar 26, 2018
@alexdima alexdima merged commit d0ad42a into microsoft:master Mar 26, 2018
alexdima added a commit that referenced this pull request Mar 26, 2018
@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.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants