Skip to content

Add accessible notifications and fix student updates - #13

Merged
theNEXlevel merged 2 commits into
mainfrom
feat/accessible-global-notifications
Oct 6, 2026
Merged

theNEXlevel merged 2 commits into
mainfrom
feat/accessible-global-notifications

Conversation

@theNEXlevel

Copy link
Copy Markdown
Contributor

Summary

  • Replace page-local MUI snackbars and transient browser alerts with a shared shadcn/Base UI toast provider, queued announcements, keyboard dismissal, and a five-second timeout.
  • Add and sync the accessible notification spec, archive the completed OpenSpec change, and include the before/after review screenshots.
  • Normalize enrollment and Yes/No enum values in student updates, and select only the student ID from Prisma updates to avoid deserializing legacy empty enum values.

Verification

  • openspec validate --specs passed (2 specs, 0 failures).
  • Chromium review of the notification provider covered live-region roles, keyboard dismissal, repeated and rapid notices, hover/focus timeout pauses, and the five-second timeout (observed about 5.3 seconds).
  • Spoken screen-reader output was not verified because no screen reader was available in the environment.

@theNEXlevel
theNEXlevel merged commit 719680d into main Oct 6, 2026
1 check passed
@theNEXlevel
theNEXlevel deleted the feat/accessible-global-notifications branch October 6, 2026 05:04
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.

1 participant