Templarian / ui.bootstrap.contextMenu

AngularJS Bootstrap UI Context Menu
MIT License
259 stars 127 forks source link

contextMenu with bootstrap 4 #137

Open danielKatagiri opened 6 years ago

danielKatagiri commented 6 years ago

I have struggled a little to get a satisfactory visual result when using the 'enabled: false' property. From what I understood, the original code add the 'disabled' class on the <li> tag, but bootstrap 4 works better when this class is added to the <a> tag.

So I changed the contextMenu.js at the line 558: from $li.addClass('disabled'); to $li.children().addClass('disabled');

Once inside there's only one <a> tag inside each <li> tag, is worked. But for production, this is not a good solution, changing the source code. Is there a better way to set this proprety and get a good result visually?

Thanks in advance.