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

Desktop: Adjust the codemirror code block colors for the dark theme #3794

Merged
merged 2 commits into from
Sep 23, 2020

Conversation

CalebJohn
Copy link
Collaborator

Fixes #3793

image

I chose these colors by taking the defaults and adjusting the lightness until the contrast was high enough
image

The default color was typically very low contrast
}

.cm-s-solarized.cm-s-light span.cm-comment {
color: #8ba1a7 !important;
Copy link
Owner

Choose a reason for hiding this comment

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

Isn't it something that should be fixed in the theme files directly (eg. themes/nord.ts, etc.)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The theme files control the way Joplin components will look and be coloured but these changes are specific to the codemirror themes. AFAIK these aren't integrated yet.

Copy link
Owner

Choose a reason for hiding this comment

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

Ah yes, that's right. Ideally we'd submit a patch upstream but I don't know how responsive they are so let's add this for now.

@laurent22 laurent22 changed the base branch from dev to release-1.2 September 23, 2020 08:34
@laurent22 laurent22 merged commit 59599d3 into laurent22:release-1.2 Sep 23, 2020
@CalebJohn CalebJohn deleted the nord-colors branch September 23, 2020 15:17
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.

Codeview code block font color too close to Nord theme background
2 participants