Skip to content

feat: add show/hide toolbar toggle in view menu#1100

Open
mvanhorn wants to merge 1 commit intochartdb:mainfrom
mvanhorn:feat/show-hide-toolbar-toggle
Open

feat: add show/hide toolbar toggle in view menu#1100
mvanhorn wants to merge 1 commit intochartdb:mainfrom
mvanhorn:feat/show-hide-toolbar-toggle

Conversation

@mvanhorn
Copy link
Copy Markdown

Fixes #497

Summary

  • Adds a "Show Toolbar" / "Hide Toolbar" toggle in the View menu, following the same pattern as existing toggles (minimap, cardinality, field attributes)
  • Toolbar visibility state is persisted to localStorage via the existing LocalConfigContext pattern
  • Translation keys added to all 22 locale files

Changes

  • local-config-context.tsx - added showToolbar / setShowToolbar to the context interface and defaults
  • local-config-provider.tsx - added localStorage-backed state and persistence for show_toolbar
  • menu.tsx - added toggle menu item in the View menu (between minimap and the zoom-on-scroll separator)
  • canvas.tsx - conditionally renders the toolbar <Controls> wrapper based on showToolbar
  • All locale files - added show_toolbar and hide_toolbar translation keys

Test plan

  • Open the View menu and verify "Hide Toolbar" appears (toolbar is visible by default)
  • Click "Hide Toolbar" and verify the toolbar disappears from the canvas
  • Open the View menu again and verify it now says "Show Toolbar"
  • Click "Show Toolbar" and verify the toolbar reappears
  • Refresh the page and verify the preference is persisted

Developed with AI assistance (Claude Code)

Add a toggle option in the View menu to show/hide the toolbar on the
canvas, following the same pattern as existing show/hide toggles
(minimap, cardinality, field attributes). The preference is persisted
to localStorage.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 17, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@mvanhorn
Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add "Show/Hide toolbar" Option in the View Menu

1 participant