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

EYB accessibility #3174

Merged
merged 7 commits into from
Jul 1, 2024
Merged

EYB accessibility #3174

merged 7 commits into from
Jul 1, 2024

Conversation

timothyPatterson
Copy link
Contributor

@timothyPatterson timothyPatterson commented Jun 26, 2024

This PR addresses two findings from an accessibility audit: 1) align legends and labels with GOVUK form guidance for pages with single questions, 2) add focus to autocomplete input element when the user presses ESC on the dropdown list. This is a known issue with the autocomplete library we are using - alphagov/accessible-autocomplete#692

Workflow

Reviewing help

  • Explains how to test locally, including how to set up appropriate data

Merging

  • This PR can be merged by reviewers. (If unticked, please leave for the author to merge)

@codecov-commenter
Copy link

codecov-commenter commented Jun 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.33%. Comparing base (988b5c8) to head (9aa764f).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #3174   +/-   ##
========================================
  Coverage    95.33%   95.33%           
========================================
  Files          209      209           
  Lines        13573    13573           
========================================
  Hits         12940    12940           
  Misses         633      633           
Files Coverage Δ
international_online_offer/forms.py 98.05% <ø> (ø)

Refactor expand your business js import into base.html
Modify triage header to remove question title and refactor question title into each form.
Ensure labels/legends are contained with form / fieldset elements.
Override govuk-fieldset__heading to align with International fonts.
Conditional display of field.label in form_field.html to remove redundant markup where label is an empty string.
@timothyPatterson timothyPatterson marked this pull request as ready for review July 1, 2024 14:35
@timothyPatterson timothyPatterson requested a review from a team as a code owner July 1, 2024 14:35
@timothyPatterson timothyPatterson merged commit b70dd50 into develop Jul 1, 2024
7 checks passed
@timothyPatterson timothyPatterson deleted the EYB-40-43-accessibility branch July 1, 2024 14:57
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.

3 participants