Skip to content

Correct swapped iPad screen ratios, names, and support links - #517

Open
mvanhorn wants to merge 1 commit into
devicekit:masterfrom
mvanhorn:fix/457-ipad-device-data
Open

mvanhorn wants to merge 1 commit into
devicekit:masterfrom
mvanhorn:fix/457-ipad-device-data

Conversation

@mvanhorn

Copy link
Copy Markdown

Device.swift.gyb and the generated switches now return (512, 683) for the 13-inch Air models, (41, 59) for iPad Air 11-inch M3 and M4, (417, 605) for iPad Pro 11-inch M4 and M5, and (3, 4) for iPad Pro 13-inch M4 and M5. Callers of screenRatio on the 13-inch iPad Air (M2, M3, and M4), the 11-inch iPad Air (M3 and M4), and the 11-inch and 13-inch iPad Pro (M4 and M5) receive a pair whose first component is the larger one, so the value describes a landscape screen.

The DeviceKit module compiles, including the updated generated device table.

Refs #457

Device.swift.gyb and the generated switches now return (512, 683) for
the 13-inch Air models, (41, 59) for iPad Air 11-inch M3 and M4, (417,
605) for iPad Pro 11-inch M4 and M5, and (3, 4) for iPad Pro 13-inch M4
and M5. description and safeDescription for the M2 through M4 Air models
and the M4 and M5 Pro models use the unparenthesized size, such as "iPad
Air 11-inch (M2)" and "iPad Pro 13-inch (M5)", and those five support
links drop /en-us/. The unreleased changelog notes the correction, and
the tests assert each portrait ratio on the device and its simulator
along with the new descriptions. Fixes devicekit#457.

Refs devicekit#457
@mvanhorn mvanhorn mentioned this pull request Oct 13, 2026
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.

1 participant