Page MenuHomePhabricator

[native] Get rid of bottomSheetPaddingTop in ConnectFarcasterBottomSheet
ClosedPublic

Authored by ashoat on Nov 4 2024, 1:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 12:23 PM
Unknown Object (File)
Sun, Dec 15, 9:12 AM
Unknown Object (File)
Sat, Nov 30, 11:53 PM
Unknown Object (File)
Nov 23 2024, 3:28 PM
Unknown Object (File)
Nov 21 2024, 12:10 PM
Unknown Object (File)
Nov 20 2024, 10:55 PM
Unknown Object (File)
Nov 20 2024, 3:36 AM
Unknown Object (File)
Nov 19 2024, 7:53 PM
Subscribers

Details

Summary

This padding was hardcoded by Ginsu to add additional space at the bottom of the BottomSheet. It is obviated by my work to consider BottomSheetHandle when setting snapPoints.

The padding was extra large because Ginsu measured the height of the button incorrectly. Once I adjusted the height of button, the padding is approximately the same as what it was before.

Depends on D13870

Test Plan

In combination with the rest of the stack, I tested each individual use of our BottomSheet component to make sure that the bottom padding was reasonable. In most cases it was unchanged; in other cases, it was changed to be more consistent with the other BottomSheets in our codebase. In all cases, there was enough padding for the "home pill" at the bottom of the screen

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable