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: apply home dark mode #3062

Merged
merged 4 commits into from
Jan 27, 2023
Merged

Conversation

krrrr38
Copy link
Contributor

@krrrr38 krrrr38 commented Jan 27, 2023

what

why

Doc top page doesn't apply dark mode correctly.

image

tests

mode asis tobe
white asis_white tobe_white
dark asis_dark tobe_dark

references

$buttonColor: var(--c-brand);
$buttonHoverColor: var(--c-brand-light);
$darkBackground: var(--c-bg-light);
$darkBackgroundBorder: var(--c-border-dark);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

When using var color, we can apply dark mode easily. All vars are defined in the following.

https://v2.vuepress.vuejs.org/reference/default-theme/styles.html#style-file

@krrrr38 krrrr38 marked this pull request as ready for review January 27, 2023 18:09
@krrrr38 krrrr38 requested a review from a team as a code owner January 27, 2023 18:09
@krrrr38 krrrr38 marked this pull request as draft January 27, 2023 18:33
@krrrr38 krrrr38 marked this pull request as ready for review January 27, 2023 18:37
@nitrocode
Copy link
Member

nitrocode commented Jan 27, 2023

@krrrr38 nice work! And thank you for the contribution!

could you add ready_for_review event to each of the skipped checks (codeql and pr labeler) ? They only seem to get started when its ready to review && a new commit appears

Copy link
Member

@nitrocode nitrocode left a comment

Choose a reason for hiding this comment

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

One comment prior to merge

@nitrocode nitrocode added this to the v0.23.0 milestone Jan 27, 2023
@krrrr38 krrrr38 force-pushed the fix-docs-home-dark-mode branch from d806a34 to e9edbd2 Compare January 27, 2023 22:26
@nitrocode nitrocode merged commit ac795ce into runatlantis:main Jan 27, 2023
DragosDumitrache pushed a commit to gocardless/atlantis that referenced this pull request Jan 30, 2023
* merge scss into index.scss

* apply darkmode on docs Home using predefined var color

* codeql/labeler github action for handling draft pr

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

Successfully merging this pull request may close these issues.

3 participants