Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated lsp-illustration for Invisible labels on LSP guide in dark theme. fixes #7866 #7893

Merged
merged 9 commits into from
Jan 2, 2025

Conversation

RonakRahane
Copy link
Contributor

@RonakRahane RonakRahane commented Dec 27, 2024

Updated the lsp-illustration image whose text were invisible in dark mode.
Replaced the 'lsp-illustration.png' image with an updated version tracked by Git LFS.

before image :-

lsp-illustration (1)

After image changes ( added a background behind the text so that it becomes visible in dark mode ) :-
Added white background for text as it is invisible in light mode and can only be seen in dark mode.

lsp-illustration

Copy link
Contributor

@ntrogh ntrogh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RonakRahane Looks good to me. Can you make sure to remove the .gitattributes file from the PR?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RonakRahane Can you remove this file from the PR?

@ntrogh ntrogh added the doc-enhancement suggested addition or improvement label Jan 2, 2025
@RonakRahane
Copy link
Contributor Author

I have deleted the .gitattribute file. can you plz review it.

@RonakRahane RonakRahane requested a review from ntrogh January 2, 2025 10:59
Copy link
Contributor

@ntrogh ntrogh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @RonakRahane

@vs-code-engineering vs-code-engineering bot added this to the January 2025 milestone Jan 2, 2025
@ntrogh ntrogh merged commit 5ff9df1 into microsoft:main Jan 2, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-enhancement suggested addition or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants