Skip to content

Conversation

@arferreira
Copy link
Contributor

@arferreira arferreira commented Jan 26, 2026

Changed --right-side-color from #808080 to #ababab in the dark theme.

Screenshot 2026-01-25 at 8 04 29 PM

Verified visually in dark theme, it's now more readable:

Screenshot 2026-01-25 at 8 41 02 PM

Part of #59845
Fixes #151422

@rustbot
Copy link
Collaborator

rustbot commented Jan 26, 2026

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez, @lolbinarycat

@rustbot rustbot added A-rustdoc-js Area: Rustdoc's JS front-end S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. labels Jan 26, 2026
@rustbot
Copy link
Collaborator

rustbot commented Jan 26, 2026

r? @GuillaumeGomez

rustbot has assigned @GuillaumeGomez.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rust-log-analyzer

This comment has been minimized.

@rustbot
Copy link
Collaborator

rustbot commented Jan 26, 2026

Some changes occurred in GUI tests.

cc @GuillaumeGomez

@rust-log-analyzer

This comment has been minimized.

@arferreira
Copy link
Contributor Author

@GuillaumeGomez I changed color from #ABABAB to #D0D0D0, now it passed for both specs AA and AAA

#D0D0D0

@GuillaumeGomez
Copy link
Member

Looks good to me, thanks! Please squash your commits then I'll put it into the build queue.

Signed-off-by: Antonio Souza <arfs.antonio@gmail.com>
@arferreira arferreira force-pushed the fix-rustdoc-contrast branch from a96e5a9 to c7788af Compare January 26, 2026 11:55
@lolbinarycat
Copy link
Contributor

typo in commit and title?

@GuillaumeGomez
Copy link
Member

What typo do you see?

Also sorry, forgot to r+ it. r+ing in the meantime.

@bors r+ rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 28, 2026

📌 Commit c7788af has been approved by GuillaumeGomez

It is now in the queue for this repository.

@rust-bors rust-bors bot 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 Jan 28, 2026
Zalathar added a commit to Zalathar/rust that referenced this pull request Jan 29, 2026
…=GuillaumeGomez

Fix contrast ratio for `Since` element in rustodoc dark theme

Changed `--right-side-color` from `#808080` to `#ababab` in the dark theme.

<img width="742" height="784" alt="Screenshot 2026-01-25 at 8 04 29 PM" src="https://github.com/user-attachments/assets/38c5f0b9-2034-429f-87db-8a0ed8209b5d" />

Verified visually in dark theme, it's now more readable:

<img width="174" height="96" alt="Screenshot 2026-01-25 at 8 41 02 PM" src="https://github.com/user-attachments/assets/d0c30409-4374-48c4-ae9c-a0aec48e8957" />

Part of rust-lang#59845
Fixes rust-lang#151422
@lolbinarycat
Copy link
Contributor

"rustodoc"

no idea if this is a typo or i'm missing something.

rust-bors bot pushed a commit that referenced this pull request Jan 29, 2026
Rollup of 8 pull requests

Successful merges:

 - #150474 (Tidy: detect ui tests subdirectory changes so `tests/ui/README.md` stays in sync)
 - #150572 (Improve move error diagnostic for `AsyncFn` closures)
 - #151596 (Fix -Zmir-enable-passes to detect unregistered enum names in declare_passes macro)
 - #151802 (Make `QueryDispatcher::Qcx` an associated type)
 - #151559 ([rustdoc] Add a marker to tell users that there are hidden (deprecated) items in the search results)
 - #151665 (Fix contrast ratio for `Since` element in rustodoc dark theme)
 - #151798 (Update `askama` to `0.15.3`)
 - #151800 (Subscribe myself to translation diagnostics)
@arferreira arferreira changed the title Fix contrast ratio for Since element in rustodoc dark theme Fix contrast ratio for Since element in rustdoc dark theme Jan 29, 2026
@arferreira
Copy link
Contributor Author

"rustodoc"

no idea if this is a typo or i'm missing something.

it was a typo on my PR title, commit looks okay, fixed title tho

@lolbinarycat
Copy link
Contributor

Commit has the same typo

rust-bors bot pushed a commit that referenced this pull request Jan 29, 2026
Rollup of 12 pull requests

Successful merges:

 - #150474 (Tidy: detect ui tests subdirectory changes so `tests/ui/README.md` stays in sync)
 - #150572 (Improve move error diagnostic for `AsyncFn` closures)
 - #151596 (Fix -Zmir-enable-passes to detect unregistered enum names in declare_passes macro)
 - #151802 (Make `QueryDispatcher::Qcx` an associated type)
 - #149110 (Implement `cast_slice` for raw pointer types)
 - #151559 ([rustdoc] Add a marker to tell users that there are hidden (deprecated) items in the search results)
 - #151665 (Fix contrast ratio for `Since` element in rustdoc dark theme)
 - #151785 (Stabilize feature(push_mut))
 - #151798 (Update `askama` to `0.15.3`)
 - #151800 (Subscribe myself to translation diagnostics)
 - #151804 (Document, sort, and tweak spellcheck entries in `typos.toml`)
 - #151805 (Fix grammar in `env::current_exe()#Security`)
@rust-bors rust-bors bot merged commit 3830f76 into rust-lang:main Jan 29, 2026
11 checks passed
@rustbot rustbot added this to the 1.95.0 milestone Jan 29, 2026
rust-timer added a commit that referenced this pull request Jan 29, 2026
Rollup merge of #151665 - arferreira:fix-rustdoc-contrast, r=GuillaumeGomez

Fix contrast ratio for `Since` element in rustdoc dark theme

Changed `--right-side-color` from `#808080` to `#ababab` in the dark theme.

<img width="742" height="784" alt="Screenshot 2026-01-25 at 8 04 29 PM" src="https://github.com/user-attachments/assets/38c5f0b9-2034-429f-87db-8a0ed8209b5d" />

Verified visually in dark theme, it's now more readable:

<img width="174" height="96" alt="Screenshot 2026-01-25 at 8 41 02 PM" src="https://github.com/user-attachments/assets/d0c30409-4374-48c4-ae9c-a0aec48e8957" />

Part of #59845
Fixes #151422
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-rustdoc-js Area: Rustdoc's JS front-end 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. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rustdoc fails contrast check for a certain element.

5 participants