ReviewOS

also looking at this

craft-native/craft

fix(mobile): preserve native editing, navigation, and durable media requests

#369
Merged chris wants to merge codex/native-vertical-scroll-tab-swipes into main
36 files +864 -185

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

packages/ios/fixtures/native-screen/NativeRenderUnitTests.swiftmodified+2-2
Changes to packages/ios/fixtures/native-screen/NativeRenderUnitTests.swift
@@ -1960,7 +1960,7 @@ final class NativeRenderUnitTests: XCTestCase {
19601960 XCTAssertEqual(controller.evaluateScript("__rowPress")?.toString(), "select-b")
19611961 }
19621962
1963 func testScrollCanCancelButtonTrackingWithoutTakingSliderOrInputTracking() throws {
1963 func testScrollCanCancelButtonAndFieldTrackingWithoutTakingValueControls() throws {
19641964 let controller = CraftNativeScreenController(config: CraftConfig())
19651965 controller.loadViewIfNeeded()
19661966 controller.render(document([[
@@ -1975,7 +1975,7 @@ final class NativeRenderUnitTests: XCTestCase {
19751975 XCTAssertTrue(form.touchesShouldCancel(in: try XCTUnwrap(find(UIButton.self, key: "action", below: controller.view))))
19761976 XCTAssertFalse(form.touchesShouldCancel(in: try XCTUnwrap(find(UISlider.self, key: "position", below: controller.view))))
19771977 XCTAssertFalse(form.touchesShouldCancel(in: try XCTUnwrap(find(UISwitch.self, key: "toggle", below: controller.view))))
1978 XCTAssertFalse(form.touchesShouldCancel(in: try XCTUnwrap(find(UITextField.self, key: "draft", below: controller.view))))
1978 XCTAssertTrue(form.touchesShouldCancel(in: try XCTUnwrap(find(UITextField.self, key: "draft", below: controller.view))))
19791979 }
19801980
19811981 func testScrollKeyboardTapPreservesTextAreaFocusAndSelectionButDismissesBackground() throws {