Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

To Allow for multiple FAQs on the same page with different filter categories #19

Closed
ARChilton opened this issue Apr 16, 2020 · 1 comment

Comments

@ARChilton
Copy link

ARChilton commented Apr 16, 2020

allFAQs = $('.qe-faq-toggle, .qe-faq-list, ol.qe-faqs-index-list li');

If you change the line above to:

allFAQs = $(this)
.closest('.qae-faqs-container')
.find('.qe-faq-toggle, .qe-faq-list, ol.qe-faqs-index-list li')

I think this will solve the issue I found whereby two FAQs on the same page will conflict with each other. This traverses up to the FAQ's container then finds the classes rather than finds them across the whole DOM.

@sungraiz
Copy link
Contributor

Hi,

Thank you for pointing out and providing the issue fix. We'll definitely look into it while working on the next update.

Regards,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants