-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Fix anchors issue when everything is collapsed #49652
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
I'm building some docs with this change, but it looks like this change makes it so that if the URL has an anchor, it doesn't auto-collapse a section that has that anchor in it? Just making sure. It looks good, but i'd like to poke around a rendering with it before giving an r+. |
@frewsxcv Here's a rendering on my server of docs with this change. If this works out for you, r=me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionally, lgtm!
Cool! Let's get this rolling then. :D @bors r+ |
📌 Commit 238583a has been approved by |
…QuietMisdreavus Fix anchors issue when everything is collapsed Fixes rust-lang#49455. @frewsxcv does it seem good for you like this? r? @QuietMisdreavus
…QuietMisdreavus Fix anchors issue when everything is collapsed Fixes rust-lang#49455. @frewsxcv does it seem good for you like this? r? @QuietMisdreavus
Rollup of 9 pull requests Successful merges: - #49510 (Fix anchor position on fields) - #49652 (Fix anchors issue when everything is collapsed) - #49702 (std: Inline some Termination-related methods) - #49728 (add emit_debug_gdb_scripts target option and ..) - #49731 (add THUMB targets to rustup manifest) - #49742 (Using X headings instead of 0.X #49739) - #49748 (proc_macro: Improve Debug representations) - #49750 (bootstrap: Remove the fast path) - #49503 (Inject the `compiler_builtins` crate whenever the `core` crate is injected) Failed merges:
Fixes #49455.
@frewsxcv does it seem good for you like this?
r? @QuietMisdreavus