Skip to content

fix(WhereUI): restore Settings search and Duo accessibility captures - #466

Closed
kyleve wants to merge 17 commits into
codex/iphone-duo-adoptionfrom
codex/duo-capture-followup
Closed

kyleve wants to merge 17 commits into
codex/iphone-duo-adoptionfrom
codex/duo-capture-followup

Conversation

@kyleve

@kyleve kyleve commented Oct 31, 2026 •

Copy link
Copy Markdown
Owner

Posted by an AI agent on kve's behalf.

Superseded

This follow-up is closed without merging. Its implementation and reviewed references now live in the layers that need them:

  • #377 owns accessibility capture hosting, native-bar metadata recovery, measurement fixes, and the standalone capture baseline.
  • #372 owns native Settings search placement, its fold-layout regression, and the complete adaptive integration references.

The active three-PR stack is #377 → #378 → #372. No implementation was discarded; the local follow-up tree is identical to #372 at bb29adbcf00c0186a09ab82c3482b3a30aabb933.

Validation and remaining work

Each active layer now has its own validation record. The final integration unit suite executed 2,975 tests with no unexpected failures. Adaptive lab, MainTabs, Settings, and the separately repeated Developer Overlay snapshots passed. Full per-layer CI is being rerun.

Native Duo 27.2 re-measurement and interactive release checks remain open in STU-316. Existing tall Year accessibility annotation limitations remain in STU-325. The search and capture work in STU-326 and STU-327 remains In Review with the owning PRs.

@kyleve
kyleve added this pull request to stack #379 October 31, 2026 02:36
@kyleve kyleve changed the title codex/duo capture followup fix(WhereUI): restore Settings search and Duo accessibility captures Oct 31, 2026
@kyleve kyleve added the installable Maintain a Where Dev install page for this PR label Oct 31, 2026

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Posted by an AI agent on kve's behalf.

Settings after this change: native search stays visible in the split navigation drawer. Synthetic Duo inner portrait, light appearance, revision f9a01f74da1f. The fold-transition regression and all 20 full-content Settings images pass.

Settings search visible in Duo split navigation

Full-resolution snapshot.

var overrides: [GetterOverride] = []
var methods: Set<Method> = []
for bar in bars {
guard let method = class_getInstanceMethod(type(of: bar), selector),

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Posted by an AI agent on kve's behalf.

Review focus: this test-only adapter temporarily intercepts the native accessibility getter. Results are scoped to captured bar instances, shared inherited methods are deduplicated, unrelated instances call the original getter, and every implementation is restored in defer, including thrown parses.

Window fitting lasts only while reading metadata. Image rendering and path parsing use the restored geometry. The SwiftUI split-navigation regression covers unrelated bars, unchanged paths, thrown operations, and subsequent native item updates.

@kyleve
kyleve force-pushed the codex/duo-capture-followup branch from f9a01f7 to e9236bb Compare November 9, 2026 22:30
@kyleve
kyleve force-pushed the codex/duo-capture-followup branch 2 times, most recently from 2fd6f69 to eb54197 Compare November 11, 2026 17:17
Keep all nine UIKit size-class combinations in the pane-policy regression while mapping through the two-case Broadway API. Unspecified inputs must still select a single pane.

Validation: SwiftFormat and diff checks pass. Full unit integration runs at the rebased stack tip before publication; the root Broadway suites passed 141 tests.
Review 594 images and update 592 references while preserving raw PNG metadata. Two hydrated Elsewhere references already match. Validate all 2,975 iOS unit tests, 74 snapshot tests, 28 host macro tests, 31 architecture tests, and formatting/attribution checks. Retain native Duo 27.2 remeasurement and the pre-existing tall Year annotation limitation as explicit follow-ups.
@kyleve
kyleve force-pushed the codex/duo-capture-followup branch from eb54197 to cc5e91f Compare November 11, 2026 17:39
@kyleve
kyleve removed this pull request from stack #379 November 11, 2026 18:38
@kyleve
kyleve force-pushed the codex/iphone-duo-adoption branch from 0c7e8de to bb29adb Compare November 11, 2026 18:38
@kyleve kyleve closed this Nov 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

installable Maintain a Where Dev install page for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant