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

Enhance Discussion Notes Chat History with Infinite Scrolling or Better Navigation Solution #9188

Open
nihal467 opened this issue Nov 22, 2024 · 4 comments · May be fixed by #9190
Open

Enhance Discussion Notes Chat History with Infinite Scrolling or Better Navigation Solution #9188

nihal467 opened this issue Nov 22, 2024 · 4 comments · May be fixed by #9190
Assignees
Labels
needs-triage question Further information is requested work-in-progress

Comments

@nihal467
Copy link
Member

Describe the bug

When scrolling from the bottom to the top of a discussion with 20 chat entries in the any "Discussion Notes" section, the page reloads unexpectedly after reaching the 15th chat. This forces the user to start scrolling from the bottom again, disrupting the navigation experience.

To Reproduce
Steps to reproduce the behavior:

  1. Navigate to the Patient tab and select any patient card.
  2. Click on the Discussion Notes section.
  3. Add 20 chat messages.
  4. Attempt to scroll from the bottom to the top of the chat history.
  5. See error

Expected behavior

The user should be able to smoothly scroll from the bottom to the top of the chat history without the page reloading or resetting the scroll position.

Screenshots

Image

@JavidSumra
Copy link
Contributor

Hey @nihal467 I would like to work on this issue

@JavidSumra
Copy link
Contributor

Hey @nihal467 same issue persist with Discussion Notes shall i fix that as well?

@github-actions github-actions bot added needs-triage question Further information is requested labels Nov 23, 2024
@JavidSumra JavidSumra linked a pull request Nov 23, 2024 that will close this issue
6 tasks
@nihal467
Copy link
Member Author

Yup fix it for both @JavidSumra

@JavidSumra
Copy link
Contributor

JavidSumra commented Nov 23, 2024

@nihal467 can you please test the PR #9190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage question Further information is requested work-in-progress
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants