Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove bolding on associated constants #88410

Merged
merged 1 commit into from
Sep 1, 2021
Merged

Conversation

camelid
Copy link
Member

@camelid camelid commented Aug 27, 2021

Associated types don't get bolded, so it looks off to have one kind
bolded and one not.

@camelid camelid added A-associated-items Area: Associated items (types, constants & functions) A-rustdoc-ui Area: Rustdoc UI (generated HTML) T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Aug 27, 2021
@rust-highfive
Copy link
Collaborator

r? @GuillaumeGomez

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 27, 2021
@camelid
Copy link
Member Author

camelid commented Aug 27, 2021

FYI, this is the PR that introduced the <b>: #39944.

@camelid
Copy link
Member Author

camelid commented Aug 27, 2021

The 'Associated Constants' section is unchanged. What changed is the pretty-printed item declaration (that should be the only change).

Before

image

After

image

@GuillaumeGomez
Copy link
Member

Please add tests in src/test/rustdoc to check the absence of the tag (check that the parent tag with the text exists but not the <b> inside it).

@camelid camelid added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 30, 2021
@camelid
Copy link
Member Author

camelid commented Aug 30, 2021

Please add tests in src/test/rustdoc to check the absence of the tag (check that the parent tag with the text exists but not the <b> inside it).

@GuillaumeGomez I added a GUI test case to check the font-weight since that will also catch if the font-weight changes because of CSS.

@camelid camelid added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 30, 2021
@rust-log-analyzer

This comment has been minimized.

Associated types don't get bolded, so it looks off to have one kind
bolded and one not.
@camelid
Copy link
Member Author

camelid commented Aug 31, 2021

@bors r=GuillaumeGomez

@bors
Copy link
Contributor

bors commented Aug 31, 2021

📌 Commit 87e39ac has been approved by GuillaumeGomez

@bors bors 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 Aug 31, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 1, 2021
Rollup of 9 pull requests

Successful merges:

 - rust-lang#86376 (Emit specific warning to clarify that `#[no_mangle]` should not be applied on foreign statics or functions)
 - rust-lang#88040 (BTree: remove Ord bound from new)
 - rust-lang#88053 (Fix the flock fallback implementation)
 - rust-lang#88350 (add support for clobbering xer, cr, and cr[0-7] for asm! on OpenPower/PowerPC)
 - rust-lang#88410 (Remove bolding on associated constants)
 - rust-lang#88525 (fix(rustc_typeck): produce better errors for dyn auto trait)
 - rust-lang#88542 (Use the return value of readdir_r() instead of errno)
 - rust-lang#88548 (Stabilize `Iterator::intersperse()`)
 - rust-lang#88551 (Stabilize `UnsafeCell::raw_get()`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 75b2ae5 into rust-lang:master Sep 1, 2021
@rustbot rustbot added this to the 1.56.0 milestone Sep 1, 2021
@camelid camelid deleted the fix-assoc-bold branch September 1, 2021 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-associated-items Area: Associated items (types, constants & functions) A-rustdoc-ui Area: Rustdoc UI (generated HTML) S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant 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.

6 participants