Skip to content

Full storage navigation fix - #246

Merged
SharonStrats merged 2 commits into
stagingfrom
fix/navigation-views
Oct 25, 2026
Merged

SharonStrats merged 2 commits into
stagingfrom
fix/navigation-views

Conversation

@SharonStrats

Copy link
Copy Markdown
Contributor

Fix navigation as the full storage view was not displaying on the storage containers.

…what is being shown

Prompt: When we look at a root storage only the folder pane container is being shown, this
should only happen on the soloPane scenario for non root resources.

Co-authored-by: Claude Opus 5.5 <claudeopus5.5-opus@openai.com>
@SharonStrats
SharonStrats requested a balanced review from Copilot October 25, 2026 00:12
@SharonStrats SharonStrats self-assigned this Oct 25, 2026
@SharonStrats SharonStrats added the bug Something isn't working label Oct 25, 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.

🟢 Approval recommended

The localized changes preserve legacy defaults, consistently apply the new view modes, and have no identified blocking issues.

No open issues 🎉

What changed in this PR

This PR adds explicit folder view modes to support full storage navigation while preserving existing solo and fullView behavior.

Changes:

  • Adds explorer, storage, and container modes, with explicit selections taking precedence over legacy options.
  • Allows storage mode to hide the storage header while retaining the sidebar and content.
  • Adds view-selection tests and bumps the package version to 4.0.1-1.
File Description
test/​folderView.test.js Tests explicit view selection and legacy defaults.
src/​folderPane.ts Resolves view modes and applies component options.
src/​components/​folder-pane-view/​FolderPaneView.ts Makes the storage header optional.
package.json Updates the package version.
package-lock.json Synchronizes the lockfile version metadata.

Review effort: Balanced

@SharonStrats SharonStrats moved this to In review in SolidOS NLNet UI Oct 25, 2026

@timea-solid timea-solid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

am excited to see a fix for this.

@SharonStrats
SharonStrats merged commit 7a0c32c into staging Oct 25, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in SolidOS NLNet UI Oct 25, 2026
@SharonStrats
SharonStrats deleted the fix/navigation-views branch October 25, 2026 19:28
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.

3 participants