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

Update DeviceModel.swift #97

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jminutaglio
Copy link
Contributor

Adds:

  • iPad Air M2 11 & 13
  • iPad Pro M4 11 & 13

Adds:
- iPad Air M2 11 & 13
- iPad Pro M4 11 & 13
@jminutaglio
Copy link
Contributor Author

adds new iPads May 2024

@vincentneo
Copy link
Collaborator

Hi @jminutaglio,

Apologies for the delay, was busy with a new app launch recently.

From what I can tell from Xcode 15.4, the correct model numbers should be:

Device Size Model
iPad Air M2 11 inch iPad14,8
iPad14,9
iPad Air M2 13 inch iPad14,10
iPad14,11
iPad Pro M4 11 inch iPad16,3-A
iPad16,3-B
iPad16,4-A
iPad16,4-B
iPad Pro M4 13 inch iPad16,5-A
iPad16,5-B
iPad16,6-A
iPad16,6-B

As you can see, there's an oddity here, with the pro models having a A/B suffix, which is why I have yet to modify the codes yet, as I do not know what it meant, and whether if it is even reflected during runtime.

In Xcode's device_traits.db there's a "ThinningConfiguration" column that corresponds to A/B, but no further ideas on it.

Interestingly the M2 iPad Pros (14,3 to 14,6) have them too. I wonder if the current implementation of ours for those devices work properly.

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.

None yet

2 participants