Skip to content

[docs] Update radio group examples#4083

Open
mj12albert wants to merge 1 commit intomui:masterfrom
mj12albert:docs/radio-group-generic
Open

[docs] Update radio group examples#4083
mj12albert wants to merge 1 commit intomui:masterfrom
mj12albert:docs/radio-group-generic

Conversation

@mj12albert
Copy link
Member

@mj12albert mj12albert commented Feb 13, 2026

Some tweaks as radio group accepts a value generic now

@mj12albert mj12albert added the docs Improvements or additions to the documentation. label Feb 13, 2026
@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 13, 2026

commit: 9f70d5e

@mj12albert mj12albert force-pushed the docs/radio-group-generic branch from 37b48cf to 9f70d5e Compare February 13, 2026 09:13
@mui-bot
Copy link

mui-bot commented Feb 13, 2026

Bundle size report

Bundle Parsed size Gzip size
@base-ui/react 0B(0.00%) 0B(0.00%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

@netlify
Copy link

netlify bot commented Feb 13, 2026

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit 37b48cf
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/698eeb039049aa00083f197e
😎 Deploy Preview https://deploy-preview-4083--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Feb 13, 2026

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit 9f70d5e
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/698eeb4f120cd00008b70c44
😎 Deploy Preview https://deploy-preview-4083--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mj12albert mj12albert marked this pull request as ready for review February 13, 2026 09:20
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 13, 2026

Greptile Overview

Greptile Summary

Updated radio group examples to leverage the new generic Value type parameter in the RadioGroup component.

  • Updated the RadioGroup wrapper component to accept and forward the generic type parameter
  • Added explicit type parameter <'ssd' | 'hdd'> in hero demo for type-safe storage type selection
  • Removed unnecessary type cast in TanStack Form demo by relying on proper generic typing, simplifying the onValueChange handler

The changes improve type safety across the documentation examples and demonstrate best practices for using the RadioGroup component with typed values.

Confidence Score: 5/5

  • This PR is safe to merge with no issues
  • The changes are documentation-only updates that improve type safety by properly utilizing the RadioGroup component's generic type parameter. All changes are consistent, well-structured, and eliminate unnecessary type casts.
  • No files require special attention

Important Files Changed

Filename Overview
docs/src/app/(docs)/react/handbook/forms/demos/components/radio-group.tsx Added generic Value type parameter to RadioGroup wrapper component to properly support typed values
docs/src/app/(docs)/react/handbook/forms/demos/hero/tailwind/index.tsx Added explicit type parameter `<'ssd'
docs/src/app/(docs)/react/handbook/forms/demos/tanstack-form/tailwind/index.tsx Removed unnecessary type cast in onValueChange handler by leveraging RadioGroup's generic typing

Last reviewed commit: 9f70d5e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants