[DashboardLayout] No CSS transitions in persistent sidebar under certain screen width

[apedroferreira]

I'm looking at

toolpad/packages/toolpad-core/src/DashboardLayout/DashboardLayout.tsx

Line 251 in 1561c23

const hasDrawerTransitions =

and at

toolpad/packages/toolpad-core/src/DashboardLayout/DashboardSidebarSubNavigation.tsx

Line 124 in 1561c23

...(hasDrawerTransitions

and I'm wondering if there's a specific reason why transitions are disabled on small viewports?

It feels a bit odd when the sidenav transitions smoothly when opening but the sub-header just snap.

Screen.Recording.2024-12-03.at.21.58.12.mov

Originally posted by @rolandjitsu in #4503

Search keywords:

[apedroferreira]

The original intention was to disable the CSS transitions only for the temporary sidebar that shows on mobile devices, but there's a bug as it can also show on the persistent sidebar.
The proposed solution is to change the logic for the hasDrawerTransitions variable so that this behavior works as originally intended.