Skip to content

Fix item transfer - #51

Merged
Dream-Master merged 7 commits into
masterfrom
fix-item-transfer
May 9, 2026
Merged

Fix item transfer#51
Dream-Master merged 7 commits into
masterfrom
fix-item-transfer

Conversation

@Kogepan229

Copy link
Copy Markdown
Member

Fixes GTNewHorizons/GT-New-Horizons-Modpack#24320

I couldn't find any valid reason why Mouse Tweaks was disabled in this GUI

@Kogepan229
Kogepan229 requested a review from a team April 19, 2026 11:04
@PLASMAchicken

Copy link
Copy Markdown
Member

I couldn't find any valid reason why Mouse Tweaks was disabled in this GUI

8d02e38

Turn off mousetweaks in the WCT gui.
It'd be better to fix it, but I can't figure out why the ME slots behave weird here so this is an easy patch for now.

fixes GTNewHorizons/NotEnoughItems#88
fixes GTNewHorizons/GT-New-Horizons-Modpack#8828
fixes GTNewHorizons/GT-New-Horizons-Modpack#6388
fixes GTNewHorizons/GT-New-Horizons-Modpack#11079

Seems like it removed the infinity booster card or other slots when scrolling/crafting

@lc-1337

lc-1337 commented Apr 19, 2026

Copy link
Copy Markdown

i guess getValidDestinationSlots can fix that

@Kogepan229

Copy link
Copy Markdown
Member Author

yes, that problem does not occur because booster slot is not included in result of getValidDestinationSlots.
but thanks for checking why it was disabled.

@lc-1337

lc-1337 commented Apr 19, 2026

Copy link
Copy Markdown

rn only trash slot excluded in that method

@Kogepan229

Copy link
Copy Markdown
Member Author

ah yes, I was stupid.
However, for some reason, the scrolling/crafting issue does not occur now.

@lc-1337 lc-1337 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nodoby forgot 100%

@Dream-Master
Dream-Master merged commit a79995a into master May 9, 2026
1 check passed
@Dream-Master
Dream-Master deleted the fix-item-transfer branch May 9, 2026 12:35
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.

Wireless Crafting Terminal tweaks

4 participants