Page MenuHomePhabricator

[web] cleanup toggle pin modal
ClosedPublic

Authored by ginsu on Dec 8 2023, 11:55 AM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 8 2024, 1:47 PM
Unknown Object (File)
Apr 6 2024, 12:20 AM
Unknown Object (File)
Apr 2 2024, 7:06 AM
Unknown Object (File)
Mar 31 2024, 7:08 AM
Unknown Object (File)
Mar 28 2024, 4:39 PM
Unknown Object (File)
Mar 27 2024, 10:31 PM
Unknown Object (File)
Mar 6 2024, 11:09 PM
Unknown Object (File)
Mar 6 2024, 11:07 PM
Subscribers

Details

Summary

This diff cleans up the one off styles in the toggle pin 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 D10267

Test Plan

Please see screenshot below

Screenshot 2023-12-08 at 2.37.23 PM.png (1×3 px, 961 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: atul, rohan, kamil.
Harbormaster returned this revision to the author for changes because remote builds failed.Dec 8 2023, 12:35 PM
Harbormaster failed remote builds in B24912: Diff 34446!
ginsu requested review of this revision.Dec 8 2023, 12:41 PM

will make sure ci passes before landing

The new modal looks nice!

This revision is now accepted and ready to land.Dec 10 2023, 8:17 AM
This revision was automatically updated to reflect the committed changes.