Skip to content

dark mode for "ALL the Clippy Lints" page #6877

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

Closed
matthiaskrgr opened this issue Mar 9, 2021 · 4 comments · Fixed by #7030
Closed

dark mode for "ALL the Clippy Lints" page #6877

matthiaskrgr opened this issue Mar 9, 2021 · 4 comments · Fixed by #7030
Assignees
Labels
A-website Area: Improving the clippy website

Comments

@matthiaskrgr
Copy link
Member

matthiaskrgr commented Mar 9, 2021

It might be neat to have a dark mode for https://rust-lang.github.io/rust-clippy/master/index.html which is a bit lighter on the eyes during nightly clippy delinting sessions.

@matthiaskrgr matthiaskrgr added the A-website Area: Improving the clippy website label Mar 9, 2021
@matthiaskrgr
Copy link
Member Author

cc @xFrednet

@matthiaskrgr matthiaskrgr changed the title dark mode for ALL the Clippy Lints page dark mode for "ALL the Clippy Lints" page Mar 9, 2021
@xFrednet
Copy link
Member

xFrednet commented Mar 9, 2021

I plan to change some things on the website in the coming month. I'll include this as well 🙃

@rustbot claim

@phansch
Copy link
Member

phansch commented Mar 9, 2021

Slight off-topic: It might also be worth considering using mdBook for the lint list as mdBook already comes with a dark mode.
Although the search functionality of mdBook does not provide results as quickly as our website - In mdBook you'd first have to click on the page in the results.

@xFrednet
Copy link
Member

xFrednet commented Mar 9, 2021

I thought about copying the mdbook or docs style to have it more unified among Rust projects. Completely using that project is a new Idea to think of. This might be worth a try :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-website Area: Improving the clippy website
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants