Differential D10317
[web] cleanup account delete modal Authored by ginsu on Dec 12 2023, 12:49 PM. Tags None Referenced Files
Details
This diff cleans up the one off styles in the account delete modal and replaces it with styles of the redesigned modal. Linear task: https://linear.app/comm/issue/ENG-5943/extendmodify-the-modal-props-api-to-follow-new-modal-designs Depends on D10316 Please see the screenshots below  error case: 
Diff Detail
Event TimelineComment Actions Looks good - just one comment inline. Also just a personal opinion on the design of the modal (I know you didn't introduce the change in this diff so no changes are needed here), but I think the little ! circle icon would look better inline with the text, either in its own column side-by-side to the text, or just next to the first line of text. Not a big deal though, just wanted to call it out
Comment Actions
I think side by side would look better too, I can make that change if we are all aligned on it.
| ||||||||||||||||||||||||||||||||||||||||||||||