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

Layout problems on iPhone 14 Pro #720

Closed
wutschel opened this issue Sep 30, 2022 · 6 comments · Fixed by #727
Closed

Layout problems on iPhone 14 Pro #720

wutschel opened this issue Sep 30, 2022 · 6 comments · Fixed by #727

Comments

@wutschel
Copy link
Collaborator

wutschel commented Sep 30, 2022

On the new iPhone 14 Pro there are layout problems. Some UI elements are not positioned correctly.

Examples:
https://abload.de/img/simulatorscreenshot-ihqigp.png (info label for supported versions)
https://abload.de/img/simulatorscreenshot-ieyc1j.png (error message)
https://abload.de/img/simulatorscreenshot-iy9fed.png (message)
https://abload.de/img/simulatorscreenshot-i6fdcq.png (server info overlaps with eye)

@wutschel
Copy link
Collaborator Author

Obviously I miss something in how the view frame itself is positioned on the screen, or on how I calculate offsets/insets. In some places I can now use CGRectGetMaxY(self.navigationController.navigationBar.frame); which on iPhone 14 Pro is 103, on iPhone Xs it is 88. This fixes the layout problem for some views.
But oi other places I just use UIApplication.sharedApplication.statusBarFrame.size.height as offset, which his equal for both 14 Pro and Xs. For these I currently could not figure out what is wrong.

@wutschel
Copy link
Collaborator Author

Compiling the App with Xcode 14 resolves the problems.

@kambala-decapitator
Copy link
Collaborator

seems not using autolayout and using manual layout improperly starts causing issues :)

I'd still suggest to release next version built with Xcode 13, after that we can switch to 14.

@wutschel
Copy link
Collaborator Author

wutschel commented Oct 1, 2022

seems not using autolayout and using manual layout improperly starts causing issues :)

You're welcome to join reworking. ;)

I'd still suggest to release next version built with Xcode 13, after that we can switch to 14.

Agreed, migrating now would require too much time for testing via TF.

@kambala-decapitator
Copy link
Collaborator

I guess this would be the last thing to adjust for the release?

@wutschel
Copy link
Collaborator Author

wutschel commented Oct 1, 2022

I would just accept the small issue and fix it with next release when migrating to Xcode 14. But I have another change I would like to add, see #721.

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 a pull request may close this issue.

2 participants