From 47cbdeaf838e274b4f7d17f0063ba0a6531f2c27 Mon Sep 17 00:00:00 2001 From: Chris Oyler Date: Thu, 21 Jan 2016 10:12:47 -0800 Subject: [PATCH] fixes #7951, also added benefit of simplifying the jquery selector chain to manage aria state for nested menus and remove a few lines of unnecessary code --- js/foundation.accordionMenu.js | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/js/foundation.accordionMenu.js b/js/foundation.accordionMenu.js index a249a68ea..4a4a082da 100644 --- a/js/foundation.accordionMenu.js +++ b/js/foundation.accordionMenu.js @@ -237,13 +237,10 @@ _this.$element.trigger('up.zf.accordionMenu', [$target]); }); }); - $target.attr('aria-hidden', true) - .find('[data-submenu]').slideUp(0).attr('aria-hidden', true).end() - .parent('.is-accordion-submenu-parent') - .attr({'aria-expanded': false}); - // $target.slideUp(this.options.slideSpeed, function() { - // $target.find('[data-submenu]').slideUp(0).attr('aria-hidden', true); - // }).attr('aria-hidden', true).parent('.has-submenu').attr({'aria-expanded': false}); + + var $menus = $target.find('[data-submenu]').slideUp(0).addBack().attr('aria-hidden', true); + + $menus.parent('.is-accordion-submenu-parent').attr('aria-expanded', false); }; /** -- 2.47.2