Page MenuHomePhabricator

[native] Add SafeAreaView to RegistrationContainer
ClosedPublic

Authored by ashoat on Apr 24 2023, 5:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 2, 3:50 PM
Unknown Object (File)
Sat, Aug 31, 6:14 AM
Unknown Object (File)
Wed, Aug 28, 11:36 AM
Unknown Object (File)
Tue, Aug 27, 7:25 PM
Unknown Object (File)
Mon, Aug 26, 11:38 PM
Unknown Object (File)
Sun, Aug 25, 1:29 PM
Unknown Object (File)
Jul 29 2024, 7:41 AM
Unknown Object (File)
Jul 29 2024, 7:41 AM
Subscribers

Details

Summary

See designs

Depends on D7598

Test Plan
beforeafter
ios
Screenshot 2023-04-24 at 8.35.43 PM.png (1×1 px, 793 KB)
Screenshot 2023-04-24 at 8.36.40 PM.png (1×1 px, 793 KB)
android
Screenshot 2023-04-24 at 8.41.29 PM.png (2×1 px, 519 KB)
Screenshot 2023-04-24 at 8.41.12 PM.png (2×1 px, 519 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I was seeing an issue where the SafeAreaView was applying the bottom inset when the KeyboardAvoidingView was up. To address this, I moved the SafeAreaView upstream to RegistrationNavigator, and moved KeyboardAvoidingView downstream to RegistrationContainer

This revision is now accepted and ready to land.Apr 26 2023, 1:14 PM