Skip to content

refactor(ui): move sidebar components to ui package for reusability #246

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Nov 27, 2025
Merged

refactor(ui): move sidebar components to ui package for reusability #246

merged 2 commits into from
Nov 27, 2025

Conversation

@pasevin
Copy link
Collaborator

@pasevin pasevin commented Nov 26, 2025

Summary

  • Add generic sidebar components (SidebarButton, SidebarLayout, SidebarSection) to @openzeppelin/ui-builder-ui package
  • Refactor builder's AppSidebar to use the new generic components
  • Enable sidebar reuse across other projects (e.g., role-manager)

Changes

Component Description
SidebarButton Generic styled button/link for sidebar actions with icon, badge, and selection states
SidebarLayout Flexible layout shell with header/content/footer slots and mobile slide-over support
SidebarSection Section wrapper with optional title for grouping sidebar items

Test plan

  • Verify builder app sidebar renders correctly on desktop
  • Verify mobile slide-over works as expected
  • Verify all sidebar interactions (create new, import, export, load contract UI) function properly
  • Run pnpm typecheck and pnpm lint pass

@pasevin
chore(common): remove old constitution bak
ef75093
@pasevin
refactor(ui): move sidebar components to ui package for reusability
d44ab4c 
- Add SidebarButton, SidebarLayout, SidebarSection to @openzeppelin/ui-builder-ui
- Refactor builder's AppSidebar to use generic components from ui package
- Enable sidebar reuse across other projects like role-manager
@pasevin pasevin requested a review from a team as a code owner November 26, 2025 22:31
@pasevin pasevin merged commit fbc8ecd into main Nov 27, 2025
11 checks passed
@pasevin pasevin deleted the refactor/move-sidebar-to-ui-package branch November 27, 2025 09:33
@oz-release-app oz-release-app bot mentioned this pull request Nov 27, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pasevin