Skip to content

feat!: remove xs size variant from form and action components - #701

Draft
FBalint wants to merge 1 commit into
cloudflare:mainfrom
FBalint:balint/remove-xs-components
Draft

feat!: remove xs size variant from form and action components#701
FBalint wants to merge 1 commit into
cloudflare:mainfrom
FBalint:balint/remove-xs-components

Conversation

@FBalint

@FBalint FBalint commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Fixes N/A — proactive API cleanup.

Removes xs from Button, Input, InputArea, InputGroup, Select, Combobox, Autocomplete, SensitiveInput, Toolbar. Types and runtime, both gone.

Breaking change, major bump.

Why
xs on interactive controls produced sub-24px hit targets. Rarely the right call, but easy to reach for.

Considered a soft-deprecation route (JSDoc + registry-driven lint rule for value-level deprecations, TS itself can't do this). Prototyped it, decided the "supported but discouraged" middle state and the extra codegen/lint machinery weren't worth it over just removing.


  • Reviews
    • bonk has reviewed the change
    • automated review not possible because: breaking API change + visual behaviour shift on Banner size="sm" — needs human eyes
  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
@pkg-pr-new

pkg-pr-new Bot commented Aug 5, 2026

Copy link
Copy Markdown
npm i https://pkg.pr.new/@cloudflare/kumo@701

commit: 7891cad

Removes the 'xs' size variant from Button, Input, InputArea, InputGroup,
Select, Combobox, Autocomplete, SensitiveInput, and Toolbar. Both the
type unions and the runtime KUMO_*_VARIANTS.size entries are gone.
Passing size='xs' will fail TypeScript compilation.

Text keeps xs (it is a real typographic step, 12px).

Banner: a size='sm' Banner used to render its Banner.Action children
at xs (h-5). It now renders them at sm (h-6.5). BannerActionSize is
narrowed to just 'sm'.

Migration: find/replace size='xs' -> size='sm'.
@FBalint
FBalint force-pushed the balint/remove-xs-components branch from dea2abe to 7891cad Compare August 5, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant