Skip to content

Add SliderBlock component with label, inline editing, and range support#270

Open
i-subham23 wants to merge 4 commits intothesysdev:mainfrom
i-subham23:feature/slider-block
Open

Add SliderBlock component with label, inline editing, and range support#270
i-subham23 wants to merge 4 commits intothesysdev:mainfrom
i-subham23:feature/slider-block

Conversation

@i-subham23
Copy link
Contributor

@i-subham23 i-subham23 commented Mar 8, 2026

Summary

  • New SliderBlock component: A high-level slider wrapper that adds a label header, inline value editing controls (text Input for continuous mode, Select dropdown for discrete mode), range support with dual controls, input validation with error messages, and debounced onValueCommit.
  • genui-lib Slider updated: Now renders SliderBlock instead of the base Slider, with an optional label prop added to the schema.
  • Storybook updated: Replaced base Slider stories with SliderBlock variants covering continuous, discrete, range, and disabled states.

- Introduce SliderBlock: a high-level slider wrapper with a label header,
  inline value controls (text Input for continuous, Select dropdown for
  discrete), range support with two controls, input validation with error
  messages, and debounced onValueCommit
- Merge sliderBlock styles into slider.scss (single file); fix input
  placement by aligning header items properly and adding flex-start on
  error state
- Disable Input and Select controls when the slider is disabled
- Fix SelectItem children to use String(o) so value "0" displays
  correctly in the trigger
- Update genui-lib Slider to render SliderBlock instead of the base
  Slider, adding optional label prop to the schema
- Replace storybook stories with SliderBlock variants (continuous,
  discrete, range, disabled)
- Export SliderBlock from the Slider barrel and dependencies

Made-with: Cursor
@vercel
Copy link

vercel bot commented Mar 8, 2026

@i-subham23 is attempting to deploy a commit to the thesys-devs Team on Vercel.

A member of the Team first needs to authorize it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant