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

Improve ayu rustdoc theme #74371

Merged
merged 2 commits into from
Jul 17, 2020
Merged

Improve ayu rustdoc theme #74371

merged 2 commits into from
Jul 17, 2020

Conversation

Aloso
Copy link
Contributor

@Aloso Aloso commented Jul 15, 2020

This PR changes the following:

  • It makes some lines darker
  • It gives the crate selector and search bar a border
    • The search bar's border turns blue when focused
  • Gives the logo a bright shadow.

For standard library crates, it would be better to invert the logo, but that would be bad for crates with a colored logo, e.g. async-std.

Before:

old

After (note that this PR no longer includes the white shadow of the logo):

new

* It makes some lines darker
* It gives the crate selector and search bar a border
  * The search bar's border turns blue when focused
* Gives the logo a bright shadow. This makes dark logos stand out more
@rust-highfive
Copy link
Collaborator

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez

Some changes occurred in HTML/CSS themes.

cc @GuillaumeGomez

@rust-highfive
Copy link
Collaborator

r? @steveklabnik

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 15, 2020
@tesuji
Copy link
Contributor

tesuji commented Jul 15, 2020

Can you post some screenshots before and after changes?

@Aloso
Copy link
Contributor Author

Aloso commented Jul 15, 2020

@lzutao I added screenshots to the first comment.

@Cldfire
Copy link
Contributor

Cldfire commented Jul 15, 2020

There's already a PR for the logo issue, so that should be removed from here: #74361

I'll take a look at the other changes later today. They look good based off of the screenshots!

@Aloso
Copy link
Contributor Author

Aloso commented Jul 15, 2020

@Cldfire done!

@GuillaumeGomez
Copy link
Member

Looks good to me, thanks! r=lzutao and me once CI pass.

@Cldfire
Copy link
Contributor

Cldfire commented Jul 15, 2020

Looks great, thanks you @Aloso! 😄

@Dylan-DPC-zz
Copy link

r? @GuillaumeGomez

@Dylan-DPC-zz
Copy link

@bors r=GuilliameGomez rollup

@bors
Copy link
Contributor

bors commented Jul 16, 2020

📌 Commit d08bb40 has been approved by GuilliameGomez

@bors
Copy link
Contributor

bors commented Jul 16, 2020

🌲 The tree is currently closed for pull requests below priority 5, this pull request will be tested once the tree is reopened

@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 16, 2020
Manishearth added a commit to Manishearth/rust that referenced this pull request Jul 16, 2020
Improve ayu rustdoc theme

This PR changes the following:

* It makes some lines darker
* It gives the crate selector and search bar a border
  * The search bar's border turns blue when focused
* ~~Gives the logo a bright shadow.~~

For standard library crates, it would be better to invert the logo, but that would be bad for crates with a colored logo, e.g. [async-std](https://docs.rs/async-std/1.6.2/async_std/).

Before:

![old](https://user-images.githubusercontent.com/15658558/87576611-ed4e0800-c6d1-11ea-9667-3924702f79e2.png)

After (note that this PR no longer includes the white shadow of the logo):

![new](https://user-images.githubusercontent.com/15658558/87576621-ef17cb80-c6d1-11ea-8e15-5d7f8b180c07.png)
This was referenced Jul 17, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 17, 2020
…arth

Rollup of 8 pull requests

Successful merges:

 - rust-lang#73101 (Resolve items for cross-crate imports relative to the original module)
 - rust-lang#73269 (Enable some timeouts in SGX platform)
 - rust-lang#74033 (Add build support for Cargo's build-std feature.)
 - rust-lang#74351 (Do not render unstable items for rustc doc)
 - rust-lang#74357 (Some `Symbol` related improvements)
 - rust-lang#74371 (Improve ayu rustdoc theme)
 - rust-lang#74386 (Add RISC-V GNU/Linux to src/tools/build-manifest as a host platform)
 - rust-lang#74398 (Clean up E0723 explanation)

Failed merges:

r? @ghost
@bors bors merged commit 874097c into rust-lang:master Jul 17, 2020
@Aloso Aloso deleted the patch-1 branch July 17, 2020 20:58
@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.

9 participants