Admin/Facility/(click to edit) goofy UI

LibreHealthIO / lh-ehr

LibreHealth EHR - Free Open Source Electronic Health Records

Other

235 stars 261 forks source link

Admin/Facility/(click to edit) goofy UI #1387

Open aethelwulffe opened 5 years ago

aethelwulffe commented 5 years ago

Screenshot from NHANS, normal demo dev-tip down, and looks even worse on my server!

muarachmann commented 5 years ago

@aethelwulffe izimodal has options for width, height and fullscreen options. This can be exploited here. Let me know if there is something to do about this and i will file in a PR.

muarachmann commented 5 years ago

Here is my screen on light

screen shot 2019-01-21 at 7 30 01 am

Here on prism

screen shot 2019-01-21 at 7 34 28 am

aethelwulffe commented 5 years ago

Not here. Test on a 1080p screen maybe? Look at the NHANS demo. Same on all my servers and local machine.

aethelwulffe commented 5 years ago

Nothing in the logs as an error.

aethelwulffe commented 5 years ago

Ah. Try renaming the default facility, then add a second facility.

aethelwulffe commented 5 years ago

After that, look again.

aethelwulffe commented 5 years ago

Always test data with a big data set...lesson learned again.

muarachmann commented 5 years ago

I will try then re-sizing my screen to get the behaviour. Thanks for taking out time to review and leave some comments. Love to have you guys around to poke a little :)

On Tue, Jan 22, 2019 at 5:07 PM Art Eaton notifications@github.com wrote:

Always test data with a big data set...lesson learned again.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/LibreHealthIO/lh-ehr/issues/1387#issuecomment-456456256, or mute the thread https://github.com/notifications/unsubscribe-auth/APl-Xo9nu71yPPdHgrFQbYFcgXHISLnJks5vFzcqgaJpZM4aIfBb .

aethelwulffe commented 5 years ago

Add and edit some facilities. I was not able to get a clean interface by doing any resizing or tab adjustments.

aethelwulffe commented 5 years ago

@muarachmann @teryhill @muarachmann Here is my "twelve merges in 9 hours" integration test site: "demouser" and "Testing lots of stuff."

The URL is https://samwise.suncoastconnection.com:3443/libre_ehr/interface/login/login.php?site=2019JAN22MERGEs Mua, please look at the edit facility before I change anything else. Totally fresh install.

My results are with Firefox specifically.