Page MenuHomePhabricator

[native] introduce onClosed callback prop to BottomSheet component

Authored by ginsu on Thu, Sep 7, 9:18 AM.
Referenced Files
Unknown Object (File)
Wed, Sep 20, 2:24 AM
Unknown Object (File)
Tue, Sep 12, 4:49 AM
F746496: Screen Recording 2023-09-07 at 12.24.34
Thu, Sep 7, 9:25 AM



In subsequent diffs we will introduce some logic where we want to navigate back from the modal bottom sheet screen back to the screen we originally were before we opened/navigated to the modal bottom sheet screen when the bottomsheet closes. Unfortunately, the bottom sheet library does not have a callback prop for this. However, it does have a callback prop called onChange that gets triggered every time the sheet position changes. We can use this and set a condition where if the sheet position index is -1 (the sheet is closed) then we call the onClosed callback prop

Test Plan

Passed in a callback which logged a message to the console as the onClosed prop and confirmed that the message appeared when the bottomsheet closed

Diff Detail

rCOMM Comm
Lint Not Applicable
Tests Not Applicable