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-navigation/Home.stxmodified+1-1
Changes to packages/ios/fixtures/native-navigation/Home.stx
@@ -834,7 +834,7 @@ function openDetailsThenUpdateBackground() {
834834 <Text testID="invalid-control-style-status" :text="invalidControlStyleStatus" />
835835 <Text testID="date-event-status" :text="dateEventStatus" />
836836 <Button testID="toggle-modal" @click="toggleModal()">Toggle modal</Button>
837 <Modal testID="native-modal" visible={modalVisible} transparent animationType="slide" style={{"opacity": modalOpacity}} @show="modalShown()" @dismiss="modalDismissed()" @request-close="modalRequested()">
837 <Modal testID="native-modal" visible={modalVisible} transparent animationType="slide" style={{"opacity": modalOpacity}} @show="modalShown()" @dismiss="modalDismissed()" @request-close="modalRequested($event)">
838838 <Text testID="modal-content">Modal content</Text>
839839 <Button testID="toggle-modal-opacity" @click="toggleModalOpacity()">Toggle modal opacity</Button>
840840 <Text testID="modal-opacity" :text="modalOpacity" />
scripts/native-renderer-contract.test.tsmodified+1-1
Changes to scripts/native-renderer-contract.test.ts
@@ -1309,7 +1309,7 @@ describe('native renderer component contract', () => {
13091309 expect(iosNavigationTest).toContain('itemsPicker.waitForExistence(timeout: 5)')
13101310 expect(androidNavigationTest).toContain('items picker should be an Android Spinner')
13111311 expect(androidNavigationTest).toContain('awaitText(activity, "items-picker-value", "2.0")')
1312 expect(navigationHome).toContain('@request-close="modalRequested()"')
1312 expect(navigationHome).toContain('@request-close="modalRequested($event)"')
13131313 expect(navigationHome).toContain('@show="modalShown()"')
13141314 expect(navigationHome).toContain('testID="clear-image-source"')
13151315 expect(navigationHome).toContain('testID="image-error-count"')