Page MenuHomePhabricator

[web] clean up thread picker modal
ClosedPublic

Authored by ginsu on Dec 12 2023, 1:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 2:05 AM
Unknown Object (File)
Sun, Nov 10, 3:01 AM
Unknown Object (File)
Sat, Nov 9, 5:12 PM
Unknown Object (File)
Sat, Nov 9, 3:15 AM
Unknown Object (File)
Fri, Nov 8, 10:38 AM
Unknown Object (File)
Thu, Oct 31, 9:36 AM
Unknown Object (File)
Thu, Oct 31, 9:36 AM
Unknown Object (File)
Thu, Oct 31, 9:36 AM
Subscribers

Details

Summary

This diff cleans up the one off styles in the thread picker 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 D10320

Test Plan

Please see the screenshots below

default:

Screenshot 2023-12-12 at 4.40.30 PM.png (1×3 px, 820 KB)

hover:

Screenshot 2023-12-12 at 4.40.20 PM.png (1×3 px, 821 KB)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu requested review of this revision.Dec 12 2023, 2:37 PM
This revision is now accepted and ready to land.Dec 14 2023, 9:55 AM
This revision was landed with ongoing or failed builds.Feb 15 2024, 12:44 AM
This revision was automatically updated to reflect the committed changes.