Skip to content

Commit 26eb6c0

Browse files
refactor: move token table to reactor-design skill
Keep the nonexistent-token warning in getting-started as a critical gotcha, but move the full token table to the design skill where it belongs alongside the existing token reference. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent f685088 commit 26eb6c0

2 files changed

Lines changed: 4 additions & 24 deletions

File tree

plugins/reactor/skills/reactor-design/SKILL.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,9 @@ Hardcoded colors are acceptable only for:
5454

5555
#### Theme Token Reference
5656

57+
> ⚠️ **`Theme.Error`, `Theme.Success`, `Theme.Warning`, `Theme.ErrorText` do NOT exist.**
58+
> Use `Theme.SystemCritical` (red/error), `Theme.SystemSuccess` (green), `Theme.SystemCaution` (yellow).
59+
5760
**Text:**
5861

5962
| Token | WinUI Resource | Purpose |

plugins/reactor/skills/reactor-getting-started/SKILL.md

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -230,34 +230,11 @@ items.Select(i => Component<Card, CardProps>(new CardProps(i)).WithKey(i.Id)).To
230230

231231
## Theme tokens (always)
232232

233-
Use `Theme.*` for all themed colors — never hardcoded hex on themed surfaces.
233+
Use `Theme.*` for all themed colors — never hardcoded hex on themed surfaces. The full token list with WinUI keys is in the `reactor-design` skill.
234234

235235
> ⚠️ **`Theme.Error`, `Theme.Success`, `Theme.Warning`, `Theme.ErrorText` do NOT exist.**
236236
> Use `Theme.SystemCritical` (red/error), `Theme.SystemSuccess` (green), `Theme.SystemCaution` (yellow).
237237
238-
**Available tokens:**
239-
240-
| Category | Token | Use for |
241-
|---|---|---|
242-
| Accent | `Theme.Accent` | Primary action buttons, links |
243-
| Accent | `Theme.AccentSecondary`, `.AccentTertiary` | Hover/pressed states |
244-
| Text | `Theme.PrimaryText` | Body text |
245-
| Text | `Theme.SecondaryText`, `.TertiaryText` | Subtitles, captions |
246-
| Text | `Theme.AccentText` | Colored/linked text |
247-
| Surface | `Theme.SolidBackground` | Page/window background |
248-
| Surface | `Theme.CardBackground` | Card/panel fill |
249-
| Surface | `Theme.SubtleFill`, `.LayerFill` | Hover/surface layers |
250-
| Stroke | `Theme.CardStroke`, `.SurfaceStroke` | Card/panel borders |
251-
| Stroke | `Theme.DividerStroke` | Separators |
252-
| Signal | `Theme.SystemCritical` | Error/danger (red) |
253-
| Signal | `Theme.SystemSuccess` | Success (green) |
254-
| Signal | `Theme.SystemCaution` | Warning (yellow/orange) |
255-
| Signal | `Theme.SystemAttention` | Info/attention (blue) |
256-
| Signal | `Theme.SystemCriticalBackground` | Error background |
257-
| Signal | `Theme.SystemSuccessBackground` | Success background |
258-
259-
For any WinUI resource not listed: `Theme.Ref("YourResourceKeyBrush")`
260-
261238
```csharp
262239
TextBlock("Hi").Foreground(Theme.PrimaryText)
263240
Border(child).Background(Theme.CardBackground).WithBorder(Theme.CardStroke, 1)

0 commit comments

Comments
 (0)