Skip to content

Conversation

DoLovya
Copy link

@DoLovya DoLovya commented Sep 12, 2025

Fix #3914

@DoLovya
Copy link
Author

DoLovya commented Sep 12, 2025

Hi @Keboo , my current PR needs approval. I did a search and it seems that since this is my first PR, can you help me. Thank you.

@nicolaihenriksen
Copy link
Contributor

@DoLovya (@Keboo), this is basically a copy-paste of @corvinsz work/PR in #3915 with only cosmetic differences!? That PR is not yet merged because of a failing test for the use of clear button in DatePicker I believe; @corvinsz is probably looking into that.

You are more than welcome to contribute with PRs to fix the open issues, but please don't take other peoples work and post as your own. Credit should go to @corvinsz for diagnosing and fixing this issue when his PR eventually merges.

@DoLovya DoLovya closed this Sep 13, 2025
@DoLovya DoLovya deleted the fix/#3914 branch September 13, 2025 01:23
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.

TextBox clear button visible even when no text
2 participants