Skip to content

consolidate sidebar children#506

Merged
kraleppa merged 6 commits intosoftware-mansion:mainfrom
samrobinsonsauce:495-consolidate-sidebar-children
Jun 24, 2025
Merged

consolidate sidebar children#506
kraleppa merged 6 commits intosoftware-mansion:mainfrom
samrobinsonsauce:495-consolidate-sidebar-children

Conversation

@samrobinsonsauce
Copy link
Contributor

Potential fix for #495.

Renders a single <.sidebar_content> component rather than duplicates which helps retain form state when we transition to a mobile view.

Copy link
Contributor

@hhubert6 hhubert6 left a comment

Choose a reason for hiding this comment

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

Looks good, but it definitely needs to be incorporated into sidebar_slide_over so that it could be reusable.

@samrobinsonsauce
Copy link
Contributor Author

samrobinsonsauce commented Jun 16, 2025

Thanks @hhubert6, i've updated the sidebar_slide_over component to leverage these changes.

Let me know what you think 😄

@kraleppa kraleppa requested review from GuzekAlan and kraleppa June 24, 2025 08:13
Copy link
Member

@kraleppa kraleppa left a comment

Choose a reason for hiding this comment

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

awesome - thanks for contribution!

@kraleppa kraleppa linked an issue Jun 24, 2025 that may be closed by this pull request
@kraleppa kraleppa merged commit 9fd0527 into software-mansion:main Jun 24, 2025
2 checks passed
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.

Refactor: Change Sidebar to not have duplicated children

4 participants