- Issue created by @brockfanning
- πΊπΈUnited States jnicola
We were able to fix this by adding ".text-formatted" selector to the accordion JS selectors, restricting this functionality in theory to just the WYSIWYG content. It worked for us, but we can't guarantee the selector will apply to your situation, and may not warrant a PR since the class can't for sure be relied on.
- πΊπΈUnited States brockfanning
We've noticed already that the selector in #2 can miss certain scenarios, like accordions created in webforms. Probably the ideal code would be for this code to somehow target only those accordions that don't already have the "aria-controls" and "id" attributes that it is adding.
-
smustgrave β
committed 37423758 on 3.1.x
Issue #3473162 by jnicola, brockfanning, smustgrave: Accordions outside...
-
smustgrave β
committed 37423758 on 3.1.x
- πΊπΈUnited States smustgrave
Tweaked #2 to not update aria or id if they are already there.
- πΊπΈUnited States brockfanning
@smustgrave I was wondering if the jQuery selector itself could maybe specify elements without the attributes. We noticed that in some cases the "text-formatted" class cannot be relied on. For example, in webforms.
Automatically closed - issue fixed for 2 weeks with no activity.