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

Some colors don't have enough contrast #2

Closed
crt0r opened this issue Oct 21, 2022 · 9 comments
Closed

Some colors don't have enough contrast #2

crt0r opened this issue Oct 21, 2022 · 9 comments

Comments

@crt0r
Copy link

crt0r commented Oct 21, 2022

First of all, thank you for developing such a good theme!

I write a lot of documentation in Markdown for my work and sometimes have a hard time reading the text in a preview or an editor. So far, the issues are with blockquotes and some code blocks.

  1. Some parts of the code block are nearly the same color as the background, so they are barely readable.
    photo_2022-10-21_11-23-33
    photo_2022-10-21_11-24-54

  2. The blockquote at the bottom of the screenshot is very light.
    photo_2022-10-21_11-21-34

@huacat1017
Copy link
Owner

First of all, Thank you for reporting these!
the code block issue seems like I don't color these. But I don't know the elements now.
the second issue: It's the light green color in my color palette, maybe make it (blockquote texts) deeper or a new color to fix it.
I have no time working on this these days. I'll fix these on Nov 5 if there is no urgent homework.

@huacat1017
Copy link
Owner

Sorry for late replying. I’m not sure when I will be free to fix. My time management has been a mess. I’m sorry to delay and delay this. Hope to use my time better.

@crt0r
Copy link
Author

crt0r commented Dec 7, 2022

Sorry for late replying. I’m not sure when I will be free to fix. My time management has been a mess. I’m sorry to delay and delay this. Hope to use my time better.

Nothing serious. I have the same issues with my time due to college and day job. Especially because it's the end of the year and it's the period of final deadlines--for this year. Just focus on what's important for you first right now, Open Source comes next. 😸

@huacat1017
Copy link
Owner

huacat1017 commented Jan 23, 2023

I opened a new issue and mentioned this to VScode team. The code block problem is not as easy as I thought.
If you want to edit it yourself, go to "C:\Users%USERNAME%\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\markdown-language-features\media\highlight.css" (Windows) to change the .hljs-subst class color by adding these lines on the file: (In my opinion, the color can fit few of light themes.)

.vscode-light .hljs-subst,
.vscode-light .hljs-function,
.vscode-light .hljs-title,
.vscode-light .hljs-params,
.vscode-light .hljs-formula {
	color: #909090; /*Change the color to what you like*/
}

They use highlight.js to get the Preview colored. You can check this issue. They recommended Markdown Preview VS Code Highlighting extension to make the syntax in the code blocks match active color theme because Shiki uses TextMate. (Shell script works well, but it only has two default themes for C++) Syntax in Markdown Preview might be a problem for a long time. (OR you can go to Settings -> markdown.styles add the CSS yourself but it is unconvenient)

If you don't care about the background. You can try Markdown Preview Enhanced and Markdown Preview Github Styling to avoid the text unable to read.

@crt0r
Copy link
Author

crt0r commented Jan 23, 2023

They use highlight.js to get the Preview colored. You can check this issue. They recommended Markdown Preview VS Code Highlighting extension to make the syntax in the code blocks match active color theme because Shiki uses TextMate.

Thanks for the detailed reply! I'll try out these extensions.

@huacat1017
Copy link
Owner

huacat1017 commented Jan 24, 2023

I tried a darker color. Are you OK with this color?

Before (#99ca82) After (#3d8b1c)
orginal green color modified sgreen color

@crt0r
Copy link
Author

crt0r commented Jan 24, 2023

I tried a darker color. Are you OK with this color?
Before (#99ca82) After (#3d8b1c)
orginal green color modified sgreen color

Yep, it's a lot more readable.

@huacat1017
Copy link
Owner

Published 0.4.1 to fix these. Enjoy!

@crt0r
Copy link
Author

crt0r commented Jan 25, 2023

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants