Feature 208: Drag and Drop to Ready/Torpor Region#209
Open
g4ttsu wants to merge 4 commits intoVTES-JOL:masterfrom
Open
Feature 208: Drag and Drop to Ready/Torpor Region#209g4ttsu wants to merge 4 commits intoVTES-JOL:masterfrom
g4ttsu wants to merge 4 commits intoVTES-JOL:masterfrom
Conversation
added 3 commits
February 19, 2026 13:53
- added drag icon to cards created in Ready and Torpor - added updateRegion Method in DeckserverRemote that swapes places for 2 cards in a region - added jquery drag and drop to li items in Ready/Torpor region in the ds.js - added min touch punch js to main.jsp
- added drop-zone to ol in cards.jsp - rename updateRegion to swapCardsInRegion - added detach card to Region - added attach card to card - added more sortable features to ds.js. Drop Zone allows li's to get dropped on ol - added addCard to RegionData with position
- added player restriction to drag and drop
Contributor
Author
|
this escalated a bit after the first commit I started to add drag and drop fuctionality to all branches a card can have in the Ready region. So now a card can be moved from 1 -> 2 but as well from 1.1 -> 1 and 1-> 1.1.1 and so on. I think logging needs to be added as well for better reference what who moved when. I tested this a bit but I guess this is a bit of a dangerous feature because cards get moved/removed/added to various places. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Feature #208: Added an Icon to the Cards in the Ready/Torpor Region to easily drag and drop those around in the same regions. I need to add "jquery.ui.touch-punch.min.js" otherwise it only works on desktop. I hope thats not an issue. Added jquery drag and drop to the relevant list items after droping an item the region will be updated with the new sort order by swaping 2 card items.