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

Migrate testimonials page to clients page #24

Merged
merged 2 commits into from
Oct 27, 2020
Merged

Conversation

ericmjl
Copy link
Collaborator

@ericmjl ericmjl commented Oct 27, 2020

Closes #20.

This was a bit harder than I had originally anticipated, but managed to get it to work.

I have used Card Columns as the layout format. Gives us a Pintrest-like view over clients. If testimonial is provided, they are shown, otherwise just logos.

Logos have a 10 px border around them added in programmatically by CSS. This helps; previously, the logos were touching the card borders directly.

Copy link
Contributor

@OriolAbril OriolAbril left a comment

Choose a reason for hiding this comment

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

Looks good, only nit is I would make the logos clickable if possible.

@ericmjl
Copy link
Collaborator Author

ericmjl commented Oct 27, 2020

@OriolAbril ok, no problemo!

@twiecki twiecki merged commit 8c1f008 into master Oct 27, 2020
@twiecki twiecki deleted the refactor-clients-age branch October 27, 2020 22:59
@twiecki
Copy link
Contributor

twiecki commented Oct 27, 2020

Very cool, thanks for the quick turn-around!

@ericmjl
Copy link
Collaborator Author

ericmjl commented Oct 27, 2020

Yes, that's right, it's changed to /clients.

@ericmjl
Copy link
Collaborator Author

ericmjl commented Oct 27, 2020

Though I'm noticing that the Akiri logo is down.

@twiecki
Copy link
Contributor

twiecki commented Oct 27, 2020 via email

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.

Testimonials should just be "clients"
3 participants