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

New SOURCE Group and TARGET Group attributes for quick identification of Dragged Item, and Filtering (wire:sortable-group.filter-updated) #60

Closed
wants to merge 2 commits into from

Conversation

markusyesloop
Copy link

To make it faster to identify the Item dragged and its Source and Target groups, I've modified the sortable:stop event

This is a non-breaking change -- it only adds additional attributes

NOTE: If Group has Source and Target attributes as True, it means the Item was dragged within that group, rather than to another group

Adding wire:sortable-group.filter-updated attribute will filter only groups updated by drag

https://user-images.githubusercontent.com/20057194/206751855-2687fb1b-9a7c-4545-b81a-e6be7b9df2ca.png

To make it faster to identify the Item dragged and its Source and Target groups, I've modified the sortable:stop event

This is a non-breaking change -- it only adds additional attributes

NOTE: If Group has Source and Target attributes as True, it means the Item was dragged within that group, rather than to another group

Adding wire:sortable-group.filter-updated attribute will filter only groups updated
@calebporzio
Copy link
Contributor

Thanks for contributing!

In order to keep this project as focused and maintainable as possible, I'm going to close this pull request.

This isn't "no", it's just "no for now".

If you disagree, or there's further demand from the community, please re-open this PR and we'll reconsider!

@markusyesloop
Copy link
Author

I think this is a needed pull request, because the enhancements make it helpful for devs to know if the Dragged item has moved within the SAME GROUP or to ANOTHER GROUP without having to roll this functionality themselves

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants