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

Set font color on annotation class to override github dark mode color #23

Merged
merged 1 commit into from
Feb 13, 2022

Conversation

AriAbr
Copy link

@AriAbr AriAbr commented Feb 8, 2022

Without this fix the annotation box is hard to read in dark mode, and unreadable in high contrast dark mode
Related to (but not a fix for) #22

@helmus
Copy link
Owner

helmus commented Feb 9, 2022

Can you share a screenshot of what this looks like in dark mode / light mode ?

@AriAbr
Copy link
Author

AriAbr commented Feb 10, 2022

Light Mode
image

Dark Mode
image

High Contrast Dark Mode
image

@AriAbr
Copy link
Author

AriAbr commented Feb 13, 2022

@helmus @kaspars-ozols Can you merge this?

@helmus helmus merged commit 99982a1 into helmus:master Feb 13, 2022
@AriAbr
Copy link
Author

AriAbr commented Feb 13, 2022

Thank you!

@AriAbr
Copy link
Author

AriAbr commented Feb 13, 2022

@helmus Can you update the chrome extension also? The change doesn't seem to be showing up there yet

@helmus
Copy link
Owner

helmus commented Feb 13, 2022 via email

@AriAbr
Copy link
Author

AriAbr commented Feb 20, 2022

@helmus Can you take a look at this? The extension still has not updated

@helmus
Copy link
Owner

helmus commented Feb 20, 2022

version 1.8.0 is pending review on the chrome store.
I expect it will take a couple of days to a week before it's accepted.
Keep in mind that you can load the extension manually as well ( you'll need to build the project )

@helmus
Copy link
Owner

helmus commented Feb 24, 2022

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

Successfully merging this pull request may close these issues.

None yet

2 participants