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

[www] Site/Starter Showcase refactor, rinse II #8411

Merged
merged 2 commits into from
Sep 26, 2018

Conversation

fk
Copy link
Contributor

@fk fk commented Sep 21, 2018

  • fix Starter showcase - tweak title #7754 - change the Starter Showcase main content headline from "All 60 Starters" to “Gatsby Starters (60)”
  • DRY — add a couple of shared components…
    • ResetFilters button
    • SidebarContainer, SidebarHeader, SidebarBody
    • ContentHeader and ContentTitle („Starters“/„Sites“) and count, adjusting title on filter and search
  • link Creator Showcase detail page GitHub icon
  • hide the Site/Starter Showcase sidebar "Filter & Refine" header for now

Here's a screenshot showing the changes for #7754 before/after (also, the Site Showcase headers now behave exactly the same):

  • No filter/search: image
  • With filters applied: image

* fix #7754 - change the Starter Showcase main content headline from "All 60 Starters" to “Gatsby Starters (60)”
* DRY — add a couple of shared components:
  * ResetFilters button
  * SidebarContainer, SidebarHeader, SidebarBody
  * ContentHeader with title („Starters“/„Sites“) and count, adjusting title on filter and search
* link Creator Showcase detail page GitHub icon
@fk fk requested a review from amberleyromo September 21, 2018 15:31
Copy link
Contributor

@amberleyromo amberleyromo left a comment

Choose a reason for hiding this comment

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

Awesome, Flo. I really like getting rid of the "Filter + Refine header" and the other slight tweaks.

@fk
Copy link
Contributor Author

fk commented Sep 21, 2018

Yay! Thanks Amberley!
I'll leave it up to you wether to merge this before #8320 or not — AFAICS the variable naming change from item to starter might be the biggest source for conflicts. Happy to help resolving the latter either way!

@fk fk requested a review from a team as a code owner September 25, 2018 14:46
@amberleyromo
Copy link
Contributor

@fk let's not block this. i have no issue with resolving the conflicts later -- i'm currently blocked on #8320 by #8517, which is going to get resolved later this week (hopefully tomorrow).

@fk
Copy link
Contributor Author

fk commented Sep 25, 2018

👌

@fk fk merged commit 92f3790 into master Sep 26, 2018
lipis added a commit to lipis/gatsby that referenced this pull request Sep 26, 2018
* 'master' of github.com:gatsbyjs/gatsby:
  feat(cache-dir): Add a button for custom 404s in development (gatsbyjs#8548)
  fix: add ecosystem team for starters YAML
  fix: fix displaying and hydrating 404 page in production builds (gatsbyjs#8510)
  fix: don't expect `application/json` type nodes to be files (gatsbyjs#8544)
  Delete cds-takeaways.md (gatsbyjs#8552)
  chore(release): Publish
  chore: remove npm-run-all dep and update deps in e2e tests (gatsbyjs#8529)
  [www] Site/Starter Showcase refactor, rinse II (gatsbyjs#8411)
  Rfc process doc (gatsbyjs#8537)
  [www] a11y improvements (gatsbyjs#8536)
  Update the GraphQL stitching blog post & docs (gatsbyjs#8516)
  fix: change teams to reduce noise for people
@fk fk deleted the topics/www-showcase-refactor-rinse-2 branch October 12, 2018 19:30
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.

Starter showcase - tweak title
2 participants