chore(theme): transparency mode not removing background of server view

[rodrigok]

Closes #ISSUE_NUMBER

Summary by CodeRabbit

Release Notes

• Style
  • Updated component background styling for improved visual consistency.
  • Modified background rendering to use transparent backgrounds instead of fixed colors.
  • Simplified background styling across UI components.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai[bot]]

Walkthrough

Two UI components had their background styling simplified: the ServersView Wrapper now uses a transparent background instead of a dark color, and the Shell component no longer applies conditional background styling to its inner container.

Changes

Cohort / File(s)	Summary
UI Component Background Styling src/ui/components/ServersView/styles.tsx, src/ui/components/Shell/index.tsx	Removed or simplified background styling in UI components. ServersView Wrapper background changed from #2f343d to transparent; Shell inner Box background prop removed (previously applied platform-conditional styling).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Simple, homogeneous CSS/styling changes across two components
• No logic modifications or API changes
• Straightforward visual tweaks requiring minimal contextual reasoning

Poem

Transparent dreams and backgrounds fade, ✨ Where colors once held dark parade, A rabbit hops through cleaner views, With lighter styles and fresher hues! 🐰

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing backgrounds from UI components to enable transparency mode in the server view.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• [ ] 📝 Generate docstrings

🧪 Generate unit tests (beta)

• [ ] Create PR with unit tests
• [ ] Post copyable unit tests in a comment
• [ ] Commit unit tests in branch chore/transparency

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 3b1c7c3c601afd4794ef2be8d8309060e08f5aa1 and e84079e8999ecc9845087d32aa570d8b34fbe912.

📒 Files selected for processing (2)

• src/ui/components/ServersView/styles.tsx (1 hunks)
• src/ui/components/Shell/index.tsx (0 hunks)

💤 Files with no reviewable changes (1)

• src/ui/components/Shell/index.tsx

🧰 Additional context used

📓 Path-based instructions (2)

**/*.{ts,tsx}

📄 CodeRabbit inference engine (CLAUDE.md)

**/*.{ts,tsx}: Follow existing TypeScript patterns with strict mode enabled All new code must pass ESLint and TypeScript checks Avoid unnecessary comments; only add comments for complex logic or non-obvious decisions

Files:

• src/ui/components/ServersView/styles.tsx

src/ui/**/*.tsx

📄 CodeRabbit inference engine (CLAUDE.md)

src/ui/**/*.tsx: Implement React UI with functional components and hooks Use Fuselage components (Box, Button, TextInput, Modal, etc.) and import from @rocket.chat/fuselage instead of raw HTML elements For Fuselage theming, validate tokens against Theme.d.ts and only use documented values Name React component files in PascalCase; non-component files should follow camelCase naming

Files:

• src/ui/components/ServersView/styles.tsx

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: check (macos-latest)
• GitHub Check: check (ubuntu-latest)
• GitHub Check: check (windows-latest)

🔇 Additional comments (1)

src/ui/components/ServersView/styles.tsx (1)

8-18: Wrapper background transparency aligns with PR intent

Switching the Wrapper background to transparent is consistent with making the server view respect the outer shell / OS transparency, while keeping the visibility logic unchanged. I don’t see any functional regressions in this file; just ensure you visually verify on all supported platforms (with transparency both enabled and disabled) that the underlying background behaves as expected.

---

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 and usage tips.}