-
Notifications
You must be signed in to change notification settings - Fork 20
Add govuk app promo to sign-in page #5035
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
Conversation
c587150
to
cdd9290
Compare
cd1cc67
to
88faf5c
Compare
88faf5c
to
997533c
Compare
LGTM 🚀 |
997533c
to
3e1495b
Compare
3e1495b
to
da70cdf
Compare
da70cdf
to
4c5fe2e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me, thanks! I’ve added two minor, non-blocking comments, but it’s fine to merge as is.
href: "/guidance/download-the-govuk-app", | ||
image_src: asset_path("app-promo/govuk-app-icon.png"), | ||
heading_text: "Get the GOV.UK app", | ||
description: "Your government services and information, on the go.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could add these to the locale files, but it might not be necessary now since we don’t know how long the promo will run.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I've added this suggestion to the image card component improvement ticket.
.gem-c-image-card__image-wrapper.gem-c-image-card__image-wrapper--one-third { | ||
flex-basis: 100px; | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noting that including text balance might be useful here, though it’s definitely not required.
.gem-c-image-card__description {
text-wrap: balance;
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I've made this change as suggested
Add a new section to the sign-in page to promote the govuk app, including link tracking. A new sign-in view stylesheet is included to style the new app promo section, this includes some overrides to the default style of the image card component.
4c5fe2e
to
f8c1474
Compare
What
Add a new section to the sign-in page to promote the govuk app, including link tracking.
A new sign-in view stylesheet is included to style the new app promo section, this includes some overrides to the default style of the image card component.
Why
Jira card: https://gov-uk.atlassian.net/browse/NAV-18518
Visual changes
Preview link: https://govuk-frontend-app-pr-5035.herokuapp.com/sign-in
Mobile
Desktop