Skip to content

Improve mobile-friendliness of API docs #21597

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 28, 2015
Merged

Improve mobile-friendliness of API docs #21597

merged 1 commit into from
Jan 28, 2015

Conversation

iKevinY
Copy link
Contributor

@iKevinY iKevinY commented Jan 24, 2015

This PR removes the min-width rule from body so that no horizontal scrolling is necessary on mobile, and also hides out-of-band information on mobile to create more room for the in-band information.

@rust-highfive
Copy link
Contributor

r? @cmr

(rust_highfive has picked a reviewer for you, use r? to override)

Removes min-width rule from body so that the page does not scroll
horizontally on mobile displays; hides out-of-band information.
@iKevinY
Copy link
Contributor Author

iKevinY commented Jan 24, 2015

@cmr Sorry, I just noticed there was already a media query for mobile, so I've moved the CSS rules there and rebased the commit.

@emberian
Copy link
Member

@bors r+ 1ec13e7 rollup

barosl added a commit to barosl/rust that referenced this pull request Jan 27, 2015
This PR removes the `min-width` rule from `body` so that no horizontal scrolling is necessary on mobile, and also hides out-of-band information on mobile to create more room for the in-band information.
Manishearth added a commit to Manishearth/rust that referenced this pull request Jan 27, 2015
This PR removes the `min-width` rule from `body` so that no horizontal scrolling is necessary on mobile, and also hides out-of-band information on mobile to create more room for the in-band information.
@bors bors merged commit 1ec13e7 into rust-lang:master Jan 28, 2015
@iKevinY iKevinY deleted the mobile-api-docs branch January 29, 2015 03:19
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.

4 participants