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

Add dual functionality of ignoreClass #67

Open
Marenek opened this issue Nov 23, 2021 · 4 comments
Open

Add dual functionality of ignoreClass #67

Marenek opened this issue Nov 23, 2021 · 4 comments

Comments

@Marenek
Copy link

Marenek commented Nov 23, 2021

I have used draggable items in fix envelopes with set ignoreClass. After the jquery-sortable-lists upgrade it has stopped working and I have to freeze it to old version.

What about some option to define ignoreClass behavior?

Or allow to define ignoreClass and ignoreClosestClass or on other way ignoreClass and ignoreWithClass. Simply two conditions in code.

@camohub
Copy link
Owner

camohub commented Nov 30, 2021

IgnoreClass has to be set on all elements which should be ignored. It means its not enough to set it on parent element, it has to be set on all child element.

It is not about two conditions. It is about the chain of all element to the root element although there is no ignoreClass set.

@Marenek
Copy link
Author

Marenek commented Nov 30, 2021

Sorry, do you really know what "closest()" does?

@camohub
Copy link
Owner

camohub commented Dec 7, 2021

What you think I dont know about closest()?
It seems you dont understand what it means if there is NO ignoreClass in the chain of elements.

@Marenek
Copy link
Author

Marenek commented Dec 8, 2021

Ok, both of us understand what closest() does. I have prepared examples of my case to explain it better. Look at https://www.kabinet.cz/stuff/html/jquery-sortable-lists/ please. There are two pages with the same code but different versions of script.

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