Repository navigation
feat(explorer): Sidebar position moves the Explorer's sidebar, the preview takes the other side (#304) - #305
Merged
Merged
Conversation
…eview takes the other side (#304) One Sidebar position row (prism.tree.side), first in Explorer > Layout, moves the Explorer's places panel and the project tree together. On Right the places sit at the window's right edge, the preview pane at the left and its toggle leads the address row; grips, hide, peek and the see-through ground all hold mirrored. Left is unchanged. Project settings no longer carries the row. Version 0.93.0. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t
… and one for projects (#304) Owner, 2026-10-07, after testing one shared row: "No, it should be two settings, one on the project tab and one on the explorer tab". Explorer > Layout's Sidebar position (explorer-side, new key prism.explorer.side, default Left) moves only the Explorer tab's places panel, with the preview pane and its toggle on the other side. Project settings gets its Sidebar position back (tree-side, prism.tree.side) and it moves only the project tree. Neither moves the other; each panel peeks from its own side. The explorerSide e2e proves both directions and that both choices outlive a restart; Find a setting finds both rows. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t
This was referenced Oct 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #304
What
Two Sidebar position settings (Left / Right), one per page, each moving only its own panel:
explorer-side, new keyprism.explorer.side, default Left): "The side the places panel sits on." It moves the Explorer tab's places panel only. A project tab's tree stays where it is.tree-side, keyprism.tree.side, unchanged, back where it was before this PR): "The side the file tree sits on." It moves the project tab's file tree only. The Explorer stays where it is.Find a setting finds both rows, each on its own page.
With the Explorer's Right:
Left is the window as it was, box for box.
Why
Owner, 2026-10-07: "fix the setting in Explorer for the sidebar where you can put it on the right side or the left side? I think that's just an empty setting for now, but actually implement it. And remember that when the sidebar goes on the right, the preview menu and button to open it would have to go on the left."
Then, after testing the first version of this PR (one shared row): "No, it should be two settings, one on the project tab and one on the explorer tab."
Decision
Two independent stores:
lib/explorerSidePrefs.ts(new) for the Explorer,lib/treePrefs.tsfor the project tree. App reads the Explorer's side for an Explorer tab and the tree's for a project tab (workspaceSide), and each panel peeks from its own side. Rule written into CLAUDE.md with both owner quotes. Version 0.92.1 -> 0.93.0 (feature).Gates
npm run typecheck: cleannpm run lint: 0 errors (7 existing warnings)npx vitest run: 2529 passed, 2 skipped (171 files)explorerSide(62 checks): both rows on their own pages with distinct subtexts; the Explorer's Right moves the places and preview and leaves the project tree exactly where it was; the project's Right moves the tree and leaves every Explorer box exactly as it was; toggle, grips, hide, peek, pin and one coat on Glacier turned round; both choices survive a restart.npm run e2e:terminal: all 17 ok (553 checks)🤖 Generated with Claude Code
https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t