Skip to content

Adds notification center to the header - #2678

Draft
Mbeaulne wants to merge 1 commit into
08-28-adds_markdown_support_for_bannersfrom
08-28-adds_notification_center_to_the_header
Draft

Adds notification center to the header#2678
Mbeaulne wants to merge 1 commit into
08-28-adds_markdown_support_for_bannersfrom
08-28-adds_notification_center_to_the_header

Conversation

@Mbeaulne

@Mbeaulne Mbeaulne commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Description

Adds a read-only Notices popover to the top-right of both application headers. It shows the active notice count and lists the same active announcements used by the homepage banners, preserving host order and rendering their Markdown content.

This intentionally does not add read/unread tracking, separate dismissal state, polling, or notification subscriptions.

Related Pull Requests

Type of Change

  • New feature

Test Instructions

image.png

image.png

  1. Supply several announcements through window.__TANGLE_ANNOUNCEMENTS__.
  2. Confirm the notice count appears in both header versions.
  3. Open the popover and confirm active notices appear in host order with Markdown links.
  4. Confirm expired notices are omitted and an empty source shows No active notices.
  5. Confirm dismissing a homepage banner does not remove it from the read-only notice list.

Automated coverage:

pnpm exec vitest run src/components/shared/NoticeCenter.test.tsx
pnpm run typecheck

@github-actions

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: 08-28-adds_notification_center_to_the_header/dbb7dc1

Mbeaulne commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

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