Skip to content

fix(content): hide empty settings panel#89

Merged
DavidBabinec merged 1 commit into
mainfrom
fix/empty-content-settings-panel
Jun 26, 2026
Merged

fix(content): hide empty settings panel#89
DavidBabinec merged 1 commit into
mainfrom
fix/empty-content-settings-panel

Conversation

@DavidBabinec

Copy link
Copy Markdown
Contributor

Summary

  • Gate the shared workspace right sidebar on real right-panel content so persisted open state cannot reserve an empty rail.
  • Preserve a closed Content settings preference when selection is cleared.
  • Add Content regressions for empty installs and deleting the last selected entry.

Root Cause

The shared workspace layout expanded from persisted right-panel state without checking whether the current workspace had right-panel content. Content also reopened the right panel for selectEntry(null), so clearing the active entry could rewrite the preference to open.

Impact

Fresh installs and empty Content collections now show the full canvas without an empty right rail. Selecting or creating an entry still opens the settings panel, and the reopen notch still appears only when an entry-backed panel exists.

Verification

  • bun run build
  • bun test
  • bun run lint
  • bunx react-doctor@latest --verbose --diff
  • bunx react-doctor@latest --verbose --scope changed --base main
@DavidBabinec DavidBabinec marked this pull request as ready for review June 26, 2026 10:12
@DavidBabinec DavidBabinec merged commit 6643353 into main Jun 26, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant