Refactor/accessibility improvements

[GeorgeGoodall]

What's New

• Added the role attribute of complimentary to the phase banner
• Added the role attribute of navigation to the skip to main content link

Why were these changes to make

• so that these components don't fail the Axe, automated accessibility testing

[matthew-shaw]

Hi @GeorgeGoodall thanks very much for the PR. However, I don't think this is something that should be done at this level. If there are accessibility issues with the generated HTML output from the Jinja templates, then they also exist with the original Nunjucks templates that these were ported from. GOV.UK Frontend Jinja is a like-for-like port of the original GOV.UK Frontend, and tested against it to ensure the markup is 100% identical, any changes made here would break that parity. So this needs to be done at source, which will end up in this package as well after release.

I would check the issues against GOV.UK Frontend and see if the problem has already been raised: https://github.com/alphagov/govuk-frontend/issues Or ask the team on the X-Gov Slack channel govuk-design-system for advice.

Thanks again