Skip to content

fix: prevent unwanted scrollbars in Windows/Tauri desktop app#471

Merged
jamiepine merged 1 commit intospacedriveapp:mainfrom
TheDarkSkyXD:fix/windows-ui-scrollbars
Mar 24, 2026
Merged

fix: prevent unwanted scrollbars in Windows/Tauri desktop app#471
jamiepine merged 1 commit intospacedriveapp:mainfrom
TheDarkSkyXD:fix/windows-ui-scrollbars

Conversation

@TheDarkSkyXD
Copy link
Contributor

Summary

  • The Tauri zoom: 1.1 factor causes w-screen/h-screen elements to overflow the actual viewport, producing unwanted horizontal and vertical scrollbars on Windows
  • Added overflow: hidden; height: 100% to html/body in global styles to suppress the outer window scrollbar
  • Replaced w-screen with w-full on the connection screen to prevent horizontal overflow

Test plan

  • Open the app in Tauri on Windows and verify no scrollbars appear on the connection screen
  • Connect to a server and navigate to Settings > Providers — verify only the provider list content area scrolls, not the outer window
  • Verify the fix doesn't affect browser-based usage (no Tauri zoom applied there)

🤖 Generated with Claude Code

The Tauri zoom factor (1.1) causes `w-screen` and `h-screen` elements
to overflow the actual viewport, producing both horizontal and vertical
scrollbars on the connection screen and settings page.

- Add `overflow: hidden; height: 100%` to html/body to suppress the
  outer window scrollbar
- Replace `w-screen` with `w-full` on the connection screen to prevent
  horizontal overflow

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5ad592c7-67d8-4cb3-9d95-669fd8bb15d7

📥 Commits

Reviewing files that changed from the base of the PR and between 6f81f39 and 16d13cc.

📒 Files selected for processing (2)
  • interface/src/components/ConnectionScreen.tsx
  • interface/src/ui/style/style.scss

Walkthrough

Modified overflow and viewport sizing behavior by changing ConnectionScreen's root container width from w-screen to w-full with overflow-hidden, and added global overflow: hidden and height: 100% constraints to html and body elements.

Changes

Cohort / File(s) Summary
Component Layout Styling
interface/src/components/ConnectionScreen.tsx
Changed root container width utility from w-screen to w-full and added overflow-hidden class to the outermost div element.
Global Stylesheet Rules
interface/src/ui/style/style.scss
Added new global CSS rule for html, body elements setting overflow: hidden and height: 100% to constrain viewport scrolling and ensure full-height rendering.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main fix: preventing unwanted scrollbars in the Windows/Tauri desktop app, which is the primary objective of both CSS changes.
Description check ✅ Passed The description is directly related to the changeset, explaining the root cause (Tauri zoom factor), the solution (CSS changes), and providing a test plan specific to the Windows/Tauri context.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.

OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required.

@jamiepine jamiepine merged commit 5f6c2a7 into spacedriveapp:main Mar 24, 2026
1 check passed
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.

2 participants