Skip to content

fix: render viewer snapshots for rotated iOS outlines#50

Open
thickfive wants to merge 1 commit into
lycorp-jp:mainfrom
thickfive:fix/viewer-rotated-ios-outline
Open

fix: render viewer snapshots for rotated iOS outlines#50
thickfive wants to merge 1 commit into
lycorp-jp:mainfrom
thickfive:fix/viewer-rotated-ios-outline

Conversation

@thickfive

@thickfive thickfive commented Jul 16, 2026

Copy link
Copy Markdown

Summary

  • Allow the Viewer API to parse rotated iOS outline headers such as App: SampleApp 874x402 (landscape-right).
  • Keeps screen populated so the viewer canvas renders instead of showing No snapshot yet.
  • Leaves the original outline text unchanged, so the orientation suffix remains visible in the Viewer outline.

Changes

  • Updated Sources/SimUse/Viewer/ViewerAPIHandlers.swift
    • Adjusted parseAppLine to accept an optional orientation suffix after the WxH screen size.
    • Kept the parsed appLabel limited to the app name instead of including the suffix.
  • Updated Tests/ViewerAPIHandlersTests.swift
    • Added a regression test covering an outline header like App: SampleApp 874x402 (landscape-right).
    • Verified the snapshot response still includes screen.width, screen.height, and a plain screen.appLabel.
  • Updated CHANGELOG.md
    • Added an Unreleased fix note for rotated iOS Simulator viewer snapshots.

Testing

  • swift test --filter ViewerAPIHandlersTests
  • make build
  • Manual viewer check with a rotated iOS Simulator window

Signed-off-by: thickfive <thickfive@live.com>
@thickfive thickfive force-pushed the fix/viewer-rotated-ios-outline branch from 870ffef to 4e76b2d Compare July 16, 2026 02:20
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