Skip to content

Restyle toasts as a solid dark pill, top-center#8

Merged
CaptainCo0L merged 1 commit into
mainfrom
restyle-toasts
Jun 29, 2026
Merged

Restyle toasts as a solid dark pill, top-center#8
CaptainCo0L merged 1 commit into
mainfrom
restyle-toasts

Conversation

@CaptainCo0L

Copy link
Copy Markdown
Owner

Problem

The toast notifications looked like a Windows app message — a light bordered card with a tinted background, icon on the left, pinned bottom-right.

Change

Restyle to a solid dark pill, top-center:

  • frontend/src/components/Toast.jsx — near-black fill (bg-ink), light text (text-paper), rounded-full, anchored top-center; dropped the per-type tint/border STYLE map; recolored status icons for the dark surface (sage tick for success, red-400 ✕ for error).
  • frontend/tailwind.config.js — flipped the toast-in keyframe to drop in from above (translateY(-8px)) to suit the top anchor.

Behavior is unchanged (success/error/info types, TTLs, stacking, role="status", portal).

Verification

Rebuilt the Docker image and triggered toasts via the UI (Playwright). Both render as dark top-center pills with legible contrast — green tick for "Batch deleted." / "Student removed.", red ✕ for "Can't delete…". Delete flows unregressed (3/3).

🤖 Generated with Claude Code

The light bordered card pinned bottom-right read like a Windows notification.
Swap it for a solid near-black pill (bg-ink) with light text, rounded-full,
anchored top-center; drop the per-type tint/border map and recolor the status
icons for the dark surface (sage tick / red-400 cross). Flip the toast-in
keyframe to drop in from above to suit the top anchor.

Behavior unchanged (types, TTLs, stacking, role=status, portal).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@CaptainCo0L CaptainCo0L merged commit bafb7a8 into main Jun 29, 2026
1 check passed
@CaptainCo0L CaptainCo0L deleted the restyle-toasts branch June 29, 2026 08:09
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