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

Delete accordion issue in accessibility statement #1825

Merged
merged 1 commit into from
Dec 16, 2021

Conversation

EoinShaughnessy
Copy link
Contributor

@EoinShaughnessy EoinShaughnessy commented Aug 17, 2021

Fixes #1823.

What we've changed

This PR removes mention of an accordion-affecting WCAG issue from the non-compliance section of our accessibility statement.

We've also removed some redundant text to avoid a clumsy repetition.

Why we've changed it

The iterated accordion component will fix the accessibility issue.

@EoinShaughnessy EoinShaughnessy added accessibility documentation User requests new documentation or improvements to existing documentation accordion labels Aug 17, 2021
@EoinShaughnessy EoinShaughnessy self-assigned this Aug 17, 2021
@EoinShaughnessy
Copy link
Contributor Author

We should not merge this PR until we're releasing the iterated component. See #1706 to keep track of iteration progress.

@netlify
Copy link

netlify bot commented Aug 17, 2021

✔️ You can preview this change here:

🔨 Explore the source changes: e41e6ea

🔍 Inspect the deploy log: https://app.netlify.com/sites/govuk-design-system-preview/deploys/61bb07d2a2ee9400071af160

😎 Browse the preview: https://deploy-preview-1825--govuk-design-system-preview.netlify.app

@36degrees 36degrees added this to the Frontend v4.0.0 milestone Aug 19, 2021
src/accessibility.md.njk Outdated Show resolved Hide resolved
@vanitabarrett vanitabarrett removed this from the Frontend v4.0.0 milestone Dec 1, 2021
@vanitabarrett vanitabarrett force-pushed the update-acc-stmt-accordion branch from aa12c57 to 0ec5d8d Compare December 3, 2021 15:30
@vanitabarrett vanitabarrett force-pushed the update-acc-stmt-accordion branch from 0ec5d8d to e41e6ea Compare December 16, 2021 09:33
@vanitabarrett vanitabarrett merged commit bd043e0 into main Dec 16, 2021
@vanitabarrett vanitabarrett deleted the update-acc-stmt-accordion branch December 16, 2021 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility accordion documentation User requests new documentation or improvements to existing documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update accessibility statement to reflect fix in iterated accordion component
4 participants