Skip to content

show slideshow pane - #248

Merged
SharonStrats merged 2 commits into
stagingfrom
slideshow
Oct 26, 2026
Merged

SharonStrats merged 2 commits into
stagingfrom
slideshow

Conversation

@SharonStrats

@SharonStrats SharonStrats commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Slideshow isn't the same as the other panes, it operates on a container that has more than one image in it, therefore it required extra code to get it to render.

@SharonStrats
SharonStrats requested a balanced review from Copilot October 26, 2026 00:05
@SharonStrats SharonStrats self-assigned this Oct 26, 2026
@SharonStrats SharonStrats added the bug Something isn't working label Oct 26, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Hiding the explorer header for slideshow folders removes the direct switch back to Contents, while Grid/List controls cannot restore browsing.

1 open issue
What changed in this PR

This PR lets the folder pane display a preferred specialized pane, such as a slideshow, instead of always showing folder contents.

Changes:

  • Replaces local pane-selection logic with the shared getPreferredPane helper.
  • Routes non-indexed containers to their preferred pane while retaining the container view when the folder pane is selected.
  • Requires a browser context before rendering selected resources.
File Description
src/​components/​folder-pane-view/​FolderPaneView.ts Uses shared pane selection and enables specialized views for containers.

Review effort: Balanced

Comment thread src/components/folder-pane-view/FolderPaneView.ts Outdated
@SharonStrats
SharonStrats merged commit f5ae226 into staging Oct 26, 2026
8 checks passed
@SharonStrats
SharonStrats deleted the slideshow branch October 26, 2026 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants