Skip to content

Conversation

@GuillaumeGomez
Copy link
Member

When there is only one crate, the dropdown is removed, creating an error (that you can see pretty easily on docs.rs for example).

r? @jyn514

@GuillaumeGomez GuillaumeGomez added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Mar 11, 2021
@rust-highfive
Copy link
Contributor

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 11, 2021
@Nemo157
Copy link
Contributor

Nemo157 commented Mar 13, 2021

It's not when there's only one crate, it's when you explicitly pass --disable-per-crate-search (as docs.rs does), this looks like it should fix #82986

@GuillaumeGomez GuillaumeGomez changed the title Prevent JS error when there is no dependency or other crate documented Prevent JS error when there is no dependency or other crate documented (or --disable-per-crate-search has been used) Mar 13, 2021
@GuillaumeGomez
Copy link
Member Author

@bors: r=Nemo157

@bors
Copy link
Collaborator

bors commented Mar 13, 2021

📌 Commit 56898ec 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 Mar 13, 2021
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Mar 13, 2021
…no-filter, r=Nemo157

Prevent JS error when there is no dependency or other crate documented (or --disable-per-crate-search has been used)

When there is only one crate, the dropdown is removed, creating an error (that you can see pretty easily on docs.rs for example).

r? `@jyn514`
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Mar 13, 2021
…no-filter, r=Nemo157

Prevent JS error when there is no dependency or other crate documented (or --disable-per-crate-search has been used)

When there is only one crate, the dropdown is removed, creating an error (that you can see pretty easily on docs.rs for example).

r? ``@jyn514``
@Dylan-DPC-zz
Copy link

@bors p=1 rollup=never

(Bumping this is rustdoc team needs it urgently :)

@bors
Copy link
Collaborator

bors commented Mar 14, 2021

⌛ Testing commit 56898ec with merge 96d67448817268496c3e50d13a6b84a9ed6264a2...

@JohnTitor
Copy link
Member

@bors retry yielding

@rust-log-analyzer
Copy link
Collaborator

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@bors
Copy link
Collaborator

bors commented Mar 14, 2021

⌛ Testing commit 56898ec with merge 2caeeb0...

@bors
Copy link
Collaborator

bors commented Mar 14, 2021

☀️ Test successful - checks-actions
Approved by: Nemo157
Pushing 2caeeb0 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Mar 14, 2021
@bors bors merged commit 2caeeb0 into rust-lang:master Mar 14, 2021
@rustbot rustbot added this to the 1.52.0 milestone Mar 14, 2021
@GuillaumeGomez GuillaumeGomez deleted the prevent-js-error-if-no-filter branch March 14, 2021 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-bors This PR was explicitly merged by bors. 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.

9 participants