Skip to content

feat(radio): update samples - #1120

Merged
ChronosSF merged 5 commits into
vnextfrom
iminchev/update-radio-samples
Oct 29, 2026
Merged

ChronosSF merged 5 commits into
vnextfrom
iminchev/update-radio-samples

Conversation

@IMinchev64

Copy link
Copy Markdown
Contributor

No description provided.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Hristo313 Hristo313 added status: in-test PR ready for testing status: verified The PR is tested and ready for a merge and removed awaiting-test status: in-test PR ready for testing labels Oct 20, 2026
ChronosSF and others added 2 commits October 29, 2026 16:31
Bring in the Astro migration (#1110), the sample type check (#1132),
AGENTS.md (#1133) and everything else merged to vnext since.

Conflicts: radio disabled and styling. vnext had only converted them
to function components; this branch had already rewritten both 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 six new radio samples (interaction-state, layout, on-off-state,
orientation, overview, tailwind-styling) get what vnext did to every
sample: no sandbox.config.json, README.md instead of ReadMe.md, in the
current format without the CodeSandbox link, the strict add:sample
tsconfig, and package versions from scripts/lib/versions.js. Under the
strict tsconfig, interaction-state guards against a null ref.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Same issue as chip/tailwind-styling: the Aktiv Grotesk @import came
before the @layer statement, and the samples browser compiles sample
CSS through Vite, whose @import handling then drops the Tailwind
imports that follow. The sample rendered without its theme or
utilities.

Put the @layer statement first and the font import right after it,
which works there and standalone.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ChronosSF
ChronosSF merged commit 28178eb into vnext Oct 29, 2026
5 checks passed
@ChronosSF
ChronosSF deleted the iminchev/update-radio-samples branch October 29, 2026 15:06
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