-
Notifications
You must be signed in to change notification settings - Fork 359
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
Made 'Treatment Summary' page responsive #3643
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
test approved |
@GokulramGHV any update |
@nihal467 The PR is ready.. I saw @khavinshankar's question just now and I've replied to it |
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.
@GokulramGHV have you tried out print:
from tailwind, this is actually easy to use
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.
Looks good, but following @khavinshankar 's suggestion and using the print modifier from Tailwind instead of clsx
would really make the code better, we should generally avoid clsx where possible (Vignesh's words).
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
test approved |
Fixes #3605
Proposed Changes
Screenshot
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist