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

Extract the back_link component to govuk_publishing_components #131

Merged
merged 2 commits into from Jan 5, 2018

Conversation

tuzz
Copy link
Contributor

@tuzz tuzz commented Jan 5, 2018

https://trello.com/c/HSre7K10/453-style-the-mvp-email-collection-page-needed-by-jan

Add Back Link component

Moved from government-frontend for use in email-alert-frontend

Also renames the namespace to gem

Originally implemented in alphagov/government-frontend#568

@tuzz tuzz force-pushed the extract-back-link branch 2 times, most recently from 5a42889 to 72e1d0f Compare January 5, 2018 15:39
.gem-c-back-link {
position: relative;
display: inline-block;
margin-top: $app-spacing-scale-3;
Copy link
Contributor

Choose a reason for hiding this comment

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

using $app- here should break, have you tried rendering this locally?

@@ -170,22 +166,23 @@ def render_component(locals)
end

it "renders radio-group with welsh translated 'or'" do
I18n.default_locale = :cy
I18n.with_locale(:cy) do
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, this is great 🌮 👍

Moved from government-frontend for use in email-alert-frontend

Also renames the namespace to `gem`

Originally implemented in alphagov/government-frontend#568
@tuzz tuzz merged commit 0a59654 into master Jan 5, 2018
@tuzz tuzz deleted the extract-back-link branch January 5, 2018 16:47
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.

None yet

2 participants