-
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
@import "govuk_publishing_components/individual_component_support"; | ||
|
||
.app-promo { | ||
padding-top: govuk-spacing(6); | ||
border-top: 4px solid $govuk-brand-colour; | ||
margin-top: govuk-spacing(8); | ||
|
||
@include govuk-media-query($from: "tablet") { | ||
margin-top: govuk-spacing(9); | ||
} | ||
|
||
.gem-c-image-card { | ||
display: flex; | ||
margin-bottom: 0; | ||
} | ||
|
||
.gem-c-image-card__description { | ||
text-wrap: balance; | ||
} | ||
|
||
.gem-c-image-card__image-wrapper.gem-c-image-card__image-wrapper--one-third { | ||
flex-basis: 100px; | ||
} | ||
} | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<% | ||
add_view_stylesheet("sign-in") | ||
# These components aren't found in the templates, but the styles are still | ||
# needed as they're used within the sign in page's `content_item`. | ||
add_gem_component_stylesheet("document-list") | ||
|
@@ -71,5 +72,23 @@ | |
text: t("help.sign_in.search_for_a_service_button"), | ||
secondary_solid: true %> | ||
</form> | ||
|
||
<div class="govuk-!-width-full app-promo"> | ||
<%= render "govuk_publishing_components/components/image_card", { | ||
two_thirds: true, | ||
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 commentThe 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 commentThe 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. |
||
data_attributes: { | ||
module: "ga4-link-tracker", | ||
ga4_link: { | ||
event_name: "navigation", | ||
type: "image card", | ||
}, | ||
}, | ||
} %> | ||
</div> | ||
|
||
</div> | ||
</div> |
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.
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