Skip to content
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

PTEDashboard to respect safe area insets #28

Merged
merged 1 commit into from
Oct 29, 2019
Merged

PTEDashboard to respect safe area insets #28

merged 1 commit into from
Oct 29, 2019

Conversation

fawkeswei
Copy link
Contributor

PTEDashboard will now appear within safe area, much easier to view/expand the log viewer on devices with a notch.

Would be much easier to view/expand log view on devices with a notch
@rivera-ernesto rivera-ernesto merged commit 6ad7fb9 into PTEz:master Oct 29, 2019
@rivera-ernesto
Copy link
Member

Thanks!

@fawkeswei
Copy link
Contributor Author

Can you release a new CocoaPods version including this fix? Greatly appreciated

@rivera-ernesto
Copy link
Member

Sure. Will try to do it today.

@@ -40,7 +40,7 @@ + (PTEDashboard *)sharedDashboard
dispatch_once(&onceToken, ^
{
CGRect frame = UIScreen.mainScreen.bounds;
_sharedDashboard = [[self alloc] initWithFrame:frame];
_sharedDashboard = [[self alloc] initWithFrame:UIEdgeInsetsInsetRect(window.frame, window.safeAreaInsets)];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The line above should be changed to

UIWindow *window = UIApplication.sharedApplication.keyWindow;

@rivera-ernesto
Copy link
Member

Can you please check this build error?

@fawkeswei
Copy link
Contributor Author

Replied in the thread

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants