🇺🇸United States @SuperfluousApostrophe

On 3.0.0-alpha4, there is an event listener added to the document element every single time that the function ariaCheck is called. This occurs on line 300 of plugin.js:

ariaCheck() {
    const toggleElement = (element, value) => {
      element
        .querySelectorAll('.tbm-toggle, .tbm-submenu-toggle')
        .forEach((toggle) => {
          toggle.setAttribute('aria-expanded', value);
        });
    };

    // Anytime there's a click outside a desktop menu that has arrows, close the menu.
    document.addEventListener('click', event => {
      if (!event.target.closest('.tbm') && !this.isMobile && this.hasArrows) {
        this.closeMenu();
      }
    });
...

I believe the specific issue is that document.addEventListener is adding an anonymous function as the handler. According to the MDN, anon functions are NOT disregarded from being added to the event listener list. So every single time ariaCheck is called, it's adding a new event listener to the document, eventually crashing the page.

This was added in 3.0.0-alpha 3 and is NOT present in alpha2. Rolling back to alpha2 addresses the immediate issue.

Thanks for the reroll! The #38 patch is now applying cleanly to the latest from the 2.x dev branch.

I worked on this at #portland2024 & updated the MR to fix another few coding standard items.

SuperfluousApostrophe → made their first commit to this issue’s fork.

Something was updated in the 8.x-2.x-dev branch today and the patch in #32 is now broken. Any chance the patch can be re-rolled?

I have a related (if not ultimately the same) issue. I have a view with exposed filers and infinite scroll. When any of the filters are clicked, the results are "correct", but I get something like 30-60 identical ajax calls. The patch in #7 does not seem to affect the behavior at all in Drupal 9.5.9.

The patch in #7 worked great for me. Prior to application, my rest endpoints worked, but I couldn't upload anything into my media library.

The solution by @Loyarc in #6 did it for me. It would be nice for the module to suggest some options such as temporary:// or public://.

Duplicates https://www.drupal.org/project/ip_consumer_auth/issues/2942049 🐛 Prevents normal website access by restricting all routes by default Needs work

After debugging, I found that the cause of my issue seems to be the same in #2942049 🐛 Prevents normal website access by restricting all routes by default Needs work . The patch in #7 is working for me.

SuperfluousApostrophe → created an issue.