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

155 press #191

Merged
merged 9 commits into from
Nov 30, 2020
Merged

155 press #191

merged 9 commits into from
Nov 30, 2020

Conversation

ann-kilzer
Copy link
Collaborator

@ann-kilzer ann-kilzer commented Oct 11, 2020

Resolves #155

Proposed Changes

Add a section for highlighting SpeakHer in the press

Description of changes in this PR

  • Carousel for displaying press items
  • Link to external pages in a new tab
  • Adjust image size on mobile
  • Consider variations of title layout
  • Add more sources
  • Figure out image loading 🏉 punt to later PR
  • Improved colors

I would eventually like to not load images from external sites, but I was having some trouble with it, so let's solve it in a follow up PR.

Testing Plan

Please view the page on desktop and mobile view, and verify the links work.

Screenshot:

SpeakHer

@ann-kilzer ann-kilzer added the WIP Work In Progress label Oct 11, 2020
@ann-kilzer ann-kilzer self-assigned this Oct 11, 2020
@ann-kilzer ann-kilzer removed the WIP Work In Progress label Oct 31, 2020
Copy link
Collaborator

@tuttiq tuttiq left a comment

Choose a reason for hiding this comment

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

Ok, let's move forward with this and think about a different design later! Thanks @ann-kilzer !

@tuttiq tuttiq mentioned this pull request Nov 29, 2020
@ann-kilzer ann-kilzer merged commit 2759cb8 into master Nov 30, 2020
@ann-kilzer ann-kilzer deleted the 155-press branch November 30, 2020 15:18
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.

As a visitor, I should see the logos of publications that SpeakHer has been featured in.
2 participants