Skip to content

fix(ui): 新任务 compose icon in both sidebar states + warning tone splits from info - #697

Merged
jackwener merged 1 commit into
mainfrom
fix/newtask-icon-warning-tone
Jul 9, 2026
Merged

fix(ui): 新任务 compose icon in both sidebar states + warning tone splits from info#697
jackwener merged 1 commit into
mainfrom
fix/newtask-icon-warning-tone

Conversation

@jackwener

Copy link
Copy Markdown
Member

Two fixes:

  1. 新任务 icon parity (maintainer instruction): expanded sidebar row showed Plus (feat(ui): icon semantic remap + stroke unification + governance contract #662) while the collapsed topbar shows SquarePen — same action, two glyphs. Unified on SquarePen (the collapsed one). icon-governance contract re-pinned; CDP probe confirms lucide-square-pen on the row.
  2. 警告 ≠ 提示: --warning was literally var(--info) — the health page's info and warning tiers rendered the identical amber. Warning now steps deeper and more orange (oklch .66/.18/55 vs info's .75/.16/70) with its own --warning-text mix. Health capture confirms the tiers are finally distinguishable; all palettes inherit since none overrode --warning.

Desktop 2296/2296 exit-code gated.

…one splits from info

- Maintainer: the expanded sidebar row (Plus since #662) and the
  collapsed topbar compose button (SquarePen) showed different icons
  for the same action — unified on SquarePen, the collapsed one, per
  instruction. icon-governance contract re-pinned.
- Found while sweeping 健康: --warning was literally var(--info) — the
  提示 and 警告 tiers rendered the same amber and could not be told
  apart. Warning now steps deeper + more orange (l .75→.66, h 70→55)
  with its own --warning-text mix; info keeps its amber. CDP capture
  confirms the two tiers finally read differently.

Desktop 2296/2296.
@jackwener
jackwener merged commit bb40619 into main Jul 9, 2026
2 of 3 checks passed
@jackwener
jackwener deleted the fix/newtask-icon-warning-tone branch July 9, 2026 16:37
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