Skip to content

chore(theme): drop the unused --text-display token #103

@yigitdot

Description

@yigitdot

Background

app/globals.css's @theme inline block defines --text-display: clamp(3.5rem, 14vw, 12.5rem) but nothing references it — grep for text-display / --text-display across app/, components/, lib/, content/ finds only the declaration. It's been dead since before #101 (it was --fs-display then; #101 just renamed it along with the rest of the scale for namespace coherence — see the review note). Tailwind v4 won't even emit a text-display utility because it's never used in a class.

Options

  1. Delete it — one-line removal from the @theme inline block. Simplest; the scale loses one rung but no behaviour changes.
  2. Wire it up — if there's an intended "display" headline size larger than --text-h1 that we want on the books, find/define the call site and use text-display. (No such call site exists today, and chore(theme): migrate footer to Tailwind theme utilities #76/feat(blog): redesign the index and post pages as a "field-log" table #99/fix(blog): capitalize post headings, center body, tighten heading rhythm #100's redesigns didn't introduce one, so this seems unlikely.)

Recommendation: option 1 unless someone has a near-term use for a display-size token.

Links

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions