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

Add new iPhone 15 Series to DeviceKit #360

Closed
wants to merge 4 commits into from
Closed

Add new iPhone 15 Series to DeviceKit #360

wants to merge 4 commits into from

Conversation

wo-ist-henry
Copy link
Contributor

Add support for ne the iPhone Series, also added the new camera type Periscope to differentiate between the zoom capabilities that the iPhone 15 Pro Max and all the others has.

wo-ist-henry and others added 4 commits September 14, 2023 11:09
Co-Authored-By: Johannes Ebeling <14994778+technocidal@users.noreply.github.com>
Co-Authored-By: Johannes Ebeling <14994778+technocidal@users.noreply.github.com>
Co-Authored-By: Johannes Ebeling <14994778+technocidal@users.noreply.github.com>
@LaurentiuUngur
Copy link

LaurentiuUngur commented Sep 15, 2023

Maybe new watches should be added as well

@wo-ist-henry
Copy link
Contributor Author

I thought I'd add them separately so there's not a huge PR coming in.

@wo-ist-henry
Copy link
Contributor Author

@LaurentiuUngur I added them with a new PR, #361

@Zandor300
Copy link
Member

@wo-ist-henry Apple is using "telephoto camera" to describe the 5x lens on iPhone 15 Pro Max, so I think it is best to stick to that. I might be better to add extra functionality to DeviceKit to check how much zoom the device's telephoto camera is:

  • iPhone 11 Pro: 2x
  • iPhone 12 Pro (Max): 2x
  • iPhone 13 Pro (Max): 3x
  • iPhone 14 Pro (Max): 3x
  • iPhone 15 Pro: 3x
  • iPhone 15 Pro Max: 5x

But that is for another time, first need to add support for the devices. I cherrypicked your PR into a release PR, I will change it back to telephoto for now over there. #362

@Zandor300 Zandor300 mentioned this pull request Sep 17, 2023
@Zandor300
Copy link
Member

Cherrypicked this PR into #362

@Zandor300
Copy link
Member

Merged to master through #362

@Zandor300 Zandor300 closed this Sep 21, 2023
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

4 participants