[web] introduce community list item component
ClosedPublic
Actions

Authored by ginsu on Dec 15 2023, 12:54 PM.

Details

Reviewers

atul
rohan
kamil

Commits

rCOMM96eec5df343a: [web] introduce community list item component

Summary

This diff introduces the community list item component. This component will be used in a community list component that I will introduce in a future diff.

For context on what this should look like here is a screenshot from the figma:

Part of https://linear.app/comm/issue/ENG-5947/list-of-all-top-level-communites-communities

Depends on D10358

Test Plan

Please see the screenshot below

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Dec 15 2023, 12:54 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptDec 15 2023, 12:54 PM

ginsu edited the summary of this revision. (Show Details)Dec 15 2023, 12:56 PM

ginsu edited the test plan for this revision. (Show Details)

ginsu added reviewers: atul, rohan, kamil.

ginsu added a parent revision: D10358: [web] introduce useNavigationSidebarTooltip hook.

ginsu edited the summary of this revision. (Show Details)

ginsu added a child revision: D10360: [web] introduce community list.Dec 15 2023, 1:10 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Dec 15 2023, 1:37 PM

Harbormaster failed remote builds in B25136: Diff 34746!

will make sure ci passes before landing

rohan accepted this revision.Dec 15 2023, 4:33 PM

rohan added inline comments.

web/navigation-sidebar/community-list-item.react.js
28 ↗	(On Diff #34746)	Just a question after seeing the screenshots in D10360 - could you share what the community list looks like if we use a medium `ThreadAvatar`?

This revision is now accepted and ready to land.Dec 15 2023, 4:33 PM

ginsu added inline comments.Dec 15 2023, 5:51 PM

web/navigation-sidebar/community-list-item.react.js
28 ↗	(On Diff #34746)	Here is what it looks like with medium thread avatars: I went with small because in the figma the dimensions for the thread avatar are 24x24 (which is the size of the small avatar)