Skip to content

SafeAreaSpacerView - fix race condition and a small refactor #3770

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 29, 2025

Conversation

M-i-k-e-l
Copy link
Collaborator

Description

SafeAreaSpacerView - fix race condition and a small refactor

Changelog

None

Additional info

None

@Inbal-Tish Inbal-Tish merged commit 8f4d248 into rn77-new-arch Jun 29, 2025
1 check passed
@Inbal-Tish Inbal-Tish deleted the rn77-new-arch-fix-safe-area-spacer-view branch June 29, 2025 12:40
M-i-k-e-l added a commit that referenced this pull request Jul 1, 2025
* Upgrade to react native 0.76.9

Some errors in native code that I did not want to commit fixes just yet
docuilib has issues (hopefully will be solved in 0.77.2

* RN77 upgrade + new arch

Android runs (still need to fix native)  |  iOS red screen  |  docuilib still has errors

* Remove TextInputDelKeyHandlerPackage (Android)

* KeyboardAccessoryView on Android - fix crash (one bug left)

Pressing on dismiss (x) only fully works the second time

* Pressing on dismiss (x) only fully works the second time - a very complex fix

* Remove unused code

* HighlighterViewManager - fix (use new API)

* Fix gesture handler changed style

* Fix images (unrelated)

* Fix PanViewScreen (unrelated)

* Fix dragging in SortableList and SortableGridList

* Update navigation

* Change iOS back to Objective-C

* Fix screen - border color

* Revert to older reanimated and gesture-handler versions

* Drawer - fix animation flickering

* Remove comment

* StackAggregator - several fixes - see #3210

* Drawer - fix color not being shown with multiple buttons (RectButton bug)

* Fix

* Downgrading reanimated and gesture handler

* Fix

* Fix iOS (setimmediate)

* remove UIManager.setLayoutAnimationEnabledExperimental

* Update RNN

* Fix snapshot

* Fix TS

* Fix TS

* Fix snapshot

* Native lib reorder and fix ios (#3750)

* Native lib - reorganize components in folders for convenience + bump version to 5.0.0

* pointing to index

* revert naming change

* fix api files

* rename HighlighterOverlay web

* Moving sub components to the parent

* fix api file and move KeyboardUtils outside folder

* Revert "fix api file and move KeyboardUtils outside folder"

This reverts commit ff64327.

* fix api json

* fix folder name to match component name

* remove specs

* fix highligther view native registration

* fix for SafeAreaSpacerView

* fix types

* fix for KeyboardTrackingView

* trying to fix failed CI build

* SafeAreaSpacesView - moving to react implementation using native SafeAreaManager

* manager changes

* fix safe area

* fix SafeAreaSpacerView again

* removing logs

* Fix highlightViewTag type

* Fix TS error

* Typescript fixes \ revert

* Revert

---------

Co-authored-by: Miki Leib <[email protected]>
Co-authored-by: M-i-k-e-l <[email protected]>

* KeyboardAwareBase - fix error

* Fix menu - "native" tag should show all (and only) native components

* Update uilib-native to snapshot

* Fix tests

* Update uilib-native to snapshot

* Fix TS error and web type

* Change from codegenNativeComponent to requireNativeComponent in order to fix error after transpilation (#3768)

* Change from codegenNativeComponent to requireNativeComponent

* Update snapshot

* Revert, move to specs and publish the typescript vesion (#3769)

* Another try

* Do not transpile specs

* Another one

* Update snapshot

* SafeAreaSpacerView - fix race condition and a small refactor (#3770)

* SafeAreaSpacerView - fix race condition and a small refactor

* Update snapshot

* Remove extra backgroundColor

---------

Co-authored-by: Inbal Tish <[email protected]>
Co-authored-by: Inbal Tish <[email protected]>
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.

2 participants