From 4d9cdf122b0ea5b5443d6cf5564c02d9270b4f64 Mon Sep 17 00:00:00 2001 From: Jakob Hilden Date: Wed, 9 Apr 2014 18:20:50 +0200 Subject: [PATCH] Bugfix for accidentally closing all accordion panels When multiple accordion panels are expanded/active (possbile when `multi_expand: true`) clicking on one of the the active panels would close **all** other panels instead of ony the current one. --- js/foundation/foundation.accordion.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/js/foundation/foundation.accordion.js b/js/foundation/foundation.accordion.js index 21480b8fa..3e21303eb 100644 --- a/js/foundation/foundation.accordion.js +++ b/js/foundation/foundation.accordion.js @@ -27,14 +27,13 @@ siblings = S('dd > .content', accordion), aunts = $('dd', accordion), settings = accordion.data(self.attr_name(true) + '-init'), - active_content = S('dd > .content.' + settings.active_class, accordion), - active_parent = S('dd.' + settings.active_class, accordion); + active_content = S('dd > .content.' + settings.active_class, accordion); e.preventDefault(); if (! S(this).closest('dl').is(accordion)) { return; } if (settings.toggleable && target.is(active_content)) { - active_parent.toggleClass(settings.active_class, false); + target.parent('dd').toggleClass(settings.active_class, false); return target.toggleClass(settings.active_class, false); } -- 2.47.2