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

fix(iOS): fix rendering `RCTRedBoxExtraData`

Open krozniata opened this issue 1 year ago • 2 comments
trafficstars

Summary:

This PR fixes rendering of RCTRedBoxExtraData

I noticed that it wasn't displaying the reload and dismiss buttons, which made it impossible to close modal and to reload JS on e.g. visionOS (on iOS it could only be closed by swiping).

PR adds these buttons back and also introduces some refactoring

Before & After:

pr-img

Changelog:

[IOS] [FIXED] - Fix rendering RCTRedBoxExtraData

Test Plan:

Make sure that RCTRedBoxExtraData displays and works as expected

krozniata avatar Feb 19 '24 20:02 krozniata

Could you clarify what broke and how this fixes it?

lunaleaps avatar Feb 20 '24 22:02 lunaleaps

@lunaleaps Sure! Before, when user opened the "Extra data" section in RedBox it was impossible to reload JS from it, because due to a bug in the layout, buttons were not visible and the modal could only be dismissed by dragging it down (which wasn't possible to do in visionOS, so it couldn't be dismissed at all).

This PR brings back the missing buttons by fixing the RCTRedBoxExtraData layout, as they were already added before, they just weren't visible.

krozniata avatar Feb 20 '24 22:02 krozniata

@lunaleaps Is there anything else that's needs to be done to get this PR merged?

krozniata avatar Mar 05 '24 21:03 krozniata

@krozniata could you show me when this appears with an example? like the bug

Biki-das avatar Jun 17 '24 13:06 Biki-das

@cipolleschi 👀

krozniata avatar Jun 24 '24 13:06 krozniata

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Jun 27 '24 11:06 facebook-github-bot

@cipolleschi merged this pull request in facebook/react-native@c9d589dab5148cbd192078967f743d38b8048c6e.

facebook-github-bot avatar Jun 28 '24 13:06 facebook-github-bot

This pull request was successfully merged by @krozniata in c9d589dab5148cbd192078967f743d38b8048c6e.

When will my fix make it into a release? | How to file a pick request?

github-actions[bot] avatar Jun 28 '24 13:06 github-actions[bot]