-
Notifications
You must be signed in to change notification settings - Fork 347
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
Tailwinds Daily Rounds Form #5799
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, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
LGTM |
@rithviknishad 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 e9ecc32
This pull request adds new form field components for blood pressure and temperature inputs, enhances the existing autocomplete and form field components with additional props and styles, and refactors the page component to use a helper function for conditional class names. These changes aim to improve the user interface and the code quality of the care_fe project.
Proposed Changes
src/Components/Patient/DailyRounds.tsx
: #4976RangeAutocompleteFormField
TemperatureFormField
BloodPressureFormField
group-hover/option:*
AutocompleteFormField
to showoptionIcon
in input box.@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at e9ecc32
classNames
function to simplify the class name prop logic (link)Autocomplete
component (link)