Page MenuHomePhabricator

[native] Measure whole KeyserverSelectionBottomSheet
ClosedPublic

Authored by ashoat on Nov 4 2024, 1:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 9:10 PM
Unknown Object (File)
Sun, Dec 15, 1:40 AM
Unknown Object (File)
Tue, Dec 3, 6:51 PM
Unknown Object (File)
Sun, Dec 1, 8:28 AM
Unknown Object (File)
Fri, Nov 29, 11:31 PM
Unknown Object (File)
Fri, Nov 29, 9:02 PM
Unknown Object (File)
Nov 20 2024, 3:21 PM
Unknown Object (File)
Nov 20 2024, 1:13 AM
Subscribers

Details

Summary

Instead of measuring just part of this, and adding up some hardcoded constants to the measured part, we can simply measure the whole thing.

In combination with the parent diff, this allows us to remove the hardcoded constants entirely, and rely purely on the measurement.

Depends on D13867

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