Skip to content

fix(v2): fix hash-link css#1387

Merged
yangshun merged 5 commits intofacebook:masterfrom
endiliey:hash-link
Apr 24, 2019
Merged

fix(v2): fix hash-link css#1387
yangshun merged 5 commits intofacebook:masterfrom
endiliey:hash-link

Conversation

@endiliey
Copy link
Contributor

Motivation

Currently, the hash-link is so hard to click.

See this gif or try it out at https://docusaurus-2.netlify.com/docs/installation/#launching-the-server-behind-a-proxy

Before I can click the hash, it already disappears
so hard to click

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Before
have to do it really fast

After

easier

Related PRs

#1385

@endiliey endiliey requested a review from yangshun as a code owner April 24, 2019 10:28
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Apr 24, 2019
@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Apr 24, 2019

Deploy preview for docusaurus-2 failed.

Built with commit 0b6a094

https://app.netlify.com/sites/docusaurus-2/deploys/5cc096322150b3017dedccc1

@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Apr 24, 2019

Deploy preview for docusaurus-preview ready!

Built with commit 0b6a094

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

transition: opacity 0.3s;
}

@media only screen and (min-width: 736px) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this removed? This is for the hover behavior only on desktop widths since there's no hover on mobile.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dank that was true :| i forget hover doesnt work on mobile

endiliey and others added 4 commits April 24, 2019 08:26
@yangshun yangshun merged commit cb8c712 into facebook:master Apr 24, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants