[web] Remove old `SidebarListModal` component
ClosedPublic
Actions

Authored by • jacek on Jul 22 2022, 8:29 AM.

Details

Reviewers

tomek
atul

Commits

rCOMM5b394475454f: [web] Remove old `SidebarListModal` component

Summary

The diff removes an old sidebar list modal as it's no longer used.

Test Plan

web app should build without errors. The component was no longer used on web.

Diff Detail

Repository

rCOMM Comm

Branch

jacek/sidebars

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• jacek created this revision.Jul 22 2022, 8:29 AM

Herald added subscribers: • abosh, • adrian, ashoat. · View Herald TranscriptJul 22 2022, 8:29 AM

• jacek added a parent revision: D4611: [web] Open `SidebarsModal` instead `SidebarListModal` on web.Jul 22 2022, 8:32 AM

Harbormaster completed remote builds in B10779: Diff 14820.Jul 22 2022, 8:44 AM

• jacek requested review of this revision.Jul 22 2022, 8:44 AM

Are there any css classes that became unused after this modal is removed?

This revision is now accepted and ready to land.Jul 25 2022, 8:44 AM

removed unused styles

• jacek requested review of this revision.Jul 26 2022, 3:41 AM

Harbormaster completed remote builds in B10871: Diff 14947.Jul 26 2022, 3:53 AM

tomek accepted this revision.Jul 26 2022, 3:55 AM

This revision is now accepted and ready to land.Jul 26 2022, 3:55 AM

Closed by commit rCOMM5b394475454f: [web] Remove old `SidebarListModal` component (authored by • jacek). · Explain WhyJul 26 2022, 8:30 AM

This revision was automatically updated to reflect the committed changes.

• jacek added a commit: rCOMM5b394475454f: [web] Remove old `SidebarListModal` component.

Revision Contents
Changeset List

Path

Size

web/

chat/

chat-thread-list.css

40 lines

modals/

chat/

sidebar-list-modal.react.js

Diff 14947

View Options

web/chat/chat-thread-list.css