Page MenuHomePhabricator

[native] Extract useKeyboardHeight from LoggedOutModal
ClosedPublic

Authored by ashoat on May 24 2024, 1:32 AM.
Tags
None
Referenced Files
F2111622: D12207.diff
Tue, Jun 25, 11:33 PM
Unknown Object (File)
Sun, Jun 23, 12:55 PM
Unknown Object (File)
Sun, Jun 23, 6:08 AM
Unknown Object (File)
Sat, Jun 22, 12:19 AM
Unknown Object (File)
Sat, Jun 22, 12:06 AM
Unknown Object (File)
Fri, Jun 21, 11:06 AM
Unknown Object (File)
Thu, Jun 20, 8:59 PM
Unknown Object (File)
Sun, Jun 16, 5:42 PM
Subscribers

Details

Summary

We'll want to use this (with slight modifications) in the next diff, where we convert LoggedOutModal to Reanimated 2 syntax.

Depends on D12198

Test Plan

I tested this diff stack by playing around with LoggedOutModal on my iOS simulator in my local environment:

  1. I navigated to LogInPanel and back
  2. I navigated to LegacyRegisterPanel and back
  3. I navigated to FullscreenSIWEPanel and back
  4. I navigated to QRCodeSignInNavigator and back
  5. I navigated to Registration and back

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

native/account/logged-out-modal.react.js
311 ↗(On Diff #40609)

I converted this from a ref to a state. This wasn't strictly necessary, since it's only used for a "spot check". I could have instead passing the ref in directly, but not its value (passing the value in directly would mean that keyboardHeightInput might have outdated values).

I decided against passing the ref in directly since it just seemed a bit messier, but I can do that if preferred.

This revision is now accepted and ready to land.May 24 2024, 7:43 AM