Only close Olivero sub-menus when resize results in a different menu format

Took a first swing at this adding a new prop to the init() function to hold onto pre-resize width values. Additionally, added Drupal.debounce to rate-limit the resize handler per the @todo in the file.

Re-rolled patch #3, Please review.

Tests will likely continue to fail as the patch was created against 9.x.3

9.3 will be ready for testing from week 3 of may.

#2 indicates new changes should be made to the 9.3.x branch. Is that true or are minor fixes like this still aimed at the 9.2.x branch?

I have 9.3 branch in my local. There are very minor changes in 9.2 and 9.3.
I made these changes in 9.3 and it still passes 9.2 and it will pass 9.3 as well.

I was unable to test the issue properly, could use more explicit details in how to test it.

How I did it:

1-Created submenus in the home link,
2-Downsized navigation page to 1600 x 900,
3-Expanded the menu,
4-Changed page to full-screen
5-Menu closes, before and after patch.

Reroll of #5

Fixed custom command failure

Needs a re-roll

This is a usability and accessibility issue, and is at least normal if not major. Content unexpectedly disappearing when I make my window bigger to try to read it sounds pretty bad, actually, so erring on the side of major.

Re-roll attached.

Forgot to stage all my changes. New reroll attached.

1. +++ b/core/themes/olivero/js/navigation.es6.js
   @@ -3,7 +3,7 @@
   +((Drupal, once, tabbable, debounce) => {
   

   I'm not opposed to integrating Drupal.debounce, but I'd like to keep it out of the scope of this issue. We have a Drupal.olivero.isDesktopNav() method that can do this.

2. +++ b/core/themes/olivero/js/navigation.es6.js
   @@ -40,6 +40,36 @@
   +      (oldWidth > 1200 && newWidth < 1200) ||
   

   I don't like that we're encoding the navigation change breakpoint here. This means that if we ever change it, we'll need to remember to change this.

Attached is a patch that compares the new and old Drupal.olivero.isDesktopNav() value and if they don't match, it will allow the submenus to close.

No interdiff as this is a different approach.

Tugboat preview: https://3191806-resize-menu-close-ualwvqluvczjwoxevbjdriwl9mah4b3j.tugbo...

Attached the same patch twice.

The desktop submenus only close when we shrink enough to get to the mobile menu, but the mobile menu closes on any resize event. Screen recording attached.

Added this tag per the last attached movie.

(user bnjmnm is teaching me how to contribute to Olivero and asked that I make this a merge request to facilitate that process)

Thanks for the update. Tagging for manual testing.

"banjo music" is not a real tag.

Updating attributions.

Adding 1.4.10 SC.