[web] introduce community list
ClosedPublic
Actions

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

Details

Reviewers

atul
rohan
kamil

Commits

rCOMM845969a02f4b: [web] introduce community list

Summary

This diff introduces the community list component that will be used in the navigation sidebar component (which will be introduced 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 D10359

Test Plan

Please see the demo videos below

User with many communities:

user with not many communities:

Screenshot 2023-12-15 at 4.04.14 PM.png (1×3 px, 946 KB)

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

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

ginsu added reviewers: atul, rohan, kamil.

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

ginsu attached a referenced file: F954850: Screenshot 2023-12-15 at 3.56.48 PM.png. (Show Details)

ginsu edited the test plan for this revision. (Show Details)Dec 15 2023, 1:05 PM

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

Harbormaster failed remote builds in B25137: Diff 34747!

will make sure ci passes before landing

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

ginsu added a parent revision: D10359: [web] introduce community list item component.

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

ginsu added a child revision: D10361: [web] introduce navigation sidebar home button.

rohan mentioned this in D10359: [web] introduce community list item component.Dec 15 2023, 4:33 PM

rohan accepted this revision.Dec 19 2023, 6:53 AM

rohan added inline comments.

web/navigation-sidebar/community-list.react.js
18–22 ↗	(On Diff #34747)	Something like this might work

This revision is now accepted and ready to land.Dec 19 2023, 6:53 AM

address comments + rebase before landing

Harbormaster completed remote builds in B25296: Diff 34933.Dec 21 2023, 3:44 PM

Closed by commit rCOMM845969a02f4b: [web] introduce community list (authored by ginsu). · Explain WhyDec 21 2023, 4:41 PM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM845969a02f4b: [web] introduce community list.

Revision Contents
Changeset List

Path

Size

web/

navigation-sidebar/

community-list.css

8 lines

community-list.react.js

29 lines

Diff 34941

View Options

web/navigation-sidebar/community-list.css