-
Notifications
You must be signed in to change notification settings - Fork 14
docs: add date input reset to React showcase #5499
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
Conversation
…t-empty-date-form-fields
|
Co-authored-by: mfranzke <787658+mfranzke@users.noreply.github.com>
Co-authored-by: mfranzke <787658+mfranzke@users.noreply.github.com>
Addresses feedback from #5440 to demonstrate empty string reset for date inputs across all framework showcases.
Changes
React showcase (
showcases/react-showcase/src/components/form/index.tsx):value={dateinput}prop to date input (line 121) - converts from uncontrolled to controlled componentsetDateinput('')to reset button handler (line 258)Vue showcase - already implements date input reset at line 60
Aligns React showcase with Angular showcase pattern where
form.get('dateinput')?.setValue('')resets date fields.💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
🔭🐙🐈 Test this branch here: https://design-system.deutschebahn.com/core-web/review/copilot/sub-pr-5440-again