Skip to content

DPC-5182 Add headings, error messages for login.gov signin fail states#2923

Merged
jhutchison-nava merged 8 commits intomainfrom
jh/DPC-5182
Mar 6, 2026
Merged

DPC-5182 Add headings, error messages for login.gov signin fail states#2923
jhutchison-nava merged 8 commits intomainfrom
jh/DPC-5182

Conversation

@jhutchison-nava
Copy link
Contributor

@jhutchison-nava jhutchison-nava commented Feb 24, 2026

🎫 Ticket

https://jira.cms.gov/browse/DPC-5182

🛠 Changes

  • Adds 2 new page templates for login.gov error response pages:

Login.gov flow cancel
Screenshot 2026-02-24 at 12 45 14 PM

Login.gov flow fail
Screenshot 2026-02-24 at 12 45 21 PM

ℹ️ Context

These pages were missing h1's, causing an accessibility error

🧪 Validation

Manually verified during development with Lookbook

A11y check

  • Resolves Accessibility login shows login failure a11y error

@jhutchison-nava jhutchison-nava requested a review from a team as a code owner February 24, 2026 17:48
Copy link
Contributor

@Jose-verdance Jose-verdance left a comment

Choose a reason for hiding this comment

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

Just a minor change, but looks good otherwise.

MEspositoE14s
MEspositoE14s previously approved these changes Mar 6, 2026
Copy link
Contributor

@MEspositoE14s MEspositoE14s left a comment

Choose a reason for hiding this comment

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

Outside of @Jose-verdance's comment, 👍

MEspositoE14s
MEspositoE14s previously approved these changes Mar 6, 2026
MEspositoE14s
MEspositoE14s previously approved these changes Mar 6, 2026
Jose-verdance
Jose-verdance previously approved these changes Mar 6, 2026
Copy link
Contributor

@Jose-verdance Jose-verdance left a comment

Choose a reason for hiding this comment

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

LGTM!

Co-authored-by: jdettmannnava <145699825+jdettmannnava@users.noreply.github.com>
Copy link
Contributor

@jdettmannnava jdettmannnava left a comment

Choose a reason for hiding this comment

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

LGTM

@jhutchison-nava jhutchison-nava merged commit 1c478c6 into main Mar 6, 2026
11 checks passed
@jhutchison-nava jhutchison-nava deleted the jh/DPC-5182 branch March 6, 2026 18:17
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.

4 participants