Skip to content

fix(capture): 修复预览宿主视图布局递归#45

Merged
iamsyc merged 1 commit intomainfrom
codex/fix-preview-layout-recursion
Apr 5, 2026
Merged

fix(capture): 修复预览宿主视图布局递归#45
iamsyc merged 1 commit intomainfrom
codex/fix-preview-layout-recursion

Conversation

@iamsyc
Copy link
Copy Markdown
Owner

@iamsyc iamsyc commented Apr 5, 2026

Summary

  • remove preview host view layout-time layer resizing to avoid recursive layout warnings
  • rely on layer autoresizing and keep scale sync on window and backing changes
  • add a focused test to ensure the host view does not duplicate the hosted preview layer

Verification

  • xcodebuild -project VoidDisplay.xcodeproj -scheme VoidDisplay -destination 'platform=macOS,arch=arm64' -derivedDataPath /Users/syc/Project/VoidDisplay/.ai-tmp/layout-recursion/DerivedData build
  • xcodebuild -project VoidDisplay.xcodeproj -scheme VoidDisplay -destination 'platform=macOS,arch=arm64' -derivedDataPath /Users/syc/Project/VoidDisplay/.ai-tmp/layout-recursion/DerivedData test -only-testing:VoidDisplayTests/ZeroCopyPreviewRendererTests

- 移除 layout 中的图层同步,改用自动尺寸调整
- 避免重复挂载同一个预览图层
- 补充宿主视图图层挂载测试
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

CI Summary

Run: Open

Check Status Details
Change Scope code code-relevant change set
Unit Tests success required
UI Smoke Tests (matrix: baseline + permissionDenied + rebuildFailed) success required; matrix: baseline + permissionDenied + rebuildFailed
Release Build Check (Release arch smoke: arm64 + x86_64) success required; Release arch smoke check (arm64 + x86_64 with thin/sign/verify)
CI Gate success required; driven by Unit Tests + UI Smoke Tests + Release Build Check

Gate rule: target main 的代码相关 PR 必须通过 Unit Tests + UI Smoke Tests + Release Build Check

@iamsyc iamsyc merged commit f4e50c8 into main Apr 5, 2026
9 checks passed
@iamsyc iamsyc deleted the codex/fix-preview-layout-recursion branch April 5, 2026 08:22
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