-
Notifications
You must be signed in to change notification settings - Fork 2
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 ph vanishing when the app enter background #381
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #381 +/- ##
=======================================
Coverage 54.01% 54.01%
=======================================
Files 168 168
Lines 3710 3710
=======================================
Hits 2004 2004
Misses 1706 1706 ☔ View full report in Codecov by Sentry. |
@@ -37,7 +37,7 @@ class PhMainPage extends HookConsumerWidget { | |||
), | |||
const SizedBox(height: 10), | |||
SizedBox( | |||
width: MediaQuery.of(context).size.width - 60, | |||
width: 250, |
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.
Have you tested this on different screen format ?
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.
250 pixels is really small there won't be any problems (I can't really test on little screen format)
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.
Works great!
Require #380