Problem/Motivation
The Drupal.behaviors.moduleFilterModulesTabs behaviour should only be ran once, otherwise anytime there's a new ajax request on the page e.g. if the Automated Logout module is enabled and doing its check, the behaviour is ran indefinitely.
Resulting in nested tabs:

Steps to reproduce
Enable the Automated Logout module and wait for an ajax request to be called.
Proposed resolution
Apply the appropriate behaviours once on the parent page.
The context variable should also be used during jQuery calls to ensure that the code works with AJAX requests.
Remaining tasks
Update javascript behaviours to support AJAX requests.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | module-filter-modules-tabs-3186276-3.patch | 870 bytes | teodyseguin |
| module_filter-ajax-behaviour-loop.png | 20.27 KB | codebymikey |
Issue fork module_filter-3186276
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
teodyseguinComment #3
teodyseguinI have made a patch here.
Comment #4
teodyseguinComment #8
smustgrave commentedThanks for the fix!
Comment #11
smustgrave commentedReverting changes as they seemed to break on D10
Comment #12
smustgrave commentedComment #13
smustgrave commentedWonder if this can be retested on 4.x? Not seeing the issue anymore?
Comment #14
smustgrave commentedComment #15
teodyseguin