Skip to content

Conversation

@natemoo-re
Copy link
Member

@natemoo-re natemoo-re commented Jan 7, 2026

Apologies for the notification! Removing theme.subText touches a lot of files but is an otherwise small change. Closes DE-329.

In simple cases with the color property, these have been replaced with theme.tokens.content.secondary.

I flagged a handful of non-text CSS Properties, which have been converted to the semantically appropriate alternative—these are worth taking a second look at commit-by-commit.

File Note
alertBadge.tsx fixed in #105958
plainTextQueryInput.tsx caret-color, use interactive token
slider/index.tsx disabled, use content.disabled
slider/thumb.tsx disabled, use content.disabled
metricChart.tsx background, use graphics.neutral.vibrant
ruleForm.tsx background, use graphics.neutral.vibrant
baseAvatar.tsx border, use border.neutral.vibrant

@linear
Copy link

linear bot commented Jan 7, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 7, 2026
natemoo-re added a commit that referenced this pull request Jan 8, 2026
Came across this in the course of #105851 and decided to refactor
`AlertBadge` to align the logic to the new token structure and simplify
the functions.
@natemoo-re natemoo-re marked this pull request as ready for review January 9, 2026 21:00
@natemoo-re natemoo-re requested review from a team as code owners January 9, 2026 21:00
@natemoo-re natemoo-re merged commit dd203aa into master Jan 9, 2026
54 checks passed
@natemoo-re natemoo-re deleted the theme/nm/subtext branch January 9, 2026 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants