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
(fix) O3-2852: Fix the viewport size of the Start Visit form on tablet #1765
(fix) O3-2852: Fix the viewport size of the Start Visit form on tablet #1765
Conversation
@denniskigen I Fixed this error. Can you review it.. |
@Madhu-mac Kindly Add the ticket link associated with the issue. |
@@ -46,7 +45,7 @@ | |||
display: flex; | |||
flex-direction: column; | |||
justify-content: space-between; | |||
height: calc(100vh - 6rem); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Madhu-mac changing the height to 100vh
breaks the UI on the desktop viewport. The correct values are:
var(--desktop-workspace-window-height)
for desktopvar(--tablet-workspace-window-height)
for tablet
As defined here
This is what the corresponding UI looks like following those changes:
Desktop
Tablet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking a stab at this, @Madhu-mac.
Requirements
Summary
Fixes the Start Visit form UI on the tablet viewport so that the form covers the entire vertical height of the viewport.
Screenshots
Before
After
Related issue
https://openmrs.atlassian.net/issues/O3-2852