Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

Partner list redesign #70

Merged
merged 4 commits into from
Apr 8, 2022
Merged

Conversation

curtisbarnard
Copy link
Contributor

Description

Type of changes

  • Bug fix
  • New Feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Screenshots (optional)

I'm having an issue getting the 3+2 config to work on desktop. The wrapper is set at a fixed width and so the section aligns relative to it instead of centering:

Wrapper Layout:
wrapper dims

3+2 Layout:
container

I'm wondering if we could change the wrapper to displaying a flexbox column that has it's items aligned center? Or perhaps theres a better way to solve this that isn't coming to my mind. Any ideas @NickSchimek or @kgottfri ?

@curtisbarnard curtisbarnard changed the base branch from master to redesign_test April 8, 2022 01:33
@curtisbarnard
Copy link
Contributor Author

Wrap section in an 80% width div and center the tiles.

@curtisbarnard curtisbarnard marked this pull request as ready for review April 8, 2022 23:44
@curtisbarnard
Copy link
Contributor Author

Cards look good
image

Going to merge with redesign_test

@curtisbarnard curtisbarnard merged commit c5575ba into redesign_test Apr 8, 2022
@curtisbarnard curtisbarnard deleted the partner_list-redesign branch April 8, 2022 23:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adjust "Our Partners" list to display as shown
2 participants