Page MenuHomePhabricator

[web] Add community drawer to CommunityPicker
ClosedPublic

Authored by inka on Jan 4 2023, 8:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 4:09 AM
Unknown Object (File)
Sun, Nov 24, 2:43 AM
Unknown Object (File)
Sun, Nov 24, 2:29 AM
Unknown Object (File)
Sun, Nov 24, 12:47 AM
Unknown Object (File)
Mon, Nov 4, 7:55 AM
Unknown Object (File)
Tue, Oct 29, 10:24 AM
Unknown Object (File)
Oct 28 2024, 8:11 AM
Unknown Object (File)
Sep 28 2024, 1:02 PM
Subscribers

Details

Summary

linear issue: https://linear.app/comm/issue/ENG-2631/drawer-sidebar-inside-of-community-picker
The designs are not done yet, so styling will be updated later on.

Test Plan

Run the web app, checked that community drawer is present in CommunityPicker and behaves as expected.

Diff Detail

Repository
rCOMM Comm
Branch
inka/community_drawer_web
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka requested review of this revision.Jan 4 2023, 9:14 AM

Fix in place community-picker width

web/sidebar/community-picker.css
27 ↗(On Diff #20582)

These could be merged.

Fix grid usage

This revision is now accepted and ready to land.Jan 26 2023, 6:17 AM
This revision now requires review to proceed.Jan 27 2023, 2:10 AM
tomek added inline comments.
web/sidebar/left-layout-aside.css
6 ↗(On Diff #21148)

Have you checked if 100% or flex: 1 work ok?

This revision is now accepted and ready to land.Jan 31 2023, 7:04 AM
web/sidebar/left-layout-aside.css
6 ↗(On Diff #21148)

This will be revisited during the styling task. I remember having some problems with the normal flex-based approach. But if you don't mind I won't change that, because this whole component actually gets deleted down the road, and all styles will be updated anyway.