fix(desktop): prevent webview from capturing drag events during pane rearrangement

[Kitenite]

Summary

• Electron webviews render in a separate compositing layer and capture drag events, breaking React Mosaic pane drag-and-drop when the cursor moves over a browser pane
• Disables pointer-events on pane content during drag so mosaic drop targets receive events correctly

Changes

• BasePaneWindow: Read isDragging from drag pane store; apply pointer-events: none on the content container while a pane drag is in progress

Test Plan

• Open a browser pane alongside other panes
• Drag a pane and move the cursor over the browser pane — drop targets should appear correctly
• Verify normal pane interaction (clicking, typing) still works when not dragging

Summary by CodeRabbit

• Bug Fixes
  • Enhanced pane drag functionality by preventing pointer events from interfering during drag operations, resulting in smoother and more responsive dragging behavior when managing workspace panes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

The change introduces a new isDragging flag from useDragPaneStore to the BasePaneWindow component. When a pane is being dragged, the content container's pointerEvents style is set to none to disable pointer interactions during the drag operation.

Changes

Cohort / File(s)	Summary
Pane Drag Interaction apps/desktop/src/renderer/screens/main/components/WorkspaceView/ContentView/TabsContent/TabView/components/BasePaneWindow/BasePaneWindow.tsx	Added conditional pointerEvents: none styling to content container based on isDragging flag from drag pane store to prevent pointer interactions during pane drag operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• feat(desktop): Add drag-and-drop panes to tab bar #856: Sets and clears the drag state flag in useDragPaneStore on drag start/end operations, which is the companion piece that populates the isDragging flag this PR consumes.
• refactor(desktop): update changes view #595: Introduces the BasePaneWindow component structure and toolbar handlers that form the foundation for this drag interaction enhancement.

Poem

🐰 A pane that floats without a care,
Now hops through drags with pointer's prayer—
When whiskers twitch and dragging's true,
No clicks shall pass, none's the view! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: preventing webview from capturing drag events during pane rearrangement, which directly addresses the core issue being fixed.
Description check	✅ Passed	The PR description comprehensively covers the issue, solution, and testing approach, though it deviates from the template structure by using a custom format with Summary, Changes, and Test Plan sections instead of the required template sections.

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

✨ 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 kitenite/browser-drag

---

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.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ⚠️ Neon database branch
• ⚠️ Electric Fly.io app
• ⚠️ Streams Fly.io app

Thank you for your contribution! 🎉