fix: adapt in-app titlebar layout to the native Windows title bar#163
fix: adapt in-app titlebar layout to the native Windows title bar#163djdevpro wants to merge 1 commit into
Conversation
The overlay drag strips (h-11/h-10) and the pl-22 traffic-light clearance exist only for the macOS overlay title bar. On Windows — which keeps a native, DWM-themed title bar (erictli#136) — they left a dead gap above the content and a large empty gutter when the sidebar was collapsed. - add an `isWindows` helper and a single `platform-<os>` class on <html> - hide the overlay drag strips on Windows (kept on macOS/Linux) - add a top border + small breathing padding under the native caption - skip the pl-22 traffic-light clearance on Windows
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThis PR introduces Windows platform detection and applies platform-specific UI adjustments across the app. It adds an ChangesWindows Platform Detection and Component UI Adaptation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
The overlay drag strips (h-11/h-10) and the pl-22 traffic-light clearance exist only for the macOS overlay title bar. On Windows — which keeps a native, DWM-themed title bar (#136) — they left a dead gap above the content and a large empty gutter when the sidebar was collapsed.
isWindowshelper and a singleplatform-<os>class onSummary by CodeRabbit
New Features
Style