Skip to content
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

Fix styling issues in the patient details page #6639

Merged
merged 3 commits into from
Nov 21, 2023

Conversation

AshrafMd-1
Copy link
Contributor

@AshrafMd-1 AshrafMd-1 commented Nov 15, 2023

WHAT

🤖 Generated by Copilot at 0d71611

Improved the layout and responsiveness of some components on smaller screens. Modified the PatientHome, RelativeDateUserMention, and ConsultationCard components to use Tailwind CSS classes for spacing, alignment, and contrast.

Proposed Changes

  • Fixes Styling issues in the patient details page #6632
  • Added margin below the Create Consultation button
  • Center aligned The Created and Last Edited cards
  • Added styling to the key and value in the Created and Last Modified fields in Consultation History

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

Merge Checklist

  • Add specs that demonstrate bug / test a new feature.
  • Update product documentation.
  • Ensure that UI text is kept in I18n files.
  • Prep screenshot or demo video for changelog entry, and attach it to issue.
  • Request for Peer Reviews
  • Completion of QA

HOW

🤖 Generated by Copilot at 0d71611

  • Improve the responsiveness and alignment of the tooltip and the user name in the RelativeDateUserMention component by using flex-wrap and justify-center classes (link)
  • Add some margin and contrast to the user name and the date in the ConsultationCard component by wrapping the RelativeDateUserMention component inside a div with ml-1 and text-black classes (link)
  • Create some space between the button and the patient details card in the PatientHome component by adding a mb-2 class to the ButtonV2 component (link)
  • Create some padding between the border and the content in the patient details card in the PatientHome component by adding a pr-2 class to the first div and a pl-2 class to the second div inside the card (link, link)

@AshrafMd-1 AshrafMd-1 requested a review from a team November 15, 2023 12:13
@AshrafMd-1 AshrafMd-1 requested a review from a team as a code owner November 15, 2023 12:13
Copy link

vercel bot commented Nov 15, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
care-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 16, 2023 2:23pm

Copy link

netlify bot commented Nov 15, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit d9a1cb9
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/655625d27d2f6b0008b89451
😎 Deploy Preview https://deploy-preview-6639--care-egov-staging.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@nihal467
Copy link
Member

LGTM

@khavinshankar khavinshankar merged commit 8262237 into ohcnetwork:develop Nov 21, 2023
34 of 36 checks passed
@AshrafMd-1 AshrafMd-1 deleted the Fix-#6632 branch November 21, 2023 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Styling issues in the patient details page
4 participants