Skip to content

refactor(frontend): entity history on tailwind-variants - #3997

Merged
Kadrian merged 1 commit into
refactor/tailwind-forms-previewfrom
refactor/tailwind-entity-history
Sep 2, 2026
Merged

refactor(frontend): entity history on tailwind-variants#3997
Kadrian merged 1 commit into
refactor/tailwind-forms-previewfrom
refactor/tailwind-entity-history

Conversation

@Kadrian

@Kadrian Kadrian commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Emotion → tailwind migration, PR 5/6: entity history (29 files) — after this, no component uses emotion.

(entity-history + timeline — stacked on #3996, stack overview in #3992. Only the theme plumbing is left for #3998)

Main change

  • every styled() block → tv(); data-driven values (badge colours, grid templates, number widths) stay inline style
  • charts (TimeStratifiedChart) resolve --color-primary-500 at runtime via getCssVarColor — downstream :root overrides keep affecting them

Worth a closer look

  • three tv-const/prop shadowing crashes caught (cell — would have called the data value, value, message); renamed with comments
  • EventCard: media queries → min-[1800px]:/min-[2500px]: variants; its icon's width: 24px !importantw-6! (beats FaIcon's w-[initial]! via merge, like the old important)
  • EntityIdsList: active/hover logic → variants; faithful (active rows never changed on hover)
  • dead code removed, behaviour-identical: TheEntityId's unused active prop, Avatar's never-used blur, SpecialText's never-passed zero
  • ConceptBubble.ts/TinyLabel.ts.tsx components

Checks: biome, tsc, vitest, vite build green; generated css spot-checked (min-[1800px]:, w-6!, blur-[6px], auto-rows).

eighth and last styled() slice; charts resolve css vars at runtime
@Kadrian
Kadrian force-pushed the refactor/tailwind-entity-history branch from 9ec007f to bff2e61 Compare September 2, 2026 13:27
@Kadrian
Kadrian merged commit 2412e61 into develop Sep 2, 2026
13 checks passed
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.

2 participants