react-native icon indicating copy to clipboard operation
react-native copied to clipboard

Fixes rendering smudges of elements with border for iOS New Architecture

Open jankosecki opened this issue 4 months ago • 2 comments

Summary:

0.73.0 introduced smudges appearing while rendering screen with elements, which use borderWidth styles as in https://github.com/facebook/react-native/issues/42604.

https://github.com/facebook/react-native/pull/38234 removed a feature flag disableTransactionCommit, which by default was false, meaning that prior to this RCTMountingManager.mm was using CATransaction for rendering (unless feature flag react_fabric:disable_transaction_commit was applied).

This change brings back CATransaction into RCTMountingManager.mm

Changelog:

[IOS] [FIXED] - Fix rendering issues of elements with borderWidth (issue 42604)

Test Plan:

Download reproduction repository from linked issue and apply changes to the local version of react-native. Using Reload button or switching screen should not show any more smudges when rendering elements with borderWidth

jankosecki avatar Feb 21 '24 19:02 jankosecki

CC @sammy-SC

mdvacca avatar Feb 21 '24 21:02 mdvacca

This should be resolved by https://github.com/facebook/react-native/commit/5fbdc994f945ee8d834632649e3b82451660e8fe

let me know if you are still seeing an issue.

We don't want to bring bar CATransaction because it has a performance cost associated with it.

sammy-SC avatar Feb 21 '24 21:02 sammy-SC

Amazing, thank you @sammy-SC .

I can confirm it resolves the unwanted behaviour visible in the reproduction repository of the linked issue.

jankosecki avatar Feb 26 '24 11:02 jankosecki

thanks for confirming @jankosecki and sorry for breaking this in the first place.

sammy-SC avatar Feb 26 '24 11:02 sammy-SC