[native] Remove `ChatThreadList` class component
ClosedPublic
Actions

Authored by atul on Sep 13 2023, 11:46 AM.

Details

Reviewers

ginsu
tomek
rohan

Commits

rCOMMabee4be80946: [native] Remove `ChatThreadList` class component

Summary

This diff finally removes the ChatThreadList class component altogether.

That said, this functional component is still cluttered and hasn't been optimized. I will work to declutter ChatThreadList and improve performance in subsequent diffs.

Depends on D9190

Test Plan

Everything continues to work as expected/before!

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Sep 13 2023, 11:46 AM

Herald added a subscriber: ashoat. · View Herald TranscriptSep 13 2023, 11:46 AM

atul published this revision for review.Sep 13 2023, 11:47 AM

Harbormaster completed remote builds in B22613: Diff 31092.Sep 13 2023, 12:02 PM

ginsu accepted this revision.Sep 13 2023, 1:30 PM

This revision is now accepted and ready to land.Sep 13 2023, 1:30 PM

Closed by commit rCOMMabee4be80946: [native] Remove `ChatThreadList` class component (authored by atul). · Explain WhySep 13 2023, 2:56 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMabee4be80946: [native] Remove `ChatThreadList` class component.

Revision Contents
Changeset List

Path

Size

native/

chat/

chat-thread-list.react.js

47 lines

Diff 31092

View Options

native/chat/chat-thread-list.react.js

This file was added.

				.container {
				display: flex;
				column-gap: 8px;
				align-items: center;
				justify-content: center;
				}

[native] Remove `ChatThreadList` class componentClosedPublicActions