feat(slider): add Slider atom component#242
Conversation
|
El componente está muy bien encaminado, pero acá hay un gap de accesibilidad en range mode. En Podés mantener un label de grupo en el root y sumar una estrategia de label por thumb que combine contexto + nombre distintivo, o agregar IDs ocultos por thumb. También agregaría un test de regresión para |
|
Gracias por el update @ludevdot . Revisé de nuevo el último commit ( El problema sigue estando en Para cerrar este punto hace falta cambiar la estrategia de labelling en range mode, por ejemplo componiendo |
|
Gracias no había caído en el |
Summary
Add the Slider atom as a complete component PR for issue #29, including Radix-based behavior, range support, Progress-aligned
color/roundedvariants, tests, and Storybook examples.Closes #29
Type of change
Repository checklist
<type>(<optional scope>): <description>Closes #NNNComponent checklist (skip if not applicable)
types.ts,use*.ts,Component.tsx,index.ts,*.stories.tsx)types.ts, not inlinetheme.cssanytypes — TypeScript strictHow to test
pnpm test -- Slider.test.tsxpnpm testpnpm exec tsc --noEmitpnpm run storybookAtoms/Sliderstories: Default, Controlled, Range, Disabled, Sizes, Colors, Rounded, WithExternalLabel, WithFieldDescription.Screenshots / recordings (if applicable)
Draft PR: screenshots can be added after visual review if needed.
Notes for reviewer
@radix-ui/react-sliderfor pointer, drag, focus, keyboard, range, and disabled behavior.thumbLabels.colorandroundedvariants.