Skip to content

#2069 Fixed/The sidebar has empty sections with dividers when collapsed if Feeds, Squads, etc. sections are collapsed #5718

Open
nchernys wants to merge 2 commits intodailydotdev:mainfrom
nchernys:fix-sidebar-empty-sections
Open

#2069 Fixed/The sidebar has empty sections with dividers when collapsed if Feeds, Squads, etc. sections are collapsed #5718
nchernys wants to merge 2 commits intodailydotdev:mainfrom
nchernys:fix-sidebar-empty-sections

Conversation

@nchernys
Copy link

Changes

Describe what this PR does

Updates the NavSection component in Section.tsx to fix empty sections on the collapsed sidebar.

  • hides the header text only
  • keeps the arrow and plus icons visible
  • adds titles to the arrow and plus icons to help user navigate the collapsed sidebar
  • moves the divider to the section bottom

Events

Did you introduce any new tracking events?

Experiment

Did you introduce any new experiments?

Manual Testing

Caution

Please make sure existing components are not breaking/affected by this PR

If relevant, please remove the comment HTML comment tags and fill the checkboxes below

On those affected packages:

  • Have you done sanity checks in the webapp?
  • Have you done sanity checks in the extension?
  • Does this not break anything in companion?

Did you test the modified components media queries?

  • MobileL (420px)
  • Tablet (656px)
  • Laptop (1020px)

Did you test on actual mobile devices?

  • iOS (Chrome and Safari)
  • Android

@vercel
Copy link

vercel bot commented Mar 12, 2026

@nchernys is attempting to deploy a commit to the Daily Dev Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Mar 12, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ rebelchris
❌ nchernys
You have signed the CLA already but the status is still pending? Let us recheck it.

@vercel
Copy link

vercel bot commented Mar 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Mar 13, 2026 6:57am

Request Review

@rebelchris
Copy link
Contributor

@claude fix lint issues and make sure circleCI is green

@claude
Copy link
Contributor

claude bot commented Mar 13, 2026

Claude Code is working…

I'll analyze this and get back to you.

View job run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants