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

(docs) Make <pre> code be readable in dark mode also #179

Merged
merged 1 commit into from
Apr 9, 2021

Conversation

perlun
Copy link
Collaborator

@perlun perlun commented Apr 9, 2021

This is an additional fix for #174. Without this, the text used #333 on black (#000) background, which is rather hard to read, to say the least.

This change retains the #333 color in light mode (where it makes sense), but changes to use inherit in dark mode, which looks much better.

@perlun perlun added the website Things related to https://perlang.org label Apr 9, 2021
@perlun perlun added this to the 0.1.0 milestone Apr 9, 2021
@perlun perlun force-pushed the fix/pre-code-unreadable-in-dark-mode branch from 1f32a59 to 8aea4d0 Compare April 9, 2021 20:23
@@ -14,6 +14,9 @@ jobs:
steps:
- uses: actions/checkout@v1

- name: Install SASS
run: npm install -g sass

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 CI failed in https://github.com/perlang-org/perlang/pull/179/checks?check_run_id=2308969743; this change is here to try and fix that. I don't really understand why it tried to rebuild the CSS stylesheet though, but I'm guessing the timestamp of the .scss file was never than the .css file in this case. It has worked until now, probably because the order of the timestamps was the reverse in all CI runs until this one.

This is an additional fix for #174. Without this, the text used #333 on
black (#000) background, which is rather hard to read, to say the least.

This change retains the #333 color in light mode (where it makes sense),
but changes to use `inherit` in dark mode, which looks much better.
@perlun perlun force-pushed the fix/pre-code-unreadable-in-dark-mode branch from 8aea4d0 to a8a2872 Compare April 9, 2021 20:26
perlun pushed a commit to perlun/darkerfx that referenced this pull request Apr 9, 2021
@perlun perlun merged commit 9c9234d into master Apr 9, 2021
@perlun perlun deleted the fix/pre-code-unreadable-in-dark-mode branch April 9, 2021 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
website Things related to https://perlang.org
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant