Skip to content

feat(chip): update samples - #1118

Merged
ChronosSF merged 10 commits into
vnextfrom
iminchev/update-chip-samples
Oct 23, 2026
Merged

ChronosSF merged 10 commits into
vnextfrom
iminchev/update-chip-samples

Conversation

@IMinchev64

Copy link
Copy Markdown
Contributor

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Hristo313 Hristo313 added status: verified The PR is tested and ready for a merge and removed awaiting-test labels Oct 19, 2026
IMinchev64 and others added 3 commits October 22, 2026 13:47
Bring in the Astro migration (#1110), the sample type check (#1132)
and AGENTS.md (#1133).

Conflicts: chip outlined, overview, size and styling. This branch had
already rewritten them as function components with new content, so its
side is kept, in vnext's style: no JSX.Element return type, which
React 19's types no longer declare, no unused React import, and the
`getElementById('root')!` mount.

The seven new chip samples (area, interaction-state, layout, remove,
state, tailwind-styling, types) get what vnext did to every sample:
no sandbox.config.json, README.md (tailwind-styling had ReadMe.md) in
the current format without the CodeSandbox link, the strict add:sample
tsconfig, and package versions from scripts/lib/versions.js.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The tailwind-styling sample rendered unstyled in the Astro build. Its
stylesheet put the Aktiv Grotesk @import before the @layer statement,
and the samples browser compiles sample CSS through Vite, whose @import
handling then drops the Tailwind imports that follow: the page got 151
bytes of CSS instead of the theme and utilities. Standalone, Tailwind's
own Vite plugin handles the imports, which hid this.

Put the @layer statement first and the font import right after it.
The page now inlines the palette and all 24 utilities the sample uses.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ChronosSF
ChronosSF merged commit 5651441 into vnext Oct 23, 2026
4 checks passed
@ChronosSF
ChronosSF deleted the iminchev/update-chip-samples branch October 23, 2026 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: verified The PR is tested and ready for a merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants