Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

Improve text on the welcome page #931

Closed
wants to merge 1 commit into from
Closed

Improve text on the welcome page #931

wants to merge 1 commit into from

Conversation

psrpinto
Copy link
Contributor

The text on the welcome page was not a proper English sentence. I took the chance to also improve the "read the docs" sentence to not mention Symfony twice.

@javiereguiluz
Copy link
Member

@regularjack thanks for your pull request. Before merging it, I'd like to ask the review from @weaverryan, our native English speaker. Thanks!

@@ -11,7 +11,7 @@
<p>
<svg id="icon-status" width="1792" height="1792" viewBox="0 0 1792 1792" xmlns="http://www.w3.org/2000/svg"><path d="M1671 566q0 40-28 68l-724 724-136 136q-28 28-68 28t-68-28l-136-136-362-362q-28-28-28-68t28-68l136-136q28-28 68-28t68 28l294 295 656-657q28-28 68-28t68 28l136 136q28 28 28 68z" fill="#759E1A"/></svg>

Your application is ready to start working on it at:
You're ready to start working on your application at:
Copy link
Member

Choose a reason for hiding this comment

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

I would split this in 2 sentences, something along the lines of:

"Your application is ready now. Start working on it at:"

@Pierstoval
Copy link
Contributor

Apart @fabpot's comment, I'm 👍 on this

@psrpinto
Copy link
Contributor Author

I also agree with @fabpot's comment. Just fixed it.

@wouterj
Copy link
Member

wouterj commented Jan 19, 2016

👍

@xabbuh
Copy link
Member

xabbuh commented Jan 19, 2016

👍 sounds better to me too

@fabpot
Copy link
Member

fabpot commented Jan 19, 2016

Thank you @regularjack.

fabpot added a commit that referenced this pull request Jan 19, 2016
This PR was submitted for the 3.0 branch but it was merged into the 2.3 branch instead (closes #931).

Discussion
----------

Improve text on the welcome page

The text on the welcome page was not a proper English sentence. I took the chance to also improve the "read the docs" sentence to not mention Symfony twice.

Commits
-------

becd595 Improve text on the welcome page
@fabpot fabpot closed this Jan 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants