-
Notifications
You must be signed in to change notification settings - Fork 477
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
Migrate to Headless UI v2 #8260
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Test summaryRun details
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud |
@khavinshankar the dropdown behavior has some anima effect and is closing slowly, try to checking for each elements presence post selection from dropdown, might solve the issue, post fixing the cypress, mark it for testing |
Mentioning the issue here as a reminder: we can work on issue 7849 only after merging this V2 PR. |
👋 Hi, @khavinshankar, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
👋 Hi, @khavinshankar, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@khavinshankar when we multi-select from dropdown in advance filters and, then click apply, it just closes the dropdown, but that shouldn't be the behavior , it should allow users to use apply button directly |
conditional approving it, will be opening a new issue to fix the multi-select component mentioned in the previous comment |
@khavinshankar Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Proposed Changes
@nihal467 no visible ui changes but all the components should be rendered as before
#7849 (comment)
Mentioning the issue here as a reminder: we can work on issue 7849 only after merging this V2 PR