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 content to placeholder descriptions on ecosystem landing page #9583

Merged
merged 2 commits into from
Nov 1, 2018

Conversation

amberleyromo
Copy link
Contributor

@amberleyromo amberleyromo commented Oct 31, 2018

  • Added content for the descriptions for all three content areas (plugins, starters, external resources)

screen shot 2018-10-30 at 7 10 30 pm

@amberleyromo amberleyromo requested a review from a team as a code owner October 31, 2018 00:10
@amberleyromo
Copy link
Contributor Author

@shannonbux would appreciate your eyes/approval. tried to make them all the same length!

Copy link
Contributor

@m-allanson m-allanson left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

This looks great to my 👀

@DSchau DSchau merged commit ec6df74 into gatsbyjs:master Nov 1, 2018
jedrichards pushed a commit to jedrichards/gatsby that referenced this pull request Nov 1, 2018
…tsbyjs#9583)

- Added content for the descriptions for all three content areas (plugins, starters, external resources)

<img width="1549" alt="screen shot 2018-10-30 at 7 10 30 pm" src="https://user-images.githubusercontent.com/3461087/47758106-92634b80-dc77-11e8-9fa5-17b63ea9ef81.png">
@@ -55,7 +55,7 @@ const EcosystemBoard = ({
/>
<EcosystemSection
title="External Resources"
description="More awesome Gatsby content, created by the community."
description="A curated list of interesting Gatsby community projects and learning resources like podcasts and tutorials."
Copy link
Contributor

Choose a reason for hiding this comment

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

I like that this is more specific than what existed before; it strikes me as just a bit long. What about getting rid of the word "interesting" (since that's a given :) and then making it into a list with commas, like "a curated list of Gatsby community projects, like podcasts, tutorials, and other learning resources"

Ok maybe that's longer now. I love commas, so sue me!

gpetrioli pushed a commit to gpetrioli/gatsby that referenced this pull request Jan 22, 2019
…tsbyjs#9583)

- Added content for the descriptions for all three content areas (plugins, starters, external resources)

<img width="1549" alt="screen shot 2018-10-30 at 7 10 30 pm" src="https://user-images.githubusercontent.com/3461087/47758106-92634b80-dc77-11e8-9fa5-17b63ea9ef81.png">
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants