-
Notifications
You must be signed in to change notification settings - Fork 491
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:Inline view of selected options of Multiselect #7287
feat:Inline view of selected options of Multiselect #7287
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! Just a minor suggestion, can we decrease the padding when an item is selected, so that it looks consistent in size with other types input fields?
…or79/care_fe into ui-enhancement/inline-view
@rithviknishad please review it once i m just fix it padding to look more consistent ui |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@Ashesh3 @khavinshankar @creator79 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
Implement showing the selected options within the input itself. Overflowing chips could be wrapped in a new line within the input itself or be horizontally scrollable in one line. Additionally, each chip could optionally have a remove button to the right of it.
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist