]> git.ipfire.org Git - thirdparty/foundation/foundation-sites.git/commitdiff
Fix dropdown destroy method 9381/head
authorDinis Correia <diniscorreia@mac.com>
Wed, 16 Nov 2016 13:58:10 +0000 (13:58 +0000)
committerDinis Correia <diniscorreia@mac.com>
Wed, 16 Nov 2016 13:58:10 +0000 (13:58 +0000)
This fixes issue #9339. When destroying a dropdown menu, the dropdown
classes wouldn’t be removed if using a different class structure (for
instance, using custom class names instead of `.menu`). Previous
versions selected all child elements of the menu but on the latest
version only direct `li` childs or  elements with `.menu` class

This commit ensures `ul` elements also get the classes removed, for
those cases where custom classes are used.

js/foundation.util.nest.js

index 82603a790b316230357967da42cd3220816305f5..67031555d41051d04c7c02259cdbe5f6a38eebae 100644 (file)
@@ -63,7 +63,7 @@ const Nest = {
         hasSubClass = `is-${type}-submenu-parent`;
 
     menu
-      .find('>li, .menu, .menu > li')
+      .find('>li, > li > ul, .menu, .menu > li')
       .removeClass(`${subMenuClass} ${subItemClass} ${hasSubClass} is-submenu-item submenu is-active`)
       .removeAttr('data-submenu').css('display', '');