Skip to content

Conversation

jsha
Copy link
Contributor

@jsha jsha commented Jan 20, 2022

For all table layouts, set overflow-wrap: break-word.

Fixes #93135

Demo: https://rustdoc.crud.net/jsha/fix-wrapped-names/std/intrinsics/index.html#functions

(Compare vs https://doc.rust-lang.org/nightly/std/intrinsics/index.html - you may have to make your browser narrower to see the effect)

r? @Nemo157

@rustbot rustbot added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Jan 20, 2022
@rust-highfive
Copy link
Contributor

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez,@Folyd

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 20, 2022
@rust-log-analyzer

This comment has been minimized.

@jsha jsha force-pushed the fix-wrapped-names branch from a49860f to c755dd3 Compare January 20, 2022 23:34
For all table layouts, set overflow-wrap: break-word.
@jsha jsha force-pushed the fix-wrapped-names branch from c755dd3 to 9d178e5 Compare January 20, 2022 23:53
@euclio
Copy link
Contributor

euclio commented Jan 21, 2022

Can we ensure that the line is not broken between the stability/portability "pills" and the item name? Looks weird when they get broken to the next line:

Screen Shot 2022-01-21 at 12 50 45 PM

@jsha
Copy link
Contributor Author

jsha commented Jan 21, 2022

@euclio - yep, we should fix that! But that's an already-existing problem. This is a narrowly targeted break-fix for a recently introduced problem. Want to file an issue?

@Nemo157
Copy link
Member

Nemo157 commented Jan 21, 2022

lgtm

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 21, 2022

📌 Commit 9d178e5 has been approved by Nemo157

@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 Jan 21, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 21, 2022
…askrgr

Rollup of 10 pull requests

Successful merges:

 - rust-lang#91965 (Add more granular `--exclude` in `x.py`)
 - rust-lang#92467 (Ensure that early-bound function lifetimes are always 'local')
 - rust-lang#92586 (Set the allocation MIN_ALIGN for espidf to 4.)
 - rust-lang#92835 (Improve error message for key="value" cfg arguments.)
 - rust-lang#92843 (Improve string concatenation suggestion)
 - rust-lang#92963 (Implement tuple array diagnostic)
 - rust-lang#93046 (Use let_else in even more places)
 - rust-lang#93109 (Improve `Arc` and `Rc` documentation)
 - rust-lang#93134 (delete `Stdin::split` forwarder)
 - rust-lang#93139 (rustdoc: fix overflow-wrap for table layouts)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 26e9357 into rust-lang:master Jan 22, 2022
@rustbot rustbot added this to the 1.60.0 milestone Jan 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

Item names in module lists get prematurely wrapped on nightly docs
7 participants