Skip to content

Call to action button now "reacts" to being hovered :) - #6810

Merged
amyrlam merged 4 commits into
react:masterfrom
joerez:patch-2
Apr 25, 2019
Merged

Call to action button now "reacts" to being hovered :)#6810
amyrlam merged 4 commits into
react:masterfrom
joerez:patch-2

Conversation

@joerez

@joerez joerez commented Apr 13, 2019

Copy link
Copy Markdown
Contributor

Added a slight scale up on the get started button upon hover.

Slight scale up on the get started button upon hover.
@joerez
joerez requested review from amyrlam and iansu as code owners April 13, 2019 05:39
@facebook-github-bot

Copy link
Copy Markdown

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

@facebook-github-bot

Copy link
Copy Markdown

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

1 similar comment
@facebook-github-bot

Copy link
Copy Markdown

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@amyrlam

amyrlam commented Apr 25, 2019

Copy link
Copy Markdown
Contributor

It's strange there is no change on hover!

Maybe we should go with a simpler fix - perhaps a color change, for us turn white on hover - similar to https://docusaurus.io/ or https://sorbet.org/. That would also match the styling of the edit button on https://facebook.github.io/create-react-app/docs/getting-started. What do you think?

@amyrlam amyrlam self-assigned this Apr 25, 2019
@joerez

joerez commented Apr 25, 2019

Copy link
Copy Markdown
Contributor Author

It's strange there is no change on hover!

Maybe we should go with a simpler fix - perhaps a color change, for us turn white on hover - similar to https://docusaurus.io/ or https://sorbet.org/. That would also match the styling of the edit button on https://facebook.github.io/create-react-app/docs/getting-started. What do you think?

I think that's a good idea!

@joerez

joerez commented Apr 25, 2019

Copy link
Copy Markdown
Contributor Author

@amyrlam That change has been made.

@amyrlam amyrlam left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Made a couple suggestions, let me know.

Thanks for making the changes! 👍

Comment thread docusaurus/website/static/css/custom.css Outdated
Comment thread docusaurus/website/static/css/custom.css Outdated
amyrlam and others added 2 commits April 25, 2019 08:43
Co-Authored-By: joerez <rezenbiz@gmail.com>
Co-Authored-By: joerez <rezenbiz@gmail.com>
@joerez

joerez commented Apr 25, 2019

Copy link
Copy Markdown
Contributor Author

Nit is good. Suggestions added.

@amyrlam amyrlam added this to the 3.0.1 milestone Apr 25, 2019

@amyrlam amyrlam left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💯

@amyrlam
amyrlam merged commit 18b399b into react:master Apr 25, 2019
@lock lock Bot locked and limited conversation to collaborators Apr 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants