re-initialization of Bootstrap addon about smartmenus HOT 6 CLOSED

Hi and thanks very much for your input! 👍

Indeed, in the current version the addon is initialized just once ondomready and any navbars that are added in the DOM dynamically after that won't be initialized. So it really makes sense to rework the addon and expose some init method that could be called at any time. I will try to do this in the next few days and will post back. Stay tuned..

from smartmenus.

That'd be great! Thank you so much.

from smartmenus.

OK, I just committed the updates. You can grab the code from here (note that the core also needed some changes so it's a good idea to use the updated one below to avoid any issues):

https://raw.githubusercontent.com/vadikom/smartmenus/master/src/addons/bootstrap/jquery.smartmenus.bootstrap.js
https://raw.githubusercontent.com/vadikom/smartmenus/master/src/jquery.smartmenus.js

All updates will be included in the next release so you don't need to worry about upgrading in the future.

You can now call $.SmartMenus.Bootstrap.init() when your navbar is available in the DOM. It's not a problem to call it multiple times, it will only handle navbars that are still not initialized.

Note that the following methods might also be handy (you can use them just fine with the Bootstrap addon, just set some id to your ul.navbar-nav so you can call them for it):

http://www.smartmenus.org/docs/#refreshInstance
http://www.smartmenus.org/docs/#destroyInstance

from smartmenus.

Awesome! Thank you so much!

FYI, the dists look like they weren't built for the last commit.

from smartmenus.

Tested, works like a champ for me! Thank you again! 🍻

from smartmenus.

Np at all! This will surely be useful to others so I really appreciate your input!

I only build the dists for new versions and I have planned some other fixes too for the new release so it will wait for now (that's why I pointed you to the "src" folder).

OK, I'm closing this.. Cheers!

from smartmenus.