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

Removes jump to #1712

Closed
wants to merge 1 commit into from
Closed

Removes jump to #1712

wants to merge 1 commit into from

Conversation

mparke
Copy link
Contributor

@mparke mparke commented May 8, 2017

closes #1670

Instead of fixing the jump links, which you can see a version of here: master...mparke:fix-docs-jump-links

this pull opts to remove them, because for the majority of endpoint pages with multiple methods, GET, POST, etc, each method is already visible on the page or is visible by scrolling slightly. The jump doesn't have much effect in most cases.

@mparke
Copy link
Contributor Author

mparke commented May 8, 2017

Closing in favor of #1713

after testing smaller screens

@mparke mparke closed this May 8, 2017
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.

1 participant