Skyblocker icon indicating copy to clipboard operation
Skyblocker copied to clipboard

Shift-left clicking slots in the fancy crafting screen moves items from old crafting screen with InventoryProfileNext installed

Open cvrunmin opened this issue 10 months ago • 8 comments

What happened?

~Yes,~ With InventoryProfileNext installed, if you shift-clicking the 5th slot on the 2nd row of inventory slots in the fancy crafting screen, the "Close" item from the actual crafting screen is "moved" to the hotbar and hence, close the screen. You trigger the "Back" item if you open the crafting screen from Skyblock menu.

If you hold shift-left click and drag from the 2nd or 3rd row of crafting slots to outside of slots, you move the glass pane to the inventory. You also move the glass pane to the hotbar if shift-left clicking the 1st and 2nd row of inventory slots, except 2nd row 5th slot which is close/back item. These should be caused by the same bug based on the reproducing logic.

EDIT: amending some wordings as it is found to be a mod compatibility issue.

Screenshots

No response

Log output

No response

Minecraft Version

1.20.4

Skyblocker Version

skyblocker-1.20.0+1.20.4.jar

Additional context

Full mod list:

  • aaron-mod-1.2.2
  • architectury-11.1.17
  • chatpatches-204.6.2
  • citresewn-1.1.4 (dicedpixels' fork)
  • cloth-config-13.0.121
  • durabilityviewer-1.20.4-fabric0.91.2-1.10.5
  • edgelesschatscreen-1.20-1.0.2
  • entityculling-fabric-1.6.4-mc1.20.4
  • fabric-api-0.96.11+1.20.4
  • fabric-language-kotlin-1.10.19+kotlin.1.9.23
  • ferritecore-6.0.3
  • gamma_shifter-1.0.4
  • ImmediatelyFast-1.2.10+1.20.4
  • InventoryProfilesNext-1.20.3-1.10.10
  • iris-1.6.17
  • libIPN-1.20.3-4.0.2
  • lithium-mc1.20.4-0.12.1
  • modmenu-9.0.0
  • ok-boomer-0.1.3+1.20.2
  • owo-lib-0.12.5+1.20.3
  • reeses_sodium_options-1.7.2+mc1.20.4-build.102
  • RoughlyEnoughItems-14.0.688-fabric
  • seamless-2.2.2-fabric-1.20.4
  • skyblocker-1.20.0+1.20.4
  • sodium-extra-0.5.4+mc1.20.4-build.116
  • sodium-fabric-0.5.8+mc1.20.4
  • tooltip-scroll-1.3.0
  • Xaeros_Minimap_24.1.1_Fabric_1.20.4
  • XaerosWorldMap_1.38.4_Fabric_1.20.4
  • yet-another-config-lib-fabric-3.3.2+1.20.4

cvrunmin avatar Apr 24 '24 13:04 cvrunmin

well that's quirky

viciscat avatar Apr 24 '24 13:04 viciscat

well that's quirky

Even more quirky, if you shift-left click an item from the inventory slot mentioned, the item is moved to the crafting table AND a glass pane is moved to the hotbar. Hope this info gives you more clues.

cvrunmin avatar Apr 24 '24 14:04 cvrunmin

I can't reproduce?? I can shift click just fine on all the slots. What mods do you have

viciscat avatar Apr 24 '24 14:04 viciscat

I can't reproduce?? I can shift click just fine on all the slots. What mods do you have

Confirmed that the issue disappear if I uninstall InventoryProfileNext-1.20.3-1.10.10 Here is the full list, if you needed:

  • aaron-mod-1.2.2
  • architectury-11.1.17
  • chatpatches-204.6.2
  • citresewn-1.1.4 (dicedpixels' fork)
  • cloth-config-13.0.121
  • durabilityviewer-1.20.4-fabric0.91.2-1.10.5
  • edgelesschatscreen-1.20-1.0.2
  • entityculling-fabric-1.6.4-mc1.20.4
  • fabric-api-0.96.11+1.20.4
  • fabric-language-kotlin-1.10.19+kotlin.1.9.23
  • ferritecore-6.0.3
  • gamma_shifter-1.0.4
  • ImmediatelyFast-1.2.10+1.20.4
  • InventoryProfilesNext-1.20.3-1.10.10
  • iris-1.6.17
  • libIPN-1.20.3-4.0.2
  • lithium-mc1.20.4-0.12.1
  • modmenu-9.0.0
  • ok-boomer-0.1.3+1.20.2
  • owo-lib-0.12.5+1.20.3
  • reeses_sodium_options-1.7.2+mc1.20.4-build.102
  • RoughlyEnoughItems-14.0.688-fabric
  • seamless-2.2.2-fabric-1.20.4
  • skyblocker-1.20.0+1.20.4
  • sodium-extra-0.5.4+mc1.20.4-build.116
  • sodium-fabric-0.5.8+mc1.20.4
  • tooltip-scroll-1.3.0
  • Xaeros_Minimap_24.1.1_Fabric_1.20.4
  • XaerosWorldMap_1.38.4_Fabric_1.20.4
  • yet-another-config-lib-fabric-3.3.2+1.20.4

cvrunmin avatar Apr 24 '24 15:04 cvrunmin

update: turning off CONTAINER_SWIPE_MOVING_ITEMS config from InventoryProfileNext will fix the issue. We can narrow down the code to inspect into here https://github.com/blackd/Inventory-Profiles/blob/537c5bcc89b1764cc630abca6654e9be1769e27e/shared-sources/src/main/java/org/anti_ad/mc/ipnext/event/ClientEventHandler.kt#L73

cvrunmin avatar Apr 25 '24 12:04 cvrunmin

Thanks for tracking down the cause to there! I will look into it but this might be hard because of kotlin shenanigans

viciscat avatar Apr 25 '24 12:04 viciscat

alright so I looked into the code and I think it's cuz it doesn't check if slot.isDisabled()... Can't find it anywhere. So I don't think we can fix it

viciscat avatar Apr 25 '24 17:04 viciscat

alright so I looked into the code and I think it's cuz it doesn't check if slot.isDisabled()... Can't find it anywhere. So I don't think we can fix it

https://github.com/Bawnorton/MixinSquared moment ? Or PR maybe

Julienraptor01 avatar Apr 25 '24 19:04 Julienraptor01

As a warning inventory sorting mods are not allowed on Hypixel (depending on the interpretation of gameplay action)

Additionally, we do also note that anything which automates any player gameplay action is strictly disallowed, be those Minecraft modifications, external software or hardware. This includes things such as (but not limited to) auto/burst clicking buttons or macros, auto-sprint, and aim assists.

https://support.hypixel.net/hc/en-us/articles/6472550754962-Hypixel-Allowed-Modifications

DonFlymoor avatar May 23 '24 15:05 DonFlymoor

looks like we cant fix it on our end. You need to open an issue at InventoryProfileNext github.

LifeIsAParadox avatar Jun 27 '24 14:06 LifeIsAParadox