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

Multiple scrolls- unnecessary: doctor connect #6940

Closed
aparnacoronasafe opened this issue Dec 29, 2023 · 5 comments · Fixed by #7198
Closed

Multiple scrolls- unnecessary: doctor connect #6940

aparnacoronasafe opened this issue Dec 29, 2023 · 5 comments · Fixed by #7198
Assignees

Comments

@aparnacoronasafe
Copy link
Member

Describe the bug
When there are over 5 staff of a particular type appearing on the doctor connect panel, the section (Doctor/staff/teleicu Hub) has a separate scroller while the whole Doctor connect panel has a second scroller. This seems confusing and unnecessery.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'any patient with consultation filed'
  2. Click on '.Doctor connect.'
  3. if there are over 5 listing in any of the section, you will see a section scroller in addition to the doctor connect panel scroller.
Screenshot 2023-12-29 at 5 44 01 PM

This was done maybe to retain all the section headings in view on the panel.

Can we think of a better visualisation? Maybe collapsable sections?

@github-project-automation github-project-automation bot moved this to Triage in Care Dec 29, 2023
@rithviknishad rithviknishad self-assigned this Dec 29, 2023
@rithviknishad rithviknishad moved this from Triage to Up Next in Care Dec 29, 2023
@rithviknishad rithviknishad added the waiting for related PR a co-related detail PR is under construction label Jan 3, 2024
@rithviknishad
Copy link
Member

Waiting for changes from #6924

@rithviknishad
Copy link
Member

rithviknishad commented Jan 3, 2024

@aparnacoronasafe I'm thinking of removing the sections, instead a single list would be shown, however, few things that'd help in improving the UX:

  1. Sort will be maintained, doctors followed by staff followed by tele icu hub.
  2. A chip will be shown in the user card indicating what group the user belongs to (doctor/staff/teleicu hub)
  3. A search bar on top to search for the users

@rithviknishad rithviknishad removed their assignment Jan 17, 2024
@rithviknishad rithviknishad removed the waiting for related PR a co-related detail PR is under construction label Jan 17, 2024
Copy link

github-actions bot commented Feb 1, 2024

Hi, @gigincg, @nihal467, @khavinshankar, @mathew-alex, @aparnacoronasafe, This issue has been automatically marked as stale because it has not had any recent activity.

@thedevildude
Copy link
Contributor

I'd like to work on the issue

@thedevildude
Copy link
Contributor

@aparnacoronasafe
@rithviknishad

What do you think of this mockup for doctor connect? I've also added three filter buttons on top for Doctors, Nurses and TeleICU Hub

DoctorConnect

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

Successfully merging a pull request may close this issue.

3 participants