Skip to content

chore: Color Tokens for Accordion, Card, Table, Tabs, and Typography. part 4 - BED-10247 - #3589

Draft
KillahDillah wants to merge 8 commits into
mainfrom
BED-10247-color-tokens-4
Draft

KillahDillah wants to merge 8 commits into
mainfrom
BED-10247-color-tokens-4

Conversation

@KillahDillah

@KillahDillah KillahDillah commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Description

Continues the Doodle UI color token migration by moving component styles into dedicated files and centralizing theme-dependent values.

  • Adds shared light and dark theme tokens for Accordion, Card, Table, Tabs, and Typography.
  • Extracts styles for Accordion, Alert, Card, IconButton, Label, Link, Table, Tabs, and Typography.
  • Standardizes class composition with cn and shares invalid-state styles across form controls.
  • Updates style assertions and adds the missing findings-types API mock to the BHE filter tests.

Motivation and Context

Resolves BED-10247

Why is this change required? What problem does it solve?

How Has This Been Tested?

Please describe in detail how you tested your changes.
Include details of your testing environment, and the tests you ran to
see how your change affects other areas of the code, etc.

Screenshots (optional):

Types of changes

  • Chore (a change that does not modify the application functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Database Migrations

Checklist:

@KillahDillah KillahDillah self-assigned this Oct 26, 2026
@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@KillahDillah
KillahDillah force-pushed the BED-10247-color-tokens-4 branch from 5a36613 to 68b9e1e Compare October 27, 2026 18:50
@KillahDillah KillahDillah changed the title chore: use cn where appropriate, carry over from previous PR chore: Color Tokens, part 4 - BED-10247 Oct 27, 2026
@KillahDillah
KillahDillah force-pushed the BED-10247-color-tokens-4 branch from 14203bf to 36b6abb Compare October 27, 2026 19:56
@KillahDillah KillahDillah changed the title chore: Color Tokens, part 4 - BED-10247 chore: Color Tokens for Accordion, Card, Table, Tabs, and Typography. part 4 - BED-10247 Oct 28, 2026

This branch has not been deployed

No deployments
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.

1 participant