ohcnetwork / care_fe

Care is a Digital Public Good enabling TeleICU & Decentralised Administration of Healthcare Capacity across States.
https://care.ohc.network
MIT License
230 stars 418 forks source link

Show more monitors in CNS on very large displays #6503

Open rithviknishad opened 12 months ago

rithviknishad commented 12 months ago

Describe the bug When CNS is opened in large 4K displays, 75% of the screen space is wasted.

To Reproduce Steps to reproduce the behavior:

  1. Visit https://deploy-preview-6396--care-net.netlify.app/facility/0c95c7f0-e1d2-4aff-83fa-933cef60d3a8/cns on a 4K display

Expected behavior On large displays, render more vitals monitors into view per page.

Screenshots

image

kshitijv256 commented 12 months ago

9 monitors if available should be fine in this size screen i guess or maybe increase the size of monitors to use most of screen with same no. as it is

I would like to work on it.

github-actions[bot] commented 11 months ago

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

ayush-seth commented 11 months ago

I can pick this up if still required

MjStar24 commented 7 months ago

hey @nihal467 I would like to work on this can you assign this to me

MjStar24 commented 7 months ago

hey @rithviknishad when I am clicking on the above it is showing no data found

Screenshot 2024-03-18 173337

rithviknishad commented 7 months ago

Try logging in as devdistrictadmin Coronasafe@123 instead.

MjStar24 commented 7 months ago

thanks @rithviknishad its working!

MjStar24 commented 7 months ago

hey @rithviknishad I have raised the pull request for this issue

what I have done :

-by default 6 monitors will displayed per view page for small screens -for large screens 9 monitors will be displayed per view page for 4k displays -for this I have used window.resize and react hooks to check the display size and change number of monitors according to display size

nihal467 commented 5 months ago

@MjStar24 Unassigning you from this issue due to inactivity. Feel free to reassign yourself if you are still working on it.