Skip to content

rustdoc: remove bad CSS font-weight on .impl, .method, etc - #102442

Merged
bors merged 1 commit into
rust-lang:masterfrom
notriddle:notriddle/header-weight
Sep 29, 2022
Merged

rustdoc: remove bad CSS font-weight on .impl, .method, etc#102442
bors merged 1 commit into
rust-lang:masterfrom
notriddle:notriddle/header-weight

Conversation

@notriddle

Copy link
Copy Markdown
Contributor

This line was added in c494a06, because at the time, the headers had these classes on them. Now, the headers are children of the <section> with the class on it.

This commit also adds a test case, to make sure the srclink font weight does not regress again.

@rustbotrustbot added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Sep 28, 2022
@rust-highfive

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @GuillaumeGomez (or someone else) soon.

Please see the contribution instructions for more information.

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez, @Folyd, @jsha

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 28, 2022
@notriddle
notriddleforce-pushed the notriddle/header-weight branch from 036fe33 to 719c88aCompareSeptember 28, 2022 22:19
@GuillaumeGomez

Copy link
Copy Markdown
Member

Thanks! r=me once CI pass.

@notriddle
notriddleforce-pushed the notriddle/header-weight branch 2 times, most recently from 4a817f4 to 3f21f07CompareSeptember 28, 2022 23:17
This line was added in c494a06, because at
the time, the headers had these classes on them. Now, the headers are
children of the `<section>` with the class on it.
This commit also adds a test case, to make sure the srclink font weight does
not regress again.
@notriddle

Copy link
Copy Markdown
ContributorAuthor

@bors r=GuillaumeGomez rollup

@bors

bors commented Sep 28, 2022

Copy link
Copy Markdown
Collaborator

📌 Commit 3f21f07 has been approved by GuillaumeGomez

It is now in the queue for this repository.

@borsbors 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 Sep 28, 2022
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Sep 29, 2022
… r=GuillaumeGomez
rustdoc: remove bad CSS font-weight on `.impl`, `.method`, etc
This line was added in c494a06, because at the time, the headers had these classes on them. Now, the headers are children of the `<section>` with the class on it.
This commit also adds a test case, to make sure the srclink font weight does not regress again.
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Sep 29, 2022
… r=GuillaumeGomez
rustdoc: remove bad CSS font-weight on `.impl`, `.method`, etc
This line was added in c494a06, because at the time, the headers had these classes on them. Now, the headers are children of the `<section>` with the class on it.
This commit also adds a test case, to make sure the srclink font weight does not regress again.
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Sep 29, 2022
… r=GuillaumeGomez
rustdoc: remove bad CSS font-weight on `.impl`, `.method`, etc
This line was added in c494a06, because at the time, the headers had these classes on them. Now, the headers are children of the `<section>` with the class on it.
This commit also adds a test case, to make sure the srclink font weight does not regress again.
@notriddlenotriddle mentioned this pull request Sep 29, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 29, 2022
Rollup of 7 pull requests
Successful merges:
- rust-lang#102214 (Fix span of byte-escaped left format args brace)
- rust-lang#102426 (Don't export `__wasm_init_memory` on WebAssembly.)
- rust-lang#102437 (rustdoc: cut margin-top from first header in docblock)
- rust-lang#102442 (rustdoc: remove bad CSS font-weight on `.impl`, `.method`, etc)
- rust-lang#102447 (rustdoc: add method spacing to trait methods)
- rust-lang#102468 (tidy: make rustc dependency error less confusing)
- rust-lang#102476 (Split out the error reporting logic into a separate function)
Failed merges:
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit e5fffbe into rust-lang:masterSep 29, 2022
@rustbotrustbot added this to the 1.66.0 milestone Sep 29, 2022
@notriddle
notriddle deleted the notriddle/header-weight branch September 29, 2022 22:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-rustdocRelevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@notriddle@rust-highfive@rustbot@GuillaumeGomez@bors