Skip to content

Refine semantic callout icon colors - #177

Open
MaggieAppleton wants to merge 1 commit into
mainfrom
maggie/visual-foundations-09-callout
Open

MaggieAppleton wants to merge 1 commit into
mainfrom
maggie/visual-foundations-09-callout

Conversation

@MaggieAppleton

@MaggieAppleton MaggieAppleton commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • give each existing editor callout type a dedicated semantic icon color
  • apply the alias only to the trigger, Nucleo glyph, and picker leading icon
  • keep titles, layout, editor behavior, and MDX/Lexical contracts unchanged

Scope

Two CSS/test files only. No new visual primitive, editor logic, product adoption, auth, permissions, or composition changes.

Stack

Verification

  • focused tests: 18 pass
  • bun test: 1,506 pass, 2 PostgreSQL skips, 0 fail
  • bun run types
  • bun run ci
  • bun run build
  • git diff --check

Component preview

Five callout types with semantic icon colours

@MaggieAppleton
MaggieAppleton added this pull request to stack #175 September 24, 2026 14:07
@MaggieAppleton
MaggieAppleton force-pushed the maggie/visual-foundations-09-callout branch from 12471fc to ef0ec5e Compare September 24, 2026 14:12
@MaggieAppleton
MaggieAppleton force-pushed the maggie/visual-foundations-09-callout branch 2 times, most recently from 1ec85c1 to f98ebf7 Compare September 24, 2026 16:39
Base automatically changed from maggie/visual-foundations-08-badge to main September 24, 2026 16:52
@MaggieAppleton
MaggieAppleton force-pushed the maggie/visual-foundations-09-callout branch from f98ebf7 to a4a4fd3 Compare September 24, 2026 16:52
@coolify-githubnext-app

coolify-githubnext-app Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

The preview deployment for chopin is ready. 🟢

Open app | Open Build Logs | Open Application Logs

Last updated at: 2026-09-24 16:54:56 CET

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