fix(desktop): ドラッグドロップの分割幅をプレビューと一致させる (#342)

[MocA-Love]

概要

Files タブから pane にファイルをドラッグして開いたとき、青いプレビューは 50% ゾーンを示しているのに、実際に開かれた file viewer が Right sidebar open view width 設定 (例: 25%) の幅で開いてしまい、プレビューと食い違う問題を修正。

原因

#336 で追加したドロップ処理では addFileViewerPane に useRightSidebarOpenViewWidth: true を渡していた。これによりストア側が Right sidebar の幅設定を split percentage に流用するため、設定が 25% だとドロップ結果の pane も 25% になっていた。プレビュー側は常に 50% ゾーンを表示するため不一致が発生。

変更点

• BasePaneWindow のドロップハンドラから useRightSidebarOpenViewWidth: true を外し、ドロップ時は常に DEFAULT (50/50) で分割するように変更。
• サイドバー右側の幅設定は、クリックで開く既存動線にのみ適用される状態に戻る。

動作確認

• bun run lint、apps/desktop の bun run typecheck パス。
• ローカルで Right sidebar open view width を 25% に設定 → Files タブからペインの右半分にドラッグ → プレビュー通り 50/50 で分割されることを想定。

Closes #342

Summary by CodeRabbit

リリースノート

• バグ修正
  • ファイルを内部的にドラッグ&ドロップした際の動作を調整しました。

[MocA-Love]

@codex review

[coderabbitai]

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: 5f9c1094-91ce-407b-bbb7-1955260b20ce

📥 Commits

Reviewing files that changed from the base of the PR and between e8739b9 and 97c6b6f.

📒 Files selected for processing (1)

• apps/desktop/src/renderer/screens/main/components/WorkspaceView/ContentView/TabsContent/TabView/components/BasePaneWindow/BasePaneWindow.tsx

💤 Files with no reviewable changes (1)

• apps/desktop/src/renderer/screens/main/components/WorkspaceView/ContentView/TabsContent/TabView/components/BasePaneWindow/BasePaneWindow.tsx

---

📝 Walkthrough

Walkthrough

BasePaneWindow のファイルドロップハンドラーから useRightSidebarOpenViewWidth: true オプションを削除し、addFileViewerPane() の呼び出しを簡素化しました。その他のドラッグ/ドロップの状態管理は変わっていません。

Changes

コホート / ファイル(s)	変更概要
ファイルドロップハンドラー修正 apps/desktop/src/renderer/screens/main/components/WorkspaceView/ContentView/TabsContent/TabView/components/BasePaneWindow/BasePaneWindow.tsx	内部ファイルドロップキャプチャハンドラーで addFileViewerPane() を呼び出す際に、useRightSidebarOpenViewWidth: true オプションを削除

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 分

Possibly related PRs

• Add v1 split previews for sidebar file drags #336: BasePaneWindow の内部ファイルドロップ動作と addFileViewerPane 呼び出しの相対位置指定オプション追加に関連
• fix(desktop): コンフリクトファイルを ConflictViewer で開くよう viewMode を強制 (#248) #253: addFileViewerPane の呼び出し箇所での viewMode フォールバック設定に関連

Poem

🐰 ✨ Right sidebar width の呼び出しが消えて
ドロップされたファイルは正しい幅に
バグの青いプレビューも満足げ
ドラッグの魔法がもう完璧だね！

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	タイトルは PR の主な変更内容（ドラッグドロップの分割幅をプレビューと一致させる）を明確に要約しており、関連する issue 番号も含まれている。
Description check	✅ Passed	PR の説明は、概要、原因、変更点、動作確認などが十分に記載されており、テンプレートの主要要件を満たしている。
Linked Issues check	✅ Passed	PR は issue #342 で報告されたドラッグドロップ時のプレビューと実際の幅の不一致を、useRightSidebarOpenViewWidth: true を除去することで解決している。
Out of Scope Changes check	✅ Passed	変更は BasePaneWindow のドロップハンドラから 1 行を削除する限定的な修正であり、issue #342 の解決に直接関連しており、スコープ外の変更は含まれていない。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/342-drop-split-width

---

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

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. 🚀

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".