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

Overflow text, z-index and reply functionality breaking in Notes section. #8662

Closed
shauryag2002 opened this issue Sep 29, 2024 · 6 comments
Closed

Comments

@shauryag2002
Copy link
Contributor

Describe the bug

  1. Overlapping elements due to z-index changes.

    • The z-index of the "created at" in the single chat header causes overlapping issues.
  2. After entering long text:

    • Entering a long single word breaks the UI.
    • Example of word:
      Testtttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttt
    • Clicking the reply button also causes the UI to break.
  3. The "Edit History" for note popup breaks if the text is too large.

To Reproduce

Steps to reproduce the behavior:

  1. Go to the Patients tab and click View patient notes.
  2. Enter a large single word (50 to 70 characters).
  3. Observe the error where the text overflows outside the container.
  4. Click the reply icon, and observe the error again as the text overflows.
  5. Click the View history icon and see the UI break again due to the long text.

Expected behavior

  • The long text should wrap or be truncated so that it doesn’t overflow the UI.

Screenshots

z-index issue:
Notes_zIndex
Text overflow issue:
Notes_text_overflow
Reply section text overflow:
Notes_reply_overflow
Edit history pop up text overflow:
Notes_edit_history

Solution Video:

Notes_chat_fix.mp4

Desktop (please complete the following information):

  • OS: Windows
  • Browser: Brave
  • Version: 1.70.119
@syedfardeenjeelani
Copy link
Contributor

@shauryag2002 i would like to try to contribute in this issue can you assign it to me

@shauryag2002
Copy link
Contributor Author

Sorry @syedfardeenjeelani, I am working on this issue and created a Pull request for this issue.

@syedfardeenjeelani
Copy link
Contributor

no problem @shauryag2002

@rithviknishad
Copy link
Member

rithviknishad commented Sep 30, 2024

Closing as not relevant. Works fine without issues for the longest english dictionary word (45 chars) pneumonoultramicroscopicsilicovolcanoconiosis.

image

@rithviknishad rithviknishad closed this as not planned Won't fix, can't repro, duplicate, stale Sep 30, 2024
@Sulochan-khadka
Copy link
Contributor

Image
Is this solved @rithviknishad

@rithviknishad
Copy link
Member

Nope, you can create a separate issue for that alone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants