-
Notifications
You must be signed in to change notification settings - Fork 263
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
Admin/Facility/(click to edit) goofy UI #1387
Comments
@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. |
Not here. Test on a 1080p screen maybe? |
Nothing in the logs as an error. |
Ah. Try renaming the default facility, then add a second facility. |
After that, look again. |
Always test data with a big data set...lesson learned again. |
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 ***@***.***> 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
<#1387 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/APl-Xo9nu71yPPdHgrFQbYFcgXHISLnJks5vFzcqgaJpZM4aIfBb>
.
|
Add and edit some facilities. I was not able to get a clean interface by doing any resizing or tab adjustments. |
@muarachmann @teryhill @muarachmann The URL is https://samwise.suncoastconnection.com:3443/libre_ehr/interface/login/login.php?site=2019JAN22MERGEs My results are with Firefox specifically. |
Screenshot from NHANS, normal demo dev-tip down, and looks even worse on my server!
The text was updated successfully, but these errors were encountered: