-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Fix tooltip position if the documentation starts with a code block #74635
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
Conversation
Some changes occurred in HTML/CSS/JS. |
f26d7ad
to
96225b1
Compare
@bors r+ rollup @GuillaumeGomez can we set highfive to contain all actively reviewing rustdoc team members so that instead of pinging the entire team for each small PR it cycles through? |
📌 Commit 96225b1 has been approved by |
Normally it's already set: https://github.com/rust-lang/highfive/blob/master/highfive/configs/rust-lang/rust.json#L28 Next time I'll just not set a reviewer. |
…anishearth Fix tooltip position if the documentation starts with a code block Fixes rust-lang#74321. Before:  After:  And in case there is text, it is not being applied:  And on mobile it isn't needed so it's not "activated":  r? @rust-lang/rustdoc
Oh, lol. Yeah, I think we should @ the entire team for larger PRs. |
…arth Rollup of 9 pull requests Successful merges: - rust-lang#73655 (va_args implementation for AAPCS.) - rust-lang#73893 (Stabilize control-flow-guard codegen option) - rust-lang#74237 (compiletest: Rewrite extract_*_version functions) - rust-lang#74454 (small coherence cleanup) - rust-lang#74528 (refactor and reword intra-doc link errors) - rust-lang#74568 (Apply rust-lang#66379 to `*mut T` `as_ref`) - rust-lang#74570 (Use forge links for prioritization procedure) - rust-lang#74589 (Update books) - rust-lang#74635 (Fix tooltip position if the documentation starts with a code block) Failed merges: r? @ghost
Fixes #74321.
Before:
After:
And in case there is text, it is not being applied:
And on mobile it isn't needed so it's not "activated":
r? @rust-lang/rustdoc