Page MenuHomePhabricator

[native] Show `ActivityIndicator` in `EditUserAvatar` during update
ClosedPublic

Authored by atul on Apr 19 2023, 11:46 AM.
Tags
None
Referenced Files
F3150510: D7529.diff
Mon, Nov 4, 11:24 PM
Unknown Object (File)
Fri, Nov 1, 9:13 AM
Unknown Object (File)
Fri, Nov 1, 9:13 AM
Unknown Object (File)
Fri, Nov 1, 9:12 AM
Unknown Object (File)
Fri, Nov 1, 9:09 AM
Unknown Object (File)
Mon, Oct 28, 8:26 PM
Unknown Object (File)
Oct 2 2024, 2:36 PM
Unknown Object (File)
Oct 1 2024, 12:31 PM
Subscribers

Details

Summary

We consume isGalleryAvatarUpdateLoading and isRemoveAvatarUpdateLoading introduced in previous diffs to display ActivityIndicator (ie spinner) when an update is in progress.

Note: Yeah there are obvious things that can be factored out. I want to get more of the error handling stuff done first and then I'll go through and consolidate things.


Depends on D7528

Test Plan

Works as expected:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul published this revision for review.Apr 19 2023, 11:47 AM
atul edited the test plan for this revision. (Show Details)
atul added inline comments.
native/avatars/edit-user-avatar.react.js
50–52 ↗(On Diff #25383)

Stole this from LoadableImage

65–75 ↗(On Diff #25383)

I just stole this from LoadableImage

This revision is now accepted and ready to land.Apr 19 2023, 11:59 AM