-
Notifications
You must be signed in to change notification settings - Fork 45
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 layout issues #1713
Fix layout issues #1713
Conversation
Deploying with Cloudflare Pages
|
4763fc1
to
ee92b30
Compare
Codecov Report
@@ Coverage Diff @@
## master #1713 +/- ##
=======================================
Coverage 82.97% 82.97%
=======================================
Files 179 179
Lines 4587 4587
Branches 819 820 +1
=======================================
Hits 3806 3806
Misses 781 781
Flags with carried forward coverage won't be shown. Click here to find out more.
|
<Box fill={isMobile}>{props.children}</Box> | ||
<Box | ||
// Prevents Grommet flex overlap issue in smaller viewport | ||
flex={{ shrink: 0 }} |
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.
this doesn't behave the same in landscape
Before https://mz-settings.oasis-wallet.pages.dev/ | After https://mz-responsivelayerfix.oasis-wallet.pages.dev/ |
---|---|
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.
switched to maxHeight. Seems better now.
ee92b30
to
2fcdafc
Compare
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.
Nice
2fcdafc
to
50a97db
Compare
#1710