Page MenuHomePhabricator

[native] Position RegistrationButton at the bottom of KeyserverSelection
ClosedPublic

Authored by ashoat on Apr 24 2023, 5:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 2, 3:50 PM
Unknown Object (File)
Tue, Aug 27, 7:42 AM
Unknown Object (File)
Tue, Aug 27, 12:57 AM
Unknown Object (File)
Mon, Aug 26, 3:18 AM
Unknown Object (File)
Fri, Aug 16, 11:23 PM
Unknown Object (File)
Wed, Aug 7, 8:45 PM
Unknown Object (File)
Aug 4 2024, 3:21 PM
Unknown Object (File)
Jul 29 2024, 7:41 AM
Subscribers

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

As of my most recent updates to D7593, the RegistrationContainer now handles scrolling, so we now need to wrap it with a container View.

Two opinionated things I did here:

  1. I opted to leave the padding in RegistrationContainer inside of moving it to the parent View, since I figured it would be good to have that extra vertical padding inside the ScrollView. I could separate the vertical padding and horizontal padding if desired, with the horizontal padding applied by the parent component but the vertical padding applied by the RegistrationContainer
  2. Decided to leave the RegistrationContainer changes here, even though they're no longer used. Figured it might be used in other cases. Can remove it (or separate into another diff) if desired
This revision is now accepted and ready to land.Apr 26 2023, 1:14 PM