Skip to content

DIG-8380 Improve accessibility of error messages#374

Merged
maheshmuralip merged 6 commits into
mainfrom
DIG-8380
Jun 2, 2026
Merged

DIG-8380 Improve accessibility of error messages#374
maheshmuralip merged 6 commits into
mainfrom
DIG-8380

Conversation

@maheshmuralip
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR rewrites the shared livewire/alerts.blade.php partial to use NHS.UK notification-banner / error-summary components, links validation errors in the summary to their corresponding form-field error spans (improving keyboard / screen-reader navigation), adds the partial to the questions view, and records the change in the changelog.

Changes:

  • Replaces the old session/event-driven alert markup with NHS.UK notification-banner and error-summary patterns, including clickable anchors to {field}-error-error IDs.
  • Adds alerts.success.generic and alerts.warnings.title translations and includes the alerts partial in questions.blade.php.
  • Bumps the v1.0.16 changelog date and adds a bullet describing the accessibility improvement.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File Description
resources/views/livewire/alerts.blade.php Full rewrite of the alerts partial using NHS.UK banner / error-summary components and an @if/@elseif chain.
resources/views/livewire/questions.blade.php Includes the alerts partial at the top of the questions view.
lang/en/alerts.php Adds success.generic and warnings.title translation keys used by the new partial.
CHANGELOG.md Updates the v1.0.16 release date and lists the accessibility improvement.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread resources/views/livewire/alerts.blade.php Outdated
Comment thread resources/views/livewire/alerts.blade.php Outdated
Comment thread resources/views/livewire/alerts.blade.php Outdated
Comment thread resources/views/livewire/alerts.blade.php Outdated
Comment thread resources/views/livewire/alerts.blade.php Outdated
Comment thread resources/views/livewire/alerts.blade.php Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@maheshmuralip maheshmuralip marked this pull request as ready for review June 2, 2026 10:29
@maheshmuralip maheshmuralip requested a review from a team as a code owner June 2, 2026 10:29
@maheshmuralip maheshmuralip merged commit 3ccc32a into main Jun 2, 2026
1 check passed
@maheshmuralip maheshmuralip deleted the DIG-8380 branch June 2, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants