Skip to content

fix(color): correct primitive grey/950 to #1F1F1F (UI Kit 5)#26

Merged
Gr1dlock merged 1 commit into
mainfrom
fix/primitive-grey-950
Jun 3, 2026
Merged

fix(color): correct primitive grey/950 to #1F1F1F (UI Kit 5)#26
Gr1dlock merged 1 commit into
mainfrom
fix/primitive-grey-950

Conversation

@Gr1dlock
Copy link
Copy Markdown
Contributor

@Gr1dlock Gr1dlock commented Jun 2, 2026

What

Primitive grey.950 was #181818, but UI Kit 5.0 (the source of truth) defines it as #1F1F1F. semantic.color.text.primary, action.primary, icon.primary, etc. all resolve to grey.950, so the most common text/foreground color rendered a shade too dark on every platform.

Changes

  • tokens/definitions/color/primitive.json: grey.950 #181818#1F1F1F
  • Regenerated committed Android/iOS color outputs (NucleusPrimitiveColors.kt, NucleusColor+Primitives.swift, NucleusColor+Semantics.swift). Web outputs are produced at publish time.

Notes

  • Verified against the UI Kit 5 Color Primitives page (grey/950 = #1F1F1F; grey/700 = #7D7D7D etc. already matched).
  • Hand-written component literals (e.g. NucleusCardTheme.kt) still hardcode 0x181818 and are intentionally left out of this token-only fix — they can be reconciled separately.
  • npm run build, typecheck, lint, and format:check all pass.

Found while adopting Nucleus tokens in app-backend SDUI V4.

🤖 Generated with Claude Code

Primitive `grey.950` was `#181818`, but UI Kit 5.0 (the source of truth)
defines it as `#1F1F1F`. Since `semantic.color.text.primary`,
`action.primary`, and friends resolve to `grey.950`, this made the most
common text/foreground color render a shade too dark on every platform.

Updates the definition and regenerates the committed Android/iOS color
outputs. Web outputs are produced at publish time.

Note: hand-written component literals (e.g. NucleusCardTheme.kt) still
hardcode 0x181818 and are intentionally left out of this token-only fix;
they can be reconciled separately.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@jaidensiu jaidensiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@Gr1dlock Gr1dlock merged commit 778754e into main Jun 3, 2026
9 checks passed
@Gr1dlock Gr1dlock deleted the fix/primitive-grey-950 branch June 3, 2026 10:27
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.

3 participants