-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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.
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
Add resizable and collapsible panel layout #5926
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this, a few notes!
- I accidentally did some duplicate work and opened this PR, sorry: Fix issue #5894: Make it possible to collapse the right-hand side of the openhands screen #5896
- Overall the code looks nice and it worked in my environment. But It seems to have introduced a regression with respect to the design being reactive to small screen widths. Here's the current one in this PR:

and main for reference:

@neubig Responsive design should now work again: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great code-wise, and I confirmed that all the functionality works as expected. Thanks for this, it'll be excellent to have!
Not sure what happened here but you can see it in the last screenshot there--there's a style regression here. Going to revert for now |
Resurrected the resizer component. (This was in place last August)

Fixes #5894
To run this PR locally, use the following command: