Conversation
jsha
added
T-rustdoc
|
Some changes occurred in HTML/CSS/JS. |
|
r? @CraftSpider (rust-highfive has picked a reviewer for you, use r? to override) |
rust-highfive
added
the
S-waiting-on-review
src/librustdoc/html/render/mod.rs
Outdated
There was a problem hiding this comment.
trait_ is a link, so that won't work.
|
Also small UI issue:
|
|
Updated so the |
There was a problem hiding this comment.
Because of this change, the anchor on Methods from Deref<Target = str> targets str, is blue and overlaps the text too.
|
:) |
|
k, I think I've got it with this latest revision. Pushed and update to the demo. |
| // We move the cursor over the "Implementations" title so the anchor is displayed. | ||
| move-cursor-to: "h2#implementations" | ||
| assert-css: ("h2#implementations a.anchor", {"color": "rgb(0, 0, 0)"}) | ||
| assert-css: ("h2#implementations", {"color": "rgb(0, 0, 0)"}) |
There was a problem hiding this comment.
It's supposed to check that the anchor is displayed, which isn't tested here.
|
☔ The latest upstream changes (presumably #92609) made this pull request unmergeable. Please resolve the merge conflicts. |
|
I'm going to set this aside for a while. The amount of work required to make everything work right is not worth the benefit. |
5 participants
Fixes #90245.
Demo:
https://rustdoc.crud.net/jsha/linked-headers/std/string/struct.String.html