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

Changes for Xcode 14, iOS 16, new devices #230

Merged
merged 11 commits into from Nov 4, 2022
Merged

Changes for Xcode 14, iOS 16, new devices #230

merged 11 commits into from Nov 4, 2022

Conversation

vincentneo
Copy link
Collaborator

@vincentneo vincentneo commented Aug 8, 2022

Changes:

  • Minimum deployment for iOS and watchOS to comply with Xcode 14.
    • iOS 9 ~> iOS 11
    • watchOS 2 ~> watchOS 4
    • Fix deprecation notices brought by min deployment change
  • Change modern MKCompassButton implementation added in fix iOS 15 compass sizing issue #219, to rectify inability to use @available on lazy variables in Xcode 14
  • Fix Map tiles are covering the track polyline on iOS 16 / Xcode 14.0 #232, overlay issue specific to iOS 16 only, where third party tiles will always try to be top most overlay
  • Changes to MapCache, using a branch for now
  • Changes to CoreGPX, using a branch for now
  • Fix iPhone 14 Pro series safe area misalignment #233, by extensively changing top elements from fixed frame initialisations to using constraints respecting safe areas
  • Rectify warnings, still 1 CoreLocation UI unresponsiveness left.

@vincentneo vincentneo linked an issue Sep 14, 2022 that may be closed by this pull request
iOS 12 as base deployment
Seems to fix said issue except:
- when user pauses and changes tile type
- when polyline overlay no longer considered current, as tile overlay keeps becoming the top overlay when map camera changes location
- not tested for pre iOS 15

Focus should be at the tile overlay rather than segment overlay it seems.
@vincentneo vincentneo changed the title Changes for Xcode 14. Changes for Xcode 14, iOS 16, new devices Sep 15, 2022
vincentneo and others added 5 commits September 18, 2022 00:36
by using constraints for basically everything. fix #233
uses https://github.com/vincentneo/MapCache/tree/ios16-add-overlay-patch fork rather than deployed 0.9 version.

special thanks to @Frank-Peter for reporting issue, testing and the extension from ec4edbd#commitcomment-84238460

Co-Authored-By: Frank-Peter <64739349+Frank-Peter@users.noreply.github.com>
resolves 'UI unresponsiveness' warning by Xcode. IMO, too insignificant to deal with alternative methods, such as the recommendation.

I did not resolve the other issue at all, since the goal of checking that is to interrupt user when location services is disabled.
@cs4alhaider
Copy link

@vincentneo thank you for this, wouldn't be better if this be merged with Xcode 14 branch?

@vincentneo
Copy link
Collaborator Author

@cs4alhaider I will be merging this to master branch today.

@vincentneo
Copy link
Collaborator Author

Considering that iOS 16.0.x and 16.1 has been out for a while, and considering no review from @merlos, I will merge this to master.

It will be using a fork of MapCache for the time being, as I deem it important enough for iOS 16 users, since it fixes #232.

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.

iPhone 14 Pro series safe area misalignment Map tiles are covering the track polyline on iOS 16 / Xcode 14.0
3 participants