-
Notifications
You must be signed in to change notification settings - Fork 437
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
Tailwind src/Components/Facility/Investigations/Reports/* #5716
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. |
👋 Hi, @khavinshankar, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@khavinshankar We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌. |
WHAT
🤖 Generated by Copilot at 5c7faea
Refactored the investigations reports page and components to use custom form fields and native HTML elements with tailwind CSS. Added a select all feature and a loading indicator to the
AutocompleteMultiSelect
component. Improved the styling and layout of theMultiSelectMenuV2
component.Proposed Changes
src/Components/Facility/Investigations/Reports/index.tsx
: #4971src/Components/Facility/Investigations/Reports/ReportTable.tsx
: #4972@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at 5c7faea
ReportTable.tsx
andindex.tsx
to improve performance and consistency (link, link, link, link, link, link, link, link, link, link)isLoading
andselectAll
props toAutocompleteMultiSelectFormField
component and implement logic for selecting and deselecting all options inAutocompleteMultiselect.tsx
andindex.tsx
to enhance functionality and user experience (link, link, link, link, link, link)ReportTable.tsx
,index.tsx
,AutocompleteMultiselect.tsx
, andMultiSelectMenuV2.tsx
to optimize code quality and readability (link, link, link, link, link, link)handleGroupSelect
function inindex.tsx
to accept aFieldChangeEvent
object and access thevalue
property instead of thetarget.value
property to fix a type error and align with theAutocompleteMultiSelectFormField
component (link)handleSelectAllClick
function inindex.tsx
as it was no longer needed after replacing theLegacyMultiSelectField
component with theAutocompleteMultiSelectFormField
component (link)dropdownOptionClassNames
function inMultiSelectMenuV2.tsx
to add thew-full
class to the option elements to ensure consistent width (link)