feat(SelectMenu): allows to clear search query on close

[DarkGhostHunter]

🔗 Linked issue

Resolves #883

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Currently, <USelectMenu> doesn't clear the search query when an option is selected, or when the menu is closed by selection.

This allows to clear it when the menu is closed, by either manually closing it or selecting an option, by using the attribute clear-search-on-close.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
ui	✅ Ready (Inspect)	Visit Preview	Nov 21, 2023 5:57pm

[DarkGhostHunter]

I reworked this PR to only clear the search query on close. In other words, the search input becomes empty when:

• The user closes the menu manually
• The user selects an option, closing the menu.

When the user selects multiple options, the menu doesn't close. This makes clear on update unnecessary. Also, it's not expected the option will change while the menu is open, so there is no need to clear on update.

[benjamincanac]

Would you mind explaining the difference with #890? I now have to choose between both 😅

[DarkGhostHunter]

Would you mind explaining the difference with #890? I now have to choose between both 😅

Sorry! When I made it I just oversaw it on the pulls.

The main difference is that the search query is kept as-is by default, where as #890 does not (clears on close by default). But #890 adds a global config to enable or disable this. I'll add that too.