-
Notifications
You must be signed in to change notification settings - Fork 2.1k
style(tokens): update label to helper-text #21133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
style(tokens): update label to helper-text #21133
Conversation
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #21133 +/- ##
=======================================
Coverage 92.61% 92.61%
=======================================
Files 515 515
Lines 38225 38225
Branches 5813 5873 +60
=======================================
Hits 35401 35401
+ Misses 2675 2674 -1
- Partials 149 150 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Closes #19462
Switch validation messages (
invalidTextandwarnText) fromlabeltohelper-textback to thehelper-texttoken instead oflabel.I checked where the validation text actually comes from, so anything using
useNormalizedInputProps, likeTextInput,Dropdown,NumberInput,Slider,RadioButton,TimePicker, etc., ends up rendering a<className="…form-requirement">. Once we swapped the Sass for.form-requirementtohelper-text-01, every one of those components picked up the new token automatically. Even components with custom markup, likeFileUploader, the checkbox/radio group wrappers, or the list-box family, still style their validation blocks with.form-requirement, so they get the same change through the shared mixin. (ProgressBaralready matched)Changelog
New
None.Changed
.cds--form-requirement, which everyuseNormalizedInputPropsconsumer renders for validation (TextInput,Dropdown,NumberInput,Slider,RadioButton,TimePickeretc.), now useshelper-text-01instead oflabel-01.FileUploader’scustom validation blocks (title + supplement inside an invalid row and the deprecated summary block) also switch tohelper-text-01so its errors match the sharedhelper textstyle.Removed
None.Testing / Reviewing
React or WC Deploy Preview> check any component (TextInput,Dropdown,NumberInput,Slider,RadioButton,TimePicker, etc.), setinvalid=trueorwarn=true, inspect the validation message and confirm.cds--form-requirementcomputes to thehelper-texttoken.Screen.Recording.2025-12-05.at.11.43.19.mov
PR Checklist
As the author of this PR, before marking ready for review, confirm you:
[ ] Updated documentation and storybook examples[ ] Wrote passing tests that cover this change[ ] Addressed any impact on accessibility (a11y)[ ] Tested for cross-browser consistencyMore details can be found in the pull request guide