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

Dosage heading is misaligned in the prescription module #8645

Closed
1 task
nihal467 opened this issue Sep 26, 2024 · 8 comments · Fixed by #8751
Closed
1 task

Dosage heading is misaligned in the prescription module #8645

nihal467 opened this issue Sep 26, 2024 · 8 comments · Fixed by #8751
Assignees
Labels

Comments

@nihal467
Copy link
Member

nihal467 commented Sep 26, 2024

Describe the bug

When we create a PRN prescription with a large note, the dosage heading is misaligned in the normal view

To Reproduce
Steps to reproduce the behavior:

  1. Go to patient consultation page
  2. Click on medicine and add a medicine (PRN) with a long indicator
  3. See error

Expected behavior

  • the dosage heading in the prn prescription should be center aligned

Screenshots

image

@nihal467 nihal467 added this to Care Sep 26, 2024
@github-project-automation github-project-automation bot moved this to Triage in Care Sep 26, 2024
@nihal467 nihal467 moved this from Triage to Up Next in Care Sep 26, 2024
@HarshikaAdarsh
Copy link

Hey, is anyone working on this issue?
If not, I would like to work on it.

@syedfardeenjeelani
Copy link
Contributor

hello @HarshikaAdarsh are you still working on it?

@ayushpatil2122
Copy link
Contributor

image

But on my machine, everything seems to be fine, even though I added a long note

@rithviknishad
Copy link
Member

@ayushpatil2122 correction, it's not note, it's indicator. I've updated the issue.

cc: @nihal467

@siddharthbaleja7
Copy link

@rithviknishad I would like to work on this issue.Can you please assign this issue to me?

@NitinPSingh
Copy link
Contributor

NitinPSingh commented Oct 5, 2024

hi @rithviknishad , what if we made it multiline it would solve problem
Image

or even if truncate it and show the full name on hover,

if no one is working i would love to contribute

@Sulochan-khadka
Copy link
Contributor

I think we can have a scrollable area for indicator, which wont ruin the UI. Also the user can scroll for complete info if needed. Because if we implement the solution formerly stated , the the duration part comes in the scroll which does not seem good.

This is the issue i am talking about:
Image

The solution i am proposing:
Image

Obviously we can style as required.

Let me know your reviews @rithviknishad @nihal467

@nihal467 nihal467 added the question Further information is requested label Oct 8, 2024
@rithviknishad
Copy link
Member

We could simply either wrap it in newline or elipsis the text beyond a certain max-width. User can anyways see the full information when they click on the row if needed.

@rithviknishad rithviknishad removed the question Further information is requested label Oct 8, 2024
Sulochan-khadka added a commit to Sulochan-khadka/care_fe that referenced this issue Oct 9, 2024
@bodhish bodhish moved this from Up Next to In Progress in Care Oct 18, 2024
@bodhish bodhish moved this from In Progress to Review required in Care Nov 10, 2024
@github-project-automation github-project-automation bot moved this from Review required to Done in Care Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

8 participants