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

ayu theme: Change doccomment color to #a1ac88 #74864

Merged
merged 1 commit into from
Jul 29, 2020

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Jul 28, 2020

Before:
image

After:
image

Close #74788

@rust-highfive
Copy link
Collaborator

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez

Some changes occurred in HTML/CSS themes.

cc @GuillaumeGomez

A change occurred in the Ayu theme.

cc @Cldfire

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 28, 2020
@tesuji tesuji changed the title ayu theme: Change doccoment color to #a1ac88 ayu theme: Change doccomment color to #a1ac88 Jul 28, 2020
Co-authored-by: Cldfire <cldfire@3grid.net>
@Cldfire
Copy link
Contributor

Cldfire commented Jul 28, 2020

Thank you! 👍

@GuillaumeGomez
Copy link
Member

If we change the color of doc comments in this theme, we have to do it in all themes. Please do so (I really like the new appearance btw!).

@tesuji
Copy link
Contributor Author

tesuji commented Jul 28, 2020

Other themes already differ doc comments and non-doc comments by colour. Or do you want to use the same color for doc comments in others?

@GuillaumeGomez
Copy link
Member

Oh indeed, my bad! It's change different tones of grey so that's why I thought it was the same. Then it's fine, let's merge this. Thanks!

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Jul 28, 2020

📌 Commit 5faef5e has been approved by GuillaumeGomez

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 28, 2020
@tesuji
Copy link
Contributor Author

tesuji commented Jul 28, 2020

The same doc comments color as Dark theme #8c9756:
With this PR: current
After:
image

bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 29, 2020
Rollup of 8 pull requests

Successful merges:

 - rust-lang#74266 (Clean up E0720 explanation)
 - rust-lang#74671 (add const generics array coercion test)
 - rust-lang#74707 (Add str::[r]split_once)
 - rust-lang#74814 (Fix RefUnwindSafe & UnwinsSafe impls for lazy::SyncLazy)
 - rust-lang#74859 (Update outdated readme)
 - rust-lang#74864 (ayu theme: Change doccomment color to `#a1ac88`)
 - rust-lang#74872 (Enable to ping RISC-V group via triagebot)
 - rust-lang#74891 (handle ConstEquate in rustdoc)

Failed merges:

r? @ghost
@bors bors merged commit 1de0211 into rust-lang:master Jul 29, 2020
@tesuji tesuji deleted the ayu-doccolor branch July 29, 2020 05:20
@cuviper cuviper added this to the 1.47.0 milestone May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Style doc comments different from normal comments in the Ayu theme
6 participants