Skip to content
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

Can't type the letter "s" #190

Closed
ajgajg1134 opened this issue Mar 28, 2018 · 11 comments
Closed

Can't type the letter "s" #190

ajgajg1134 opened this issue Mar 28, 2018 · 11 comments

Comments

@ajgajg1134
Copy link

If you go to a docs.rs page, click into the search box the letter "s" doesn't type. (I assume this has something to do with the way "S" is used as a shortcut for search. (Sometimes it just works, I haven't found the pattern, but this page is an example of it not working: https://docs.rs/yup-oauth2/1.0.7/yup_oauth2/?search=)

Tested on Firefox and Chrome on two different Windows machines.

@onur
Copy link
Member

onur commented Mar 28, 2018

Hi @ajgajg1134, thanks for reporting.

rustc used in docs.rs have this issue and I believe @GuillaumeGomez fixed this issue last week (rust-lang/rust#49152). Updating nightly in docs.rs should fix this issue.

@GuillaumeGomez
Copy link
Member

I confirm it has been fixed.

@ruuda
Copy link

ruuda commented Apr 18, 2018

I still cannot type an s in the search box at docs.rs. Will this situation resolve itself automatically at some point, or do all of the bad docs have to be rebuilt?

Related issue (resolved): rust-lang/rust#49075.

@ivanovaleksey
Copy link

ivanovaleksey commented Apr 19, 2018

According to this comment some upgrades should be done on docs.rs side to support fix was made.

@CAD97
Copy link

CAD97 commented Apr 21, 2018

For people dealing with the issue: The search results page doesn't have the issue, so typing anything that isn't an s then waiting will allow you to type ss.

I can confirm that a crate released 4 hours ago's documentation on docs.rs still has this issue, so we're waiting somewhat patiently for a rustdoc update in the docs.rs pipeline (and then a rebuild for every crate's documentation that got built by the bad rustdoc...)

@ivanovaleksey
Copy link

@CAD97 is rebuilding everything is possible? I believe just fixing the issue would be enough.
Crates documentation would be updated on new releases.

@CAD97
Copy link

CAD97 commented Apr 21, 2018

@ivanovaleksey Yeah, just updating rustdoc will fix it for new releases, but that also means that any crate that got hit by the issue would have to notice that they did and release a new minor version just to fix the docs.

Either way, for the issue to not happen on any of docs.rs' documentation, the user has to wait for a new build of the documentation, whether that's because all bad documentation was rebuilt automatically or the crate owner pushed a new version (and even then, the old version's documentation is still available).

@sinesc
Copy link

sinesc commented May 12, 2018

This also happens with "-" which expands/closes description blocks.

@z2oh
Copy link

z2oh commented Sep 25, 2018

Is rebuilding the docs for old crates a possibility? There are lots of crates that still haven't had an update since this bug was fixed.

@jyn514
Copy link
Member

jyn514 commented Dec 7, 2019

@z2oh rebuilding automagically is #464

(also, hi Jeremy 😆)

@jyn514
Copy link
Member

jyn514 commented Dec 7, 2019

Both 's' and '-' are fixed on the latest version.

@jyn514 jyn514 closed this as completed Dec 7, 2019
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

No branches or pull requests

9 participants