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

Custom CSS class namespace option? #27

Closed
cossssmin opened this issue Oct 2, 2016 · 2 comments
Closed

Custom CSS class namespace option? #27

cossssmin opened this issue Oct 2, 2016 · 2 comments

Comments

@cossssmin
Copy link

First of all: very nice plugin, I love that it's lightweight and has lots of options.

Question: would you consider an option for custom class names when instantiating the menu?

iw- is fine for most users, but one might need a custom namespace, in order to have the generated menu classes synced with existing code/namespacing in their app.

Thanks!

@cossssmin
Copy link
Author

Whatever, I guess this is not maintained anymore.

Closing this, I gave up on using the plugin when I went through it's code and saw how it adds HTML markup on page load for every element that it was triggered on (what if you have a page with 100 elements that you want to use the contextmenu on? you get 100 <ul>s injected when you visit the page...)

@s-yadav
Copy link
Owner

s-yadav commented Oct 18, 2016

@hellocosmin Yes, I know some parts needs a rewrite for better performance.
But I am quite busy with other priorities this days. Maybe you can contribute here for those changes?

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

2 participants