Updating the tooltip to use Care's custom components and replacing the PageTitle with Page component. #5640
Conversation
|
@yogeshbhutkar is attempting to deploy a commit to the Coronasafe Team on Vercel. A member of the Team first needs to authorize it. |
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Refer #5006 on how to replace some components.
To add tooltip to a button, use ButtonV2 and set it's tooltip and tooltipClass props.
| @@ -146,8 +146,8 @@ export default function InventoryLog(props: any) { | |||
| </p> | |||
| </td> | |||
| <td> | |||
| <Tooltip | |||
| title={ | |||
| <ToolTip | |||
There was a problem hiding this comment.
The custom tooltip component also is deprecated if you've read the doc comments of that component. Refer #5006 on how to replace each component properly.
For tooltip we are using classes over component because, less JSX the better!
| </button> | ||
| </Tooltip> | ||
| )} | ||
| <button |
There was a problem hiding this comment.
You've to replace this button with ButtonV2
| className="btn btn-default" | ||
| > | ||
| <span className="text-red-500"> | ||
| <i className="fas fa-exclamation-circle pr-2"></i> |
There was a problem hiding this comment.
Font-awesome icons can be migrated to use CareIcons.
|
Also, update the PRs title to be something meaningful like what your PR does. And it's nice to have screenshots too! |
yogeshbhutkar
changed the title
WHAT
🤖 Generated by Copilot at d228bbe
Refactored the inventory log UI to use common components. This change enhances the code quality and the user experience of the facility inventory log page.
Proposed Changes
src/Components/Facility/InventoryLog.tsx: #4957@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers