Skip to content

Conversation

@SjaakSchilperoort
Copy link
Member

@SjaakSchilperoort
Copy link
Member Author

Zo lijkt ie wel goed te werken, op zowel Android als iOS

@jonathan-waarneming-nl
Copy link
Collaborator

Ik zie dit bij iOS:
Screenshot 2025-08-08 at 12 48 22

@SjaakSchilperoort
Copy link
Member Author

Zo beter?

@SjaakSchilperoort
Copy link
Member Author

Zo beter?

Kennelijk niet. ObsIdentify op iOS is in orde, maar op Android krijg ik dit. Het valt niet mee ...

Screenshot_1754655365

@SjaakSchilperoort
Copy link
Member Author

SjaakSchilperoort commented Aug 8, 2025

Wat hier kennelijk aan de hand is (mbv chatgpt):

The Lightbox is not a React Navigation modal — it's a raw Modal from React Native (wrapped inside react-native-image-viewing). This changes the safe-area behavior compared to navigation-managed modals:

Platform Who applies safe area?
iOS No automatic insets in full-screen custom modals → you must add them yourself
Android (API 30+ edge-to-edge) The library internally uses SafeAreaView or system insets → safe area is already applied

De huidige implementatie gebruikt SafeAreaView van React Native, en die doet precies wat nodig is: insets op iOS en niets op Android. Als we SafeAreaView/useSafeAreaInsets van react-native-safe-area-context gaan gebruiken, moeten we die enkel toepassen op iOS. Dat zou mijn case fixen.

Maar van de mix van drie repo's, twee platforms en twee safe-area views word ik behoorlijk confuus. Laten we eerst maar de updates de deur uit doen, dan kijk ik hier op een rustig moment nog eens naar.

@SjaakSchilperoort SjaakSchilperoort marked this pull request as draft August 8, 2025 13:18
@jonathan-waarneming-nl
Copy link
Collaborator

Maar van de mix van drie repo's, twee platforms en twee safe-area views word ik behoorlijk confuus. Laten we eerst maar de updates de deur uit doen, dan kijk ik hier op een rustig moment nog eens naar.

Groot gelijk 😅

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.

3 participants