Page MenuHomePhabricator

[web] cleanup subchannels modal
ClosedPublic

Authored by ginsu on Dec 12 2023, 2:36 PM.
Tags
None
Referenced Files
F3507109: D10322.diff
Fri, Dec 20, 7:25 PM
Unknown Object (File)
Nov 17 2024, 10:52 PM
Unknown Object (File)
Nov 13 2024, 12:14 AM
Unknown Object (File)
Nov 7 2024, 7:03 AM
Unknown Object (File)
Oct 31 2024, 9:36 AM
Unknown Object (File)
Oct 31 2024, 9:36 AM
Unknown Object (File)
Oct 31 2024, 9:36 AM
Unknown Object (File)
Oct 31 2024, 9:36 AM
Subscribers

Details

Summary

This diff cleans up the one off styles in the subchannels 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 D10321

Test Plan

please see screenshot below

Screenshot 2023-12-12 at 5.37.17 PM.png (1×3 px, 843 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 12 2023, 2:39 PM
Harbormaster failed remote builds in B25009: Diff 34550!
ginsu requested review of this revision.Dec 12 2023, 2:42 PM

will make sure ci passes before landing

This revision is now accepted and ready to land.Dec 14 2023, 11:43 AM
ginsu edited the summary of this revision. (Show Details)

rebase before landing

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.