Skip to content

Add callouts and logo#5341

Merged
selbekk merged 2 commits intofacebook:docusaurusfrom
selbekk:add-callouts-and-logo
Oct 8, 2018
Merged

Add callouts and logo#5341
selbekk merged 2 commits intofacebook:docusaurusfrom
selbekk:add-callouts-and-logo

Conversation

@selbekk
Copy link
Copy Markdown
Contributor

@selbekk selbekk commented Oct 7, 2018

localhost_3000_

This pull request simplifies the front page of the docs a bit, as it contained a few sections we don't need for a first release.

The React logo was borrowed from the template folder - I guess it's fine to duplicate like this?

Adresses #5238

Copy link
Copy Markdown
Contributor

@amyrlam amyrlam left a comment

Choose a reason for hiding this comment

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

looks good! 👍

@amyrlam
Copy link
Copy Markdown
Contributor

amyrlam commented Oct 8, 2018

Not sure about the Travis failure https://travis-ci.org/facebook/create-react-app/jobs/438291393#L2874, but it seems like it can be merged to docusaurus

@selbekk
Copy link
Copy Markdown
Contributor Author

selbekk commented Oct 8, 2018

Yeah all of my PRs have failed their Travis check, but it doesn't look like anything related to my code.

Merging then :)

@selbekk selbekk merged commit bc1b021 into facebook:docusaurus Oct 8, 2018
@selbekk selbekk deleted the add-callouts-and-logo branch October 8, 2018 05:31
@lock lock Bot locked and limited conversation to collaborators Jan 18, 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