Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

contextMenu with bootstrap 4 #137

Open
danielKatagiri opened this issue May 25, 2018 · 0 comments
Open

contextMenu with bootstrap 4 #137

danielKatagiri opened this issue May 25, 2018 · 0 comments

Comments

@danielKatagiri
Copy link

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.

@danielKatagiri danielKatagiri changed the title contextMenu with bootstrap contextMenu with bootstrap 4 May 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant