-
Notifications
You must be signed in to change notification settings - Fork 19
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
Move custom meta content above licence in footer #511
Conversation
It appears that the custom meta content should be above the licence rather than underneath it. The examples in the design system samples show it[0] but it's not really covered by the official documentation[1] (which was what the library was based on in the early days) [0] https://govuk-frontend-review.herokuapp.com/components/footer [1] https://design-system.service.gov.uk/components/footer/
✅ Deploy Preview for govuk-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
a7e2747
to
ee6e044
Compare
guide/lib/examples/footer_helpers.rb
Outdated
| Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nullam | ||
a porta purus. Fusce faucibus aliquam massa sed eleifend. | ||
- footer.with_meta_html do | ||
| Built with love by x-govuk. |
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.
Might be useful to show including a link in this example (which would need the govuk-footer__link
applied); not sure what the correct way of doing that would be, off the top of my heading, but I guess using the govuk_link
helper.
| Built with love by x-govuk. | |
| Built with love by X-GOVUK. |
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.
Good point. I think there's room for a #govuk_footer_link_to
helper as the ones in the example don't have the govuk-link
class.
Good spot! Think we can improve the example a little, but nothing more to add. |
Co-authored-by: paulrobertlloyd <me+git@paulrobertlloyd.com>
eb9414e
to
b975d8e
Compare
Ooh, |
It appears that the custom meta content should be above the licence rather than underneath it.
The examples in the design system samples show it but it's not really covered by the official documentation (which was what the library was based on in the early days).