It is fine/collapsed on the Home Dashboard initially, but going to other sections such as Articles list, it expands again.

When I do a refresh in Chrome with the mobile view active, the logo is initially expanded. When I resize the mobile window just a little bit, the logo shrinks and everything jumps into place. When I do a refresh again the logo is back to its initial expanded state.

Is this animation by design? its annoying haha

only other thing - which I dont think is related to this PR is

On mobile, navigate to the User Action Log page
Click bottom right corner to launch menu
Note the menu is open - correct
Note you cannot get to HomeDashboard by scrolling - broken, but probably ok as you can collapse the open Users menu to see it. (Home Dashboard is off screen, and you cannot scroll to is, you have to collapse the open menu leaf to get to it)

When I do a refresh in Chrome with the mobile view active, the logo is initially expanded. When I resize the mobile window just a little bit, the logo shrinks and everything jumps into place. When I do a refresh again the logo is back to its initial expanded state.

I cannot replicate this today. On a real mobile.

It is fine/collapsed on the Home Dashboard initially, but going to other sections such as Articles list, it expands again.

I cannot replicate this today.

I have tested this item ✅ successfully on b0d82e0

When I resize the mobile window

What I will say, is that the web is meant to be browsed not squished.

You are correct that there are some issues that a refresh of the page fixes, however its very rare to resize from a desktop view to a mobile view and back to a desktop view without refreshing and so testing that is not really a valid test I dont believe.

What I will say, is that the web is meant to be browsed, not squished.

There are still assumptions with the current code, especially for the transition to mobile or going from mobile to desktop.

Could we tackle those?

Probably yes by transforming the cookie value to a JSON string (store state for logo, menu state and header toolbar), but I see little advantage to complicate further the code to iron out a couple of glitches which are edge cases (e.g. clicking on the save button on mobile view will not reload the page with the toolbar open). There are trade-offs (perf vs covering all cases), and as is the script is not very efficient/performant, and my whole approach here was to eliminate as much as possible the annoying open/close animations on page reload and not to re-architecture the way we put together those underlaying parts

however its very rare to resize from a desktop view to a mobile view and back to a desktop view without refreshing and so testing that is not really a valid test I dont believe.

^^ thats my way of saying I agree with you, its a non-issue at this stage. If you are going to rotate a mobile device from landscape to portrait and back again I dont expect a website to handle that perfectly, even in 2021, and its no harm to refresh.

I have tested this item ✅ successfully on cdfbb5f

I have tested this item ✅ successfully on cdfbb5f

A quick test as I tested intensively previously

RTC

Thanks!