HomePhabricator
Diffusion Comm 22479ac32638

[native] Replace `scrollPos` instance variable w/ ref

Tags
None
Referenced Files
F753668: ff537d.png
Sep 13 2023, 8:23 AM
File Not Attached
Subscribers
None

Description

[native] Replace scrollPos instance variable w/ ref

Summary:
As part of converting ChatThreadList to a functional component we'll need to replace the scrollPos instance variable with a ref.


Depends on D9135

Test Plan:
Things seem to continue to work as expected. Haven't used useRef a whole lot, so would be good to get feedback from reviewers w/ more familiarity.

Logging values seems to result in correct values:

ff537d.png (562×1 px, 380 KB)

Reviewers: ashoat, ginsu, rohan, tomek

Reviewed By: ashoat

Differential Revision: https://phab.comm.dev/D9136

Details

Provenance
atulAuthored on Sep 11 2023, 3:46 PM
Reviewer
ashoat
Differential Revision
D9136: [native] Replace `scrollPos` instance variable w/ ref
Parents
rCOMM2ce8245fa6e2: [native] Pull static fns out of `ChatThreadList` to `ChatThreadListUtils`
Branches
Unknown
Tags
Unknown