Skip to content

feat(genre-tree): add controlsOrientation prop to GenreTreeView - #184

Merged
Andreas-Garcia merged 2 commits into
developfrom
feature/controls-orientation
Oct 3, 2026
Merged

Andreas-Garcia merged 2 commits into
developfrom
feature/controls-orientation

Conversation

@Andreas-Garcia

Copy link
Copy Markdown
Member

Summary

Expose genre-tree-view 1.14.0's controlsOrientation on GenreTreeView so consumers (grow frontend) can lay out the floating zoom/pop/fit controls horizontally.

Changes

  • Bump @behindthemusictree/genre-tree-view catalog pin 1.13.0 → 1.14.0.
  • GenreTreeView: new controlsOrientation?: ControlsOrientation prop (default "vertical" via the lib), threaded into the wheel, pop-core and stacked renderers (TreeWheel, TreeWheelRadialPopCore, TreePerRoot). Outline view is skipped — it has no floating controls and the lib's GenreTreeOutlineProps omits the prop.
  • Re-export the ControlsOrientation type alongside GenreTreeViewMode.
  • CHANGELOG [Unreleased] entries.

Test plan

  • Pass-through tests for each renderer wrapper (incl. outline not receiving it) and for GenreTreeView into pop-core, wheel and stacked views.
  • pnpm lint && pnpm build && pnpm test green locally.

🤖 Generated with Claude Code

Andreas-Garcia and others added 2 commits October 3, 2026 15:06
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 3, 2026 13:08
@Andreas-Garcia
Andreas-Garcia merged commit 5a2a32e into develop Oct 3, 2026
2 of 3 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants