feat: color lines by special tag - #182
Conversation
vycdev
left a comment
There was a problem hiding this comment.
The new special-tag palette needs to follow the palette fallback semantics already on develop.
|
|
||
| SPECIAL_TAG_LINES = []; | ||
| Consts.tags.names.forEach((name, index) => { | ||
| const backgroundColor = Consts.colors.tags.lineBackground[index]; |
There was a problem hiding this comment.
Current develop makes shorter special-tag palettes repeat across every configured tag. Indexing lineBackground directly means ['red'] colors only the first tag while the existing background/foreground palettes color all tags, and the dark/light arrays have the same issue below. Please use the shared palette helper for base, dark, and light line backgrounds and add a short-palette regression.
4db757d to
3e3583d
Compare
|
Addressed the short-palette feedback and rebased onto current Verification: This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying. |
Summary
todo.colors.tags.lineBackgroundVerification
npm test— passed: 231 testsnpm run compile— passed: webpack development build completednpm run format:check— passed: all matched files use Prettier formattinggit diff --check upstream/develop...HEAD— passedRisk
Closes #93
This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.