longbridgeapp / gpui-component

UI components write in GPUI.
Apache License 2.0
150 stars 20 forks source link

Saving the Layout is Broken on the latest main #302

Closed stormasm closed 2 weeks ago

stormasm commented 2 weeks ago

Jason,

You probably already know this :)

But saving the layout is currently not working with your latest fixes in the past day.

A simple test is to reduce the list on the left side of the screen. Then close out of everything and bring it back up. And the list appears like it looked before you shrink it down

Everything is working fine at this point... https://github.com/huacnlee/gpui-component/tree/7de1c6981e22317f00fea629ae32c44b5ef317c6

Thanks ! You are doing amazing work on helping us to better understand gpui. Michael

PS. Once you get it working I will be happy to test again..

huacnlee commented 2 weeks ago

Yes, I will to this. I just merge a part of work.

stormasm commented 2 weeks ago

Thank you very much ! It is now all working beautifully 😄