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

fix(v2): adjust first-level heading offset #2005

Merged
merged 1 commit into from Nov 17, 2019

Conversation

lex111
Copy link
Contributor

@lex111 lex111 commented Nov 17, 2019

Motivation

Currently, the same problem is observed as in v1 (take this opportunity, look at that PR and merge it please!). If during the search we click the page itself (i.e. first-level heading or h1), then we get a URL of the form https://v2.docusaurus.io/docs/introduction/#__docusaurus and the heading is incorrectly positioned (see Plan)

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Before After
image image

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Nov 17, 2019
@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-2 ready!

Built with commit d87ce2b

https://deploy-preview-2005--docusaurus-2.netlify.com

@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-preview ready!

Built with commit d87ce2b

https://deploy-preview-2005--docusaurus-preview.netlify.com

@yangshun yangshun merged commit dda10be into facebook:master Nov 17, 2019
@yangshun yangshun added the pr: bug fix This PR fixes a bug in a past release. label Nov 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: bug fix This PR fixes a bug in a past release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants