Skip to content

Rustdoc: search page loads docs page before search results #45971

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Closed
nrc opened this issue Nov 13, 2017 · 0 comments
Closed

Rustdoc: search page loads docs page before search results #45971

nrc opened this issue Nov 13, 2017 · 0 comments
Assignees
@GuillaumeGomez
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

@nrc
Copy link
Member

nrc commented Nov 13, 2017

E.g., if you navigate to https://doc.rust-lang.org/nightly/std/index.html?search=Vec, then you see the docs for the std page for a second or two (longer if you're internet is bad, I assume) before you get the search results for Vec.

cc @GuillaumeGomez
@nrc nrc added T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-dev-tools-rustdoc labels Nov 13, 2017
@GuillaumeGomez GuillaumeGomez self-assigned this Nov 13, 2017
@TimNN TimNN added the C-enhancement Category: An issue proposing an enhancement or a PR with one. label Nov 14, 2017
@GuillaumeGomez GuillaumeGomez mentioned this issue Nov 23, 2017
Merged
bors added a commit that referenced this issue Dec 7, 2017
@bors
Auto merge of #46221 - GuillaumeGomez:doc-script-load-speedup, r=nrc
bb1bd88 
Speedup search loading when search url is received

Fixes #45971.

r? @nrc

cc @rust-lang/docs
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@GuillaumeGomez GuillaumeGomez

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
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nrc @TimNN @GuillaumeGomez