Skip to content

Improve ayu doc source line number contrast #76374

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Sep 9, 2020
Merged

Conversation

pickfire
Copy link
Contributor

@pickfire pickfire commented Sep 5, 2020

Improve contrast of foreground line number.

Before

image

After

image

r? @Cldfire

I think we should add the line for light and dark theme too, it looks better and clearer that way.

Improve contrast of foreground line number.
@LeSeulArtichaut LeSeulArtichaut added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 5, 2020
@jyn514 jyn514 added A-rustdoc-themes Area: Themes for HTML pages generated by rustdoc T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Sep 5, 2020
@Cldfire
Copy link
Contributor

Cldfire commented Sep 6, 2020

Looks good to me, thank you! 😄

(I don't think I have any bors perms? so someone else will need to r+ this)

I agree the line would look nice on the other themes as well, if other people feel the same it would be nice to add that.

@pickfire
Copy link
Contributor Author

pickfire commented Sep 6, 2020

I may be thinking of adding that later on. @Cldfire I added you as a reviewer because you are the one that added ayu, if any just to notify you.

r? @jyn514

@jyn514
Copy link
Member

jyn514 commented Sep 6, 2020

@bors r=Cldfire

@bors
Copy link
Collaborator

bors commented Sep 6, 2020

📌 Commit d80415a has been approved by Cldfire

@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 Sep 6, 2020
@jyn514
Copy link
Member

jyn514 commented Sep 6, 2020

@bors rollup=always

@bors
Copy link
Collaborator

bors commented Sep 7, 2020

⌛ Testing commit d80415a with merge 78e663278763336735fcbcc87af971a313df37b4...

@bors
Copy link
Collaborator

bors commented Sep 7, 2020

💔 Test failed - checks-actions

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

jyn514 commented Sep 7, 2020

The failure is #76442.

@bors retry

@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 Sep 7, 2020
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Sep 7, 2020
Improve ayu doc source line number contrast

Improve contrast of foreground line number.

Before

![image](https://user-images.githubusercontent.com/4687791/92305696-1bf2ab80-efbc-11ea-8b5c-a24c4f6261e0.png)

After

![image](https://user-images.githubusercontent.com/4687791/92305700-2a40c780-efbc-11ea-9061-dbfcb1e71980.png)

r? @Cldfire

I think we should add the line for light and dark theme too, it looks better and clearer that way.
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 9, 2020
Rollup of 10 pull requests

Successful merges:

 - rust-lang#76162 (Make duration_since documentation more clear)
 - rust-lang#76355 (remove public visibility previously needed for rustfmt)
 - rust-lang#76374 (Improve ayu doc source line number contrast)
 - rust-lang#76379 (rustbuild: Remove `Mode::Codegen`)
 - rust-lang#76389 (Fix HashMap visualizers in Visual Studio (Code))
 - rust-lang#76396 (Fix typo in tracking issue template)
 - rust-lang#76401 (Add help note to unconstrained const parameter)
 - rust-lang#76402 (Update linker-plugin-lto.md to contain up to rust 1.46)
 - rust-lang#76403 (Fix documentation for TyCtxt::all_impls)
 - rust-lang#76498 (Update cargo)

Failed merges:

 - rust-lang#76458 (Add drain_filter method to HashMap and HashSet)

r? `@ghost`
@bors bors merged commit 3cec71e into rust-lang:master Sep 9, 2020
@rustbot rustbot added this to the 1.48.0 milestone Sep 9, 2020
@pickfire pickfire deleted the patch-4 branch September 9, 2020 09:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-rustdoc-themes Area: Themes for HTML pages generated by rustdoc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants