Skip to content

refactor(notifications): unify notification UX into sidebar and add rich feed interactions#252

Draft
justjxke wants to merge 3 commits intobetter-auth:mainfrom
justjxke:refactor/notification-panel
Draft

refactor(notifications): unify notification UX into sidebar and add rich feed interactions#252
justjxke wants to merge 3 commits intobetter-auth:mainfrom
justjxke:refactor/notification-panel

Conversation

@justjxke
Copy link
Contributor

@justjxke justjxke commented Mar 2, 2026

Why

  • Replace the old /notifications page flow with a single right-sidebar notifications experience.
  • Improve clarity and actionability of notifications (status context, per-item actions, filtering, bulk actions).
  • Iterate quickly on UI with dummy seeded notifications for local validation.

How

  • Removed page-based notifications surface and consolidated behavior in the sidebar feed.
  • Added richer notification model + UI:
    • status chips, contextual secondary text, grouped sections, tab filters
    • select mode, bulk actions, mute/unmute flow, undo for mark-as-read
    • improved hover/selection/action affordances and header controls
  • Added extensive dummy notification data for realistic local stress-testing and visual QA.
  • Added safeguards for non-numeric demo IDs in mark-read flow to avoid GitHub API 404s.

Tests

  • pnpm fmt:check
  • pnpm typecheck
  • pnpm lint (existing repo warnings unchanged, no new errors)

Draft note

This PR is intentionally Draft because it still contains demo/dummy notification data used for local development and UX validation.
Before merge, we should remove or fully gate dummy notification paths for production.

Notes

I'd heavily appreciate design feedback, what you want removed, added, changed ect. This took quite a while, most of it brainstorming on the design and interaction flow. There's definitely room for improvement.

Screenshots

image CleanShot 2026-03-02 at 11 06 13@2x CleanShot 2026-03-02 at 11 06 43@2x CleanShot 2026-03-02 at 11 07 08@2x

@vercel
Copy link

vercel bot commented Mar 2, 2026

@justjxke is attempting to deploy a commit to the better-auth Team on Vercel.

A member of the Team first needs to authorize it.

@justjxke justjxke force-pushed the refactor/notification-panel branch from c7efe67 to ea40dbd Compare March 2, 2026 19:18
@justjxke
Copy link
Contributor Author

hmmm i should work on this again and improve the UI but idk if it's worth it.

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