feat: auto-close menus and adjust logo size when navbar collapses

[poirazis]

• Menus automatically close when collapsible navbar is collapsed
• Menu state is persisted and restored when navbar expands
• Logo height is capped at 32px when collapsible is enabled to prevent navbar overflow
• Ensures smooth UX without unnecessary scrolling in collapsible mode

addresses #17659

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

[NDCallahan]

just a note from an implementer's perspective:

Logo height is capped at 32px when collapsible is enabled to prevent navbar overflow

I don't think limiting the size to 32px is the right approach for the logo in the navigation bar. Perhaps it's good enough temporarily, but I foresee others wanting to utilize a logo from a navigation perspective and not be capped at 32 px. I did create #issue17659 so perhaps this is sufficient for now.

EDITS
I could see the logo being capped, when the navigation bar collapses, but not when collapse is enabled. To do this, why wouldn't the logo dynamically decrease in size within the parent element?

[melohagan]

Hey @poirazis

I know it's been a minute, but would you mind providing screenshots and/or video of a BEFORE and AFTER the code change?

That way I can quickly get up to speed with the functionality, and give it a review.

[poirazis]

hey @melohagan I will redo this
I messed the branch