Skip to content
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

fix: reset selected field #6040

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DubemObi
Copy link

Fixes #6012

I worked on the select field, to enable the reset of a selected field or remove a selected option.

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

  • Added functionality to reset the selected field
  • Modified onClick handler in MenuItem to reset selection if the option matches the current value
  • Updated onChange callback to handle null for resetting selection

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings

Copy link

Welcome!

Hello there, congrats on your first PR! We're excited to have you contributing to this project.
By submitting your Pull Request, you acknowledge that you agree with the terms of our Contributor License Agreement.

Generated by 🚫 dangerJS against e3bbb74

@FelixMalfait
Copy link
Member

Thanks!!! Let's hold before merging this / just want to confirm with @Bonapara this is really what we want, it feels a bit confusing to me

@Weiko
Copy link
Member

Weiko commented Jun 27, 2024

Hi @DubemObi,
Thanks for your contribution 🙏. As @FelixMalfait mentioned above, we rewrote the issue there #6064, do you think you can still tackle this? Let us know so we can assign you to the new issue 🙏

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

Successfully merging this pull request may close these issues.

Ability to Reset Selected Field Value to Empty
3 participants