Skip to content

Fixes medium and low accessibility issues - #70

Merged
FrostyApeOne merged 3 commits into
mainfrom
fix/308664-accessibility-audit-medium-low
Oct 23, 2026
Merged

FrostyApeOne merged 3 commits into
mainfrom
fix/308664-accessibility-audit-medium-low

Conversation

@jack-nimbleapproach

@jack-nimbleapproach jack-nimbleapproach commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Fixes from accessibility audit:

  • Removed unnecessary headings and corrected existing heading hierarchy
  • Removed broken aria reference for name field on invite contributor page
  • Moved legend into fieldset on confirmation page
  • Added original filename to make remove and delete buttons unique
  • Added message to dismiss buttons on notifications page to associate button with message

@jack-nimbleapproach
jack-nimbleapproach marked this pull request as ready for review October 23, 2026 12:50
<h1 class="govuk-heading-xl govuk-!-margin-bottom-9">
@Model.MainHeading
@Model.MainHeading test
</h1>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

what is this 'test'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@FrostyApeOne
FrostyApeOne merged commit 9632791 into main Oct 23, 2026
1 check passed
@FrostyApeOne
FrostyApeOne deleted the fix/308664-accessibility-audit-medium-low branch October 23, 2026 13:20
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.

2 participants