Skip to content

ref(core): consolidate alertBadge files and remove chonk naming #105579

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
JonasBa wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ref(core): consolidate alertBadge files and remove chonk naming #105579

JonasBa wants to merge 1 commit into from

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Dec 31, 2025

Summary

Consolidates alertBadge.chonk.tsx into alertBadge.tsx and removes chonk naming prefixes.

Changes

  • Moved all content from alertBadge.chonk.tsx to alertBadge.tsx
  • Renamed function:
    • makeChonkAlertBadgeDiamondBackgroundTheme → makeAlertBadgeDiamondBackgroundTheme
  • Renamed styled component:
    • ChonkAlertBadgeDiamondBackground → DiamondBackground (defined inline)
  • Made AlertBadgeProps interface internal-only (removed export)
  • Deleted alertBadge.chonk.tsx

Test plan

  • ✅ Linting passes
  • ✅ Knip passes (no unused exports)
  • ✅ No remaining chonk references (except theme.colors.chonk)

@JonasBa JonasBa requested a review from a team as a code owner December 31, 2025 18:25
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JonasBa