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

support for multiple drag handles #2709

Merged

Conversation

nickfulcher
Copy link
Contributor

Description

Currently when options handle is defined it applies event listeners to the first instance that has the handle class. These changes add the events to all elements that has the handle class allowing for multiple drag handles with in a grid item

@@ -55,6 +55,8 @@ export class DDDraggable extends DDBaseImplement implements HTMLElementExtendOpt
protected dragElementOriginStyle: Array<string>;
/** @internal */
protected dragEl: HTMLElement;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should remove that if you are using a list instead... and fix line 146

@adumesny adumesny merged commit 790888a into gridstack:master Jun 24, 2024
adumesny added a commit to adumesny/gridstack.js that referenced this pull request Jun 24, 2024
* more for gridstack#2709
* corerctly removed prev dragEl code and use list
@adumesny
Copy link
Member

fixed your incorrect code... didn't get to test touch devices, hopefully didn't break e.target case....

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

Successfully merging this pull request may close these issues.

2 participants