Page MenuHomePhabricator

[native] Consider BottomSheetHandle size when setting snapPoints
ClosedPublic

Authored by ashoat on Nov 4 2024, 1:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 31, 3:55 PM
Unknown Object (File)
Tue, Dec 31, 3:55 PM
Unknown Object (File)
Tue, Dec 31, 3:54 PM
Unknown Object (File)
Tue, Dec 31, 3:54 PM
Unknown Object (File)
Sun, Dec 29, 3:22 AM
Unknown Object (File)
Fri, Dec 27, 2:50 AM
Unknown Object (File)
Thu, Dec 26, 9:30 PM
Unknown Object (File)
Thu, Dec 26, 10:27 AM
Subscribers

Details

Summary

The height of BottomSheetHandle is always the same, and we should take it into account when setting snapPoints. More context here.

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