Skip to content

Improve stability of rustdoc #50023

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

Closed
Diggsey opened this issue Apr 17, 2018 · 8 comments
Closed

Improve stability of rustdoc #50023

Diggsey opened this issue Apr 17, 2018 · 8 comments
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@Diggsey
Copy link
Contributor

Diggsey commented Apr 17, 2018

Recently there have been a lot of issues introduced with rustdoc.

Reported issues:

Additional issues:

  • Docs are now incredibly slow to load
  • Search bar is bugged (cannot type 's' into the search bar!)
  • Anchor links broken to collapsed sections
  • Search showing duplicate results
  • Some items are shown in search results that don't exist on the page it links to

It makes me really sad that the documentation is now so much worse to use than it used to be.

If this is because development of rustdoc can't keep up with changes to the compiler internals, then there's not really anything that can be done here, but from an outside perspective it seems like many of these issues stem from new features being added to rustdoc, many of which have... questionable value, and are not sufficiently tested.

The problem is made worse because https://docs.rs/ uses the nightly compiler to build its documentation. I don't want to see this change because a lot of crates are nightly-only, or have nightly-specific features that it is important to document, but the net result is that the canonical documentation for many rust crates is now almost unusable, especially on slow internet connections.

@GuillaumeGomez
Copy link
Member

Bugs are generally fixed in the next days they're reported. However, if they hit beta release, then it'll take 12 weeks to be fixed in stable. For docs.rs, we work with the current maintainer and he makes release when he thinks they're worth enough an internal release.

@Xaeroxe
Copy link
Contributor

Xaeroxe commented Apr 17, 2018

Search bar is bugged (cannot type 's' into the search bar!)

This is infuriating, I'd like to get this fixed ASAP. Are there any other relevant issues or PRs I can chime in on or help with? This is the only issue I was able to find that mentioned this.

@GuillaumeGomez
Copy link
Member

@Xaeroxe: It's been fixed weeks ago. Refer to my previous message for more details.

@varkor
Copy link
Member

varkor commented Apr 18, 2018

Would it plausible to separate the UI from the content/generation/rustc integration, so that UI bug fixes or tweaks could be made available much sooner?

@GuillaumeGomez
Copy link
Member

It could but it needs to be discussed. Generally they get backported. This case is an exception since no one found out the bug before a few versions passed (which is strange).

@QuietMisdreavus QuietMisdreavus added T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. and removed T-rustdoc labels May 15, 2018
@QuietMisdreavus
Copy link
Member

To me this issue seems like it doesn't have a concrete suggestion to make? At best it's an aggregation of other issues that were reported elsewhere. If you have issues to report, report them separately, so that we can tackle them one-by-one.

As for the thrust of the OP, rustdoc doesn't have the same "stability" concerns that the compiler does; we add features as we see fit and change the display just as often. In fact, when we were discussing the --extend-css option, it was explicitly said that we don't have a guarantee for the exact HTML of the output. To provide a stability story for rustdoc, we'll need to define what that actually means.

@XAMPPRocky XAMPPRocky added the C-enhancement Category: An issue proposing an enhancement or a PR with one. label Aug 27, 2018
@kzys
Copy link
Contributor

kzys commented Sep 27, 2018

I agree with @QuietMisdreavus. This issue is not actionable.

Regarding regressions, Rustdoc will have Puppeteer-based tests (#53319) that would allow us to make sure that all features (incl. JavaScript) are working, while we need to write tests first though.

@Mark-Simulacrum
Copy link
Member

Closing per the above, as this is not actionable. We do our best :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

9 participants