Navigation Expand/Collapse logic is not working properly in conjunction with Big Pipe

Created MR where the mentioned if clause is replaced by a check for the element in the context.
Also updated the specific Nightwatch test for this functionality to use big_pipe.

Tested manually and at first glance it works well, but a review from more experienced JS developer would be appreciated.

Hi @plopesc,

I followed the same steps you mentioned and performed a fresh setup with the following modules. After installing the modules, the collapse and expand functionality worked as expected for me.

Does this issue occur consistently for you, or is it intermittent? Please let me know if there are any specific steps or configurations you have applied so I can look more effectively.

I suddenly discovered that we don't need context and behavior.

Tested manually and it behaves as expected, fixing the reported bug.

However, the JS here is out of my technical knowledge. So I prefer to have the RTBC signoff from a more experienced FE developer.

Thank you!

So ran the test-only feature and appears to be passing. Possible it could be something else?

@smustgrave i'm not sure. what you mean? tests green. we need review here

Unless nightwatch doesn’t work in the test only feature I’d expect the test to fail without the fix

Then I would solve the testing problem in the next task. Here we have a critical bug.

I have no idea how the BigPipe works and how well it is handled by Nightwatch. But now the Navigation module does not work at all in Olivero and partly in Claro.

https://www.drupal.org/project/drupal/issues/3506880 🐛 Nightwatch test Expand/Collapse passed with broken Navigation Active

I’ll leave for someone else then. But typically see that when tests are pushed to a follow up they usually don’t happen

Made some new changes in the Nightwatch test to ensure that cache is cleared before reloading the navigation after installing bigpipe and it failed locally without the JS changes.

However, the test-only job stays green. I don't see whether the specific nightwatch test was executed either. Could it be something wrong with the CI job?

I'm not sure the test only job supports nightwatch at all.

It might be worth a second, test only MR, to show the nightwatch test failing without the fix just to make sure it's not a bigger CI issue.

Created MR 11227 that demonstrates that test-only changes break the Nightwatch test, while the original MR 11124 is still green.

The code looks great, some nice simplifications while addressing the bug.

I've tested with and without big_pipe. The expand/collapse logic works in both scenarios. I've also tested all of the triggers for expand/collapse, which include the hamburger, close icon, and overlay on mobile as well as the bottom left arrow on desktop. All of those scenarios work as designed.

RTBC for me. Nice work all!

Committed d0e11e1 and pushed to 11.x. Thanks!

Could be good to have for 11.1.x, doesn't cherry pick cleanly so need a new MR

nod_ → closed merge request !11124

@nod_ if I'm not mistaken, this bug was a regression introduced by 📌 Use a placeholder for the navigation toolbar Active to adapt Navigation to 📌 Render the navigation toolbar in a placeholder Active and looks like that none of them were backported to 11.1.x.

Not sure if we need to backport this one to 11.1.x then.

My bad thanks for checking this!

Automatically closed - issue fixed for 2 weeks with no activity.