Skip to content

Fix some assorted rustdoc bugs. #15982

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 7 commits into from
Jul 26, 2014
Merged

Conversation

alexcrichton
Copy link
Member

Sadly there's still a lot of open issues, but this tackles some of the more pressing ones. Each commit has its own description along with the issues it closes.

Apparently keypress doesn't quite work in all browsers due to some not invoking
the handler and jquery not setting the right `which` field in all circumstances.
According to http://stackoverflow.com/questions/2166771 switching over to
`keydown` works and it appears to do the trick. Tested in Safari, Firefox, and
Chrome.

Closes rust-lang#15011
I'm not entirely sure if the correct space can be inferred when cleaning
Generics, so the impl has been switched to take the space explicitly.

Closes rust-lang#15099
These are lang items now, so the internal representations need to be
re-translated back to the original structures manually.

Closes rust-lang#15185
Closes rust-lang#15800
Previously a `pub use` would not rename the destination in rustdoc, it would
always use the destination ident instead of the renamed ident.
These were all just previously skipped.

Closes rust-lang#15648
bors added a commit that referenced this pull request Jul 26, 2014
Sadly there's still a lot of open issues, but this tackles some of the more pressing ones. Each commit has its own description along with the issues it closes.
@bors bors closed this Jul 26, 2014
@bors bors merged commit 98f4863 into rust-lang:master Jul 26, 2014
@alexcrichton alexcrichton deleted the rustdoc-fixes branch August 1, 2014 16:52
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.

3 participants