[juliajforesti]

Proposed changes (including videos or screenshots)

ARCH-2499
Fixes layout regressions in the secondary sidebar feature preview introduced by #42819:

• Rows did not reach the edges of the list. The row wrapper and the filter tablists added paddingInline={4}, so the hover and selected surfaces stopped short of the edges and read as pills. The padding is removed.
• Filters were misaligned with the rooms and group headers. The filters rendered a bare Item without an inset, while rooms and group headers use the md inset. They now render SidebarItem, so all rows share the same inset. The filter dividers follow that inset (marginInline={12}).
• The sticky group header was transparent. Before the migration the collapse group bar carried the sidebar surface; the Item group header does not, so rooms scrolling under the pinned header showed through it. The virtualized group wrapper now takes surface-sidebar.

All changes live in the application, under the feature preview's RoomList; Fuselage and other SidebarItem consumers are untouched.

Issue(s)

Regression from #42819 (unreleased), so no changeset.

Steps to test or reproduce

1. Enable the secondary sidebar feature preview.
2. Hover and select rooms and filters (All, Favorites, Discussions, omnichannel filters): the surface fills the full row width.
3. Check that filter icons, group headers and room avatars line up.
4. Scroll the room list: the pinned group header stays opaque over the rooms passing beneath it.

Further comments

🤖 Generated with Claude Code

[dionisio-bot]

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.73%. Comparing base (d7dc383) to head (aebb414).
⚠️ Report is 5 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #42901   +/-   ##
========================================
  Coverage    70.73%   70.73%           
========================================
  Files         5424     5423    -1     
  Lines       230979   230979           
  Branches     42082    42080    -2     
========================================
+ Hits        163381   163391   +10     
+ Misses       62291    62273   -18     
- Partials      5307     5315    +8     

Flag	Coverage Δ
e2e	58.90% <100.00%> (-0.04%)	⬇️
e2e-api	45.90% <ø> (-0.03%)	⬇️
unit	71.28% <100.00%> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[cubic-dev-ai]

No issues found across 5 files

_{View guided diff | Re-trigger cubic}

[github-actions]

🖥️ UI preview

Removed from ghcr.io/rocketchat/rocket.chat-web. Add the preview label again to publish a new one.