-
Notifications
You must be signed in to change notification settings - Fork 118
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
feat(select): implement select all #746
Conversation
Sorry for the late reply, haven't had much time. Here 1000 items: I don't think we need any big refactors in this task. Feel free to suggest improvements though. Also I realized clear all button removes disabled options. Is this intended? Should it stay that way? @buseselvi |
@ogunb If there is a selected disabled option, clear all shouldn't clear them. Can we fix this here? Thank you for noticing. 🙏🏻 |
Looks good imo but let's add one more check. Since we don't have any demo projects on hand right now, we can ask for @AykutSarac to test it in his existing project before merging. |
Works fine at our project. |
Good job! 🚀 I have a few notes:
|
c233487
to
2359b67
Compare
🎉 This PR is included in version 2.4.0-beta.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 3.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Closes #520