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

Moment Template Banner align choice cards (only) top with article count/body top #930

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

paul-daniel-dempsey
Copy link
Contributor

@paul-daniel-dempsey paul-daniel-dempsey commented Jul 18, 2023

What does this change?

Update our new Moment Templates w/ Choice Cards so the top of the choice cards component is vertically aligned to the top of the article count copy, to it’s left, and if there is no article count copy it should be vertically aligned to the top of the main banner copy, also to it’s left. Currently we vertically centre this component.

before
image

after (with article count)
image

Accessibility test checklist

@paul-daniel-dempsey paul-daniel-dempsey changed the title initial upload Moment Template Banner vertically align the top of choice cards in our templates Jul 18, 2023
@paul-daniel-dempsey paul-daniel-dempsey changed the title Moment Template Banner vertically align the top of choice cards in our templates Moment Template Banner align choice cards (only) top with article count/body top Jul 18, 2023
@paul-daniel-dempsey
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants