Anchor tags in megamenu cause menus to close about smartmenus HOT 3 CLOSED

Sorry for the late reply! It's been very hectic times here lately. :(

I can see the issue you are facing. To be honest, I am hesitant if it's a good idea to change the behavior for links inside mega dropdown sub menus since in many cases the way it currently works would still be the expected behavior (i.e. hide the sub menu on click even for links in mega dropdowns). So I would suggest you to use a simple additional fix:

// do not hide the mega dropdown sub menu if a link inside it is clicked
$(document).on('click', 'ul.mega-menu a', function(e) {
    e.stopPropagation();
});

It will not hurt your configuration if this behavior is changed in the script core in the future and you decide to upgrade.

from smartmenus.

I tried stopping propagation but it disabled bootstrap's elements relying on clicks too (naturally). I did change the code in the plugin to use an and operator (&&) from an or (||) and this seemed to do the trick nicely. I am aware changing plugins is a bad idea but in the interest of time this seemed to solve everything nicely and I documented the change in our own version of the plugin.

If you're interested, I just changed:

if (this.visibleSubMenus.length && !$.contains(this.$root[0], e.target) || $(e.target).is('a')) {

to:

if (this.visibleSubMenus.length && !$.contains(this.$root[0], e.target) && $(e.target).is('a')) {

I appreciate the help though, thank you vadikom.

from smartmenus.

I am not sure if you tested with the exact code I posted, but I believe it should work correctly with all Bootstrap JS components. I tested putting an accordion in a mega dropdown:

http://getbootstrap.com/javascript/#collapse-example-accordion

and the clicks work just fine for the accordion and do not hide the sub menu.

The change you've done means that the sub menus will be hidden only when a link on your page that is not contained in your menu tree is clicked.

Anyway, thanks for your input! Cheers!

from smartmenus.