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

Fix some styling on the showcase page #7014

Merged
merged 5 commits into from
Aug 13, 2018
Merged

Fix some styling on the showcase page #7014

merged 5 commits into from
Aug 13, 2018

Conversation

alexandernanberg
Copy link
Contributor

What

Some style fixes to the showcase page

  • Align icons properly
  • Fix overlapping filter nav (see screenshot)

Before and after shots

image

We probably want to add some more styling to to search input and some other areas, might create a new PR for that

@KyleAMathews
Copy link
Contributor

KyleAMathews commented Aug 4, 2018

Deploy preview for using-postcss-sass failed.

Built with commit 6b00fda

https://app.netlify.com/sites/using-postcss-sass/deploys/5b71739cc965925091c6960c

@gatsbybot
Copy link
Collaborator

gatsbybot commented Aug 4, 2018

Deploy preview for using-drupal ready!

Built with commit 6b00fda

https://deploy-preview-7014--using-drupal.netlify.com

@gatsbybot
Copy link
Collaborator

gatsbybot commented Aug 4, 2018

Deploy preview for gatsbygram ready!

Built with commit 6b00fda

https://deploy-preview-7014--gatsbygram.netlify.com

@KyleAMathews KyleAMathews requested a review from fk August 8, 2018 07:08
@@ -132,6 +132,10 @@ const styles = {
borderBottom: `none !important`, // i know i know
boxShadow: `none !important`, // but people really want this
},
searchInput: {
paddingLeft: `1.4rem`,
fontFamily: typography.options.headerFontFamily.join(`,`),
Copy link
Contributor

Choose a reason for hiding this comment

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

Hey @alexandernanberg! 👋
Thank you for this — looking good!

Just one request regarding the search input — could you please

  • remove the input's border
  • reduce the input width
  • adjust the input placeholder color
  • add focus styles

so that they match the search form in the header?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good, done that now! 👍

@KyleAMathews
Copy link
Contributor

KyleAMathews commented Aug 11, 2018

Deploy preview for gatsbyjs failed.

Built with commit 6b00fda

https://app.netlify.com/sites/gatsbyjs/deploys/5b71739ac965925091c69600

@alexandernanberg
Copy link
Contributor Author

@fk The search input should look a lot more like the on in the header now. We still might want to change the mobile layout of it though, but that could be another PR I guess.

@KyleAMathews
Copy link
Contributor

Deploy preview for using-remark failed.

Built with commit 6b00fda

https://app.netlify.com/sites/using-remark/deploys/5b71739dc965925091c6961e

Copy link
Contributor

@fk fk left a comment

Choose a reason for hiding this comment

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

Perfect! Thank you @alexandernanberg! 🤗

@fk fk merged commit 5a6ec9c into gatsbyjs:master Aug 13, 2018
@alexandernanberg alexandernanberg deleted the fix/www-showcase branch August 13, 2018 12:19
m-allanson added a commit to phacks/gatsby that referenced this pull request Aug 14, 2018
* master: (597 commits)
  Add a site(https://mojaave.com) to showcase list (gatsbyjs#7275)
  feat: create a doc for open source pair programming sessions (gatsbyjs#7266)
  [docs] Add video lesson to the StaticQuery docs (gatsbyjs#7249)
  [v2] docs - update page query docs (gatsbyjs#7285)
  [v2] docs "Styling" overview (gatsbyjs#7288)
  Remove delay (gatsbyjs#7273)
  add site (gatsbyjs#7291)
  Adding new site to the showcase. (gatsbyjs#7281)
  chore(release): Publish
  initial webpack externals support (gatsbyjs#7245)
  add missing package dependencies (gatsbyjs#7259)
  add: custom configuration overview
  (gatsbyjs#7231): tutorial part four updates (gatsbyjs#7240)
  [www] Fix showcase search, checkbox styles (gatsbyjs#7014)
  (gatsbyjs#6584): Restructure plugin overview and plugin authoring pages (gatsbyjs#7229)
  Use Hubspot form for email subscription (gatsbyjs#7233)
  Adding bootstrap CV starter (gatsbyjs#7207)
  Stub Articles and new names (gatsbyjs#7200)
  Improve readability of verbose logging code in wordpress source plugin (gatsbyjs#7146)
  Update hash link to scroll to right section of page (gatsbyjs#7161)
  ...
porfirioribeiro pushed a commit to porfirioribeiro/gatsby that referenced this pull request Aug 22, 2018
* Fix some styling on the showcase page

* Better search input style

* Better aligned checkbox

* No outline for focused search input
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.

4 participants