Rollup of 4 pull requests #101590
Closed
Rollup of 4 pull requests #101590
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was added in 51f26ac to help with the display of an `<h3>` tag that has a `<span class='in-band'>` inside. The way implementation lists were rendered was changed in 34bd2b8 to have `<code class='in-band'>`, making this CSS unused. Then it was turned into a `<div>` in 9077d54 without issue. Finally, the header itself acquired the `in-band` class in 76a3b60.
This CSS was added in 5e01ba3, and served to override CSS right above it that set a 5px padding for several kinds of buttons in the same toolbar. The CSS that it overrode is still there, but now it only applies to `#settings-menu > a`, so there's nothing to override.
Fix hermit warnings This fixes two `unused_imprt` and one `dead_code` warning for hermit.
…list-h3-span-in-band, r=Dylan-DPC rustdoc: remove unused CSS `#implementations-list > h3 > span.in-band` This was added in 51f26ac to help with the display of an `<h3>` tag that has a `<span class='in-band'>` inside. The way implementation lists were rendered was changed in 34bd2b8 to have `<code class='in-band'>`, making this CSS unused. Then it was turned into a `<div>` in 9077d54 without issue. Finally, the header itself acquired the `in-band` class in 76a3b60.
…-div, r=GuillaumeGomez rustdoc: remove unused CSS `div.impl-items > div` This was added in 9077d54 to override the style on `<div>` tags that were acting as headers. These `<div>` tags were replaced with `<section>` tags in 32f6260, but this CSS was probably already redundant even then (the headers had already been replaced with real `<h3>` and `<h4>` tags in 76a3b60).
… r=GuillaumeGomez
rustdoc: remove no-op CSS `#settings-menu { padding: 0 }`
This CSS was added in 5e01ba3, and served to override CSS right above it that set a 5px padding for several kinds of buttons in the same toolbar:
https://github.com/rust-lang/rust/blob/5e01ba36c9f1037c4cf3e7421413fc6c41f85d05/src/librustdoc/html/static/css/rustdoc.css#L1400-L1409
The CSS that it overrode is still there, but now it only applies to `#settings-menu > a`, so there's nothing to override.
https://github.com/rust-lang/rust/blob/24d69920201563f0ee7b530f1cda0f171b205cc2/src/librustdoc/html/static/css/rustdoc.css#L1445-L1454
rustbot
added
T-libs
|
@bors r+ rollup=never p=4 |
bors
added
the
S-waiting-on-bors
|
is there a reason why not all 7 rollup=always PRs were rolled up in this? |
|
I was going to ask the same question @compiler-errors did ask. |
bors
added
S-waiting-on-author
|
Okay. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Successful merges:
#implementations-list > h3 > span.in-band#101554 (rustdoc: remove unused CSS#implementations-list > h3 > span.in-band)div.impl-items > div#101580 (rustdoc: remove unused CSSdiv.impl-items > div)#settings-menu { padding: 0 }#101584 (rustdoc: remove no-op CSS#settings-menu { padding: 0 })Failed merges:
r? @ghost
@rustbot modify labels: rollup
Create a similar rollup