Page MenuHomePhabricator
Differential D8279

[web] Style and disable "Save Avatar" button when `userAvatarSaveInProgress`
ClosedPublic
Actions

Authored by atul on Jun 21 2023, 12:52 PM.
Tags
None
Referenced Files
F3409226: D8279.id27977.diff
Wed, Dec 4, 3:20 PM
F3409176: D8279.id28092.diff
Wed, Dec 4, 3:11 PM
F3409067: D8279.id27979.diff
Wed, Dec 4, 2:51 PM
F3407563: D8279.diff
Wed, Dec 4, 6:43 AM
Unknown Object (File)
Thu, Nov 28, 9:25 AM
Unknown Object (File)
Sat, Nov 23, 1:47 PM
Unknown Object (File)
Sat, Nov 23, 1:11 PM
Unknown Object (File)
Sat, Nov 23, 1:11 PM
View All Files
Subscribers
tomek

Details

Reviewers
ashoat
ginsu
rohan
inka
michal
kamil
Commits
rCOMM370a9b620d5d: [web] Style and disable "Save Avatar" button when `userAvatarSaveInProgress`
Summary
  1. Show spinner when save avatar request is in progress.
  2. Disable button when save avatar request is in progress.
  3. Sneak in disable button change for CommunityCreationModal

Depends on D8278

Test Plan

Works as expected:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Jun 21 2023, 12:52 PM
Herald added a subscriber: tomek. · View Herald TranscriptJun 21 2023, 12:52 PM
atul edited the test plan for this revision. (Show Details)Jun 21 2023, 12:53 PM
atul added inline comments.Jun 21 2023, 12:55 PM
web/sidebar/community-creation/community-creation-modal.react.js
196 ↗(On Diff #27977)

Sorry snuck this quick change in here to keep things symmetrical

atul published this revision for review.Jun 21 2023, 12:57 PM
rohan added inline comments.Jun 21 2023, 1:04 PM
web/avatars/emoji-avatar-selection-modal.react.js
106–107 ↗(On Diff #27977)

The video looks good, just wondering if saveButtonContent meant to be inside of the div?

Harbormaster completed remote builds in B20405: Diff 27977.Jun 21 2023, 1:11 PM
atul added inline comments.Jun 21 2023, 2:09 PM
web/avatars/emoji-avatar-selection-modal.react.js
106–107 ↗(On Diff #27977)

Yeah, fixed this in next diff. I'll rebase and fix it here instead

atul added a child revision: D8280: [web] Display `errorMessage` in `saveAvatarButton` on failure.Jun 21 2023, 2:09 PM
atul updated this revision to Diff 27979.Jun 21 2023, 2:12 PM

address @rohan's feedback

Harbormaster completed remote builds in B20407: Diff 27979.Jun 21 2023, 2:31 PM
rohan accepted this revision.Jun 22 2023, 7:25 AM
This revision is now accepted and ready to land.Jun 22 2023, 7:25 AM
atul updated this revision to Diff 28077.Jun 23 2023, 1:53 PM

rebase and land

This revision was landed with ongoing or failed builds.Jun 23 2023, 2:07 PM
Closed by commit rCOMM370a9b620d5d: [web] Style and disable "Save Avatar" button when `userAvatarSaveInProgress` (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM370a9b620d5d: [web] Style and disable "Save Avatar" button when `userAvatarSaveInProgress`.
Harbormaster completed remote builds in B20491: Diff 28077.Jun 23 2023, 2:15 PM

Revision Contents
Changeset List

PathSize
web/
avatars/
7 lines
15 lines
sidebar/
community-creation/
1 line

Diff 28092

View Options

web/avatars/emoji-avatar-selection-modal.css

Loading...
View Options

web/avatars/emoji-avatar-selection-modal.react.js

Loading...
View Options

web/sidebar/community-creation/community-creation-modal.react.js

Loading...