Skip to content
This repository is currently being migrated. It's locked while the migration is in progress.

Alert: Document dismissible alert persistence#6164

Open
babsdenney wants to merge 4 commits into
mainfrom
5709-dismissable-alert
Open

Alert: Document dismissible alert persistence#6164
babsdenney wants to merge 4 commits into
mainfrom
5709-dismissable-alert

Conversation

@babsdenney
Copy link
Copy Markdown
Contributor

@babsdenney babsdenney commented Apr 21, 2026

Summary

Clarify in the alert component docs that dismissible alerts do not persist their dismissed state by default. Added notes to both the main alert and slim alert sections advising that applications must implement state persistence if alerts should remain dismissed across browser/app sessions.

Related Issue

Closes #5709

Preview Environment Links

Open Preview Environment

Clarify in the alert component docs that dismissible alerts do not persist their dismissed state by default. Added notes to both the main alert and slim alert sections advising that applications must implement state persistence if alerts should remain dismissed across browser/app sessions.
@babsdenney babsdenney self-assigned this Apr 21, 2026
Copilot AI review requested due to automatic review settings April 21, 2026 22:27
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 updates the Alert component documentation to clarify that dismissible alerts do not persist their dismissed state across browser/app sessions by default, and that consumers must implement their own persistence when needed.

Changes:

  • Added a documentation note in the web “Dismissible” alert example clarifying non-persistence across sessions.
  • Added a parallel documentation note in the mobile app “Dismissible” alert example clarifying non-persistence across sessions.

@babsdenney babsdenney marked this pull request as ready for review April 21, 2026 22:33
@babsdenney babsdenney requested a review from a team as a code owner April 21, 2026 22:33
Copy link
Copy Markdown
Contributor

@jeana-adhoc jeana-adhoc left a comment

Choose a reason for hiding this comment

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

LGTM! Today I learned that we can have dismissable slim alerts too!

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.

Dismissible alert does not stay dismissed in future sessions

3 participants