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

Add some front page content #5398

Merged
merged 7 commits into from
Oct 11, 2018
Merged

Conversation

selbekk
Copy link
Contributor

@selbekk selbekk commented Oct 11, 2018

Preview: https://create-react-app-stngeqjqos.now.sh

This pull request adds some content to the front page, and fixes a design faux-pas with the mobile navigation.

This doesn't need to be the final content, of course, but it's a lot better than not having anything 👍

I created the update screenshot with carbon, which can be updated quite easily later if we want.

Addresses #5238

content: `With Create React App, you get to focus on **writing React, not boilerplate**.
All you need to do is run a command, install some dependencies, and decide what's for dinner.

npx create-react-app my-app
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can tweak the CSS so the left hand side margin isn't so large in code blocks, what do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure I'll try to tune it down a bit :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, that's actually indentation - I'll just remove the indentation and we should be good 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

Cool, yeah it seemed not as large in the .md files.

How did you generate https://create-react-app-stngeqjqos.now.sh/? When poking around, I followed this and pulled your branch locally.

Copy link
Contributor Author

@selbekk selbekk Oct 11, 2018

Choose a reason for hiding this comment

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

How did you generate https://create-react-app-stngeqjqos.now.sh/?

I use now from zeit - it's free and pretty easy to use. Once you've installed their CLI and logged in, you can do something like:

cd docusaurus/webpage
yarn build # to generate the html files
cd build/create-react-app # this is where the built files are
now # deploy to a unique URL

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks! I'll have to check that out, and carbon too

{
title: 'Webpack 4',
content:
'Webpack 4 gives you lightning fast rebuilds and code-splitting out of the box',
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider punctuation (e.g. add periods) throughout to be consistent?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I landed on removing the punctuation that was there, as i thought it looked better with no punctuation than a period after a ton of one-sentence paragraphs. Totally agree that it should be consistent!

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks

I guess not sure since some have two sentences? If you prefer no periods, prob better to just leave it as it was... periods for the two sentences, no punctuation for the others...

Copy link
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, i'd say let's get this in!

@amyrlam amyrlam merged commit 358ed99 into facebook:docusaurus Oct 11, 2018
@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