Skip to content

navigation_rustdoc: show whether a build failed in the Versions list #562

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

Merged
merged 1 commit into from
Jan 11, 2020
Merged

navigation_rustdoc: show whether a build failed in the Versions list #562

merged 1 commit into from
Jan 11, 2020

Conversation

yvrhdn
Copy link

@yvrhdn yvrhdn commented Jan 11, 2020

Changes in this PR are very similar to #546. In fact, the code introduced here is exactly the same as in crate_details.hbs

This PR impacts the navigation bar in the rustdoc view: in the dropdown menu, versions that failed to build will also have a ⚠️ sign now.

Screenshot taken with changes from this PR:

Screenshot 2020-01-11 at 04 30 58

This introduces the same change as
3cf195e
to navigation_rustoc.
@yvrhdn
Copy link
Author

yvrhdn commented Jan 11, 2020

We could avoid the code duplication by extracting this part into a separate .hbs file. But I'm not sure if that is worth doing for about 10 lines of code.
I also don't see any other pages doing that.

@jyn514
Copy link
Member

jyn514 commented Jan 11, 2020

I think for this little code it's fine to have some duplication.

@jyn514 jyn514 merged commit 7e16fc6 into rust-lang:master Jan 11, 2020
@yvrhdn yvrhdn deleted the navigation_rustdoc_warning branch January 11, 2020 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants