HomePhabricator
Diffusion Comm 349d0eec12f8

[web] Move `editAvatarBadge` div to `EditUserAvatarMenu`

Tags
None
Referenced Files
F597727: 23d369.png
Jun 22 2023, 10:56 AM
File Not Attached
Subscribers
None

Description

[web] Move editAvatarBadge div to EditUserAvatarMenu

Summary:
Currently only the pencil icon is "clickable," we want the whole thing to be "clickable."

Known regression:

23d369.png (744×726 px, 77 KB)

The menu is appearing right below the icon rather than below the "badge." Will address this layout issue in subsequent diff.


Depends on D8261

Test Plan: "Whole button" is clickable.

Reviewers: ashoat, ginsu, rohan, inka, michal, kamil

Reviewed By: kamil

Subscribers: tomek

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

Details

Provenance
atulAuthored on Jun 20 2023, 11:52 AM
Reviewer
kamil
Differential Revision
D8263: [web] Move `editAvatarBadge` div to `EditUserAvatarMenu`
Parents
rCOMMb6e7da51ff11: [web] Display `loadingSpinner` over `EditUserAvatar` when change in progress
Branches
Unknown
Tags
Unknown