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

Update footer to take licence text and copyright text #1314

Closed

Conversation

georgeeker
Copy link

No description provided.

@georgeeker georgeeker changed the title Update footer to take licence text and copyright text WIP:Update footer to take licence text and copyright text May 2, 2019
@georgeeker georgeeker changed the title WIP:Update footer to take licence text and copyright text Update footer to take licence text and copyright text May 2, 2019
@georgeeker
Copy link
Author

See:
#1291
#1290

- name: with custom licence text
description: Custom licence section
data:
licence: GOVUK Licence
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @georgeeker, this will need to have a .text or .html property to work.

See the preview: https://govuk-frontend-review-pr-1314.herokuapp.com/components/footer/with-custom-licence-text/preview

It's not triggering {% if params.licence.html or params.licence.text %}

- name: with custom copyright text
description: Custom copyright section
data:
logoText: Copyright
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be added to Nunjucks template too?

@36degrees
Copy link
Contributor

36degrees commented May 17, 2019

Thanks, @georgeeker!

There's a couple of things that still need thinking about here.

The OGL logo is included as an SVG as part of the footer. With the change as it currently stands, even if you override the copyright text you still get the OGL logo, because it's outside of the {% if %} block:

localhost_3000_components_footer_with-custom-licence-text_preview

We could move the {% if %} block to wrap the SVG as well, but:

  • this would mean that users overriding the HTML would need to know to wrap their licence in govuk-footer__licence-description
  • I think there may be instances where users want to overwrite the copyright text whilst still preserving the OGL logo

Secondly, removing the crest makes the footer a lot smaller, so I think we probably need some design input as to whether that's an issue or not.

localhost_3000_components_footer_with-custom-copyright-text_preview

There might also again be cases where a user wants to override the copyright text without removing the crest, although it's less of an issue as they only have to include a specific class name on the link, rather than having to redeclare an entire SVG.

@kellylee-gds kellylee-gds moved this from Needs review to In progress in Design System Sprint Board May 21, 2019
@36degrees
Copy link
Contributor

Hi George,

Thanks for your work on this so far. I just wanted to check whether you were interested in carrying on working on this?

If we don't hear from you in the next 7 days then we'll close this PR (keeping things tidy helps us keep track of what's being worked on) but please feel free to re-open if you do want to pick this up again.

Addressing this is something we plan to do, but we've not got capacity to pick up ourselves right now.

Thanks,

Ollie

@36degrees
Copy link
Contributor

As mentioned in the last comment we're going to close this for now, but please feel free to re-open if you want to pick this up again.

Thanks again for your contribution.

@36degrees 36degrees closed this Jun 4, 2019
Design System Sprint Board automation moved this from In progress to Done Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

6 participants