Skip to content

Adding functionality to copy and paste nodes with card cloner.#316

Open
Cyphecx wants to merge 5 commits intoDirewolf20-MC:1.20.1from
Cyphecx:1.20.1
Open

Adding functionality to copy and paste nodes with card cloner.#316
Cyphecx wants to merge 5 commits intoDirewolf20-MC:1.20.1from
Cyphecx:1.20.1

Conversation

@Cyphecx
Copy link

@Cyphecx Cyphecx commented Nov 17, 2024

Using LaserIO, I really felt it was missing the ability to copy entire nodes. Having to copy and paste to each card individually was really holding back the scalability of a LaserIO contraption.

So I added the ability to save a node configuration to the card cloner and paste that data back to other nodes.

Shift-Right clicking a node with a card cloner will save card info to the cloner item which can then be pasted on other nodes with a right click. This uses the same reach distance as the wrench. Shift-Right clicking will clear the copied node(but not the saved card, did not want to affect the existing logic)

laseriodemomain.mp4

Does not copy node connections.

The cloner will check the user's card holder to make sure they and the node combined have enough items to paste the configuration.

laseriodemofailure.mp4

Added the dimension and xyz of copied node to the tooltip.
image

Extra cards are placed in the card colder, and if theres no room, dropped for the player to pickup.

laseriodemodrop.mp4

Even though the card pasting functionality tries to make sure there is enough room, there are still scenarios it will end up dropping items, so I allowed node pasting to drop items as well.

I think theres still some more room for refinement, but the current state will bring a much needed functionality to users of LaserIO.

@Cyphecx Cyphecx marked this pull request as ready for review November 17, 2024 23:57
@ItsYoshover
Copy link

Hello, I was playing around with this and noticed that when one does not have a card holder, the items are always voided. To be more specific when trying to copy a node WITH items TO one without, no cards were put into my inventory. The card holder overflow protection was working as intended. I will be looking into this but I thought I would let you know.

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