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

Key Modifiers are ignored in Forge > 47.1.22 #9806

Closed
djnifos opened this issue Nov 28, 2023 · 11 comments
Closed

Key Modifiers are ignored in Forge > 47.1.22 #9806

djnifos opened this issue Nov 28, 2023 · 11 comments

Comments

@djnifos
Copy link

djnifos commented Nov 28, 2023

Minecraft Version: {Minecraft version}
1.20.1

Forge Version: {Forge version. Version number, not latest/rb}
Any version from 47.1.23 -> 47.2.8

Logs: {Link(s) to GitHub Gist with full latest.log and/or crash report}
https://gist.github.com/djnifos/ffb0da53ad6174f83190d246011b4799

Steps to Reproduce:

  1. Bind a keybind to a modifier + a key
  2. Press the key without the modifier
  3. Action happens anyways

Description of issue:
The game ignores the modifier keys missing. Tested shift, control, and alt. Behavior does not exist in 47.1.22 or previous. This reduces available keybinds to users which impacts playability in large packs.

The debug log doesn't show anything from what I could tell but it's included.

@djnifos djnifos added the Triage This request requires the active attention of the Triage Team. Requires labelling or reviews. label Nov 28, 2023
@LexManos
Copy link
Member

Unable to reproduce in any of those versions using any modifier. Tried a simple setup, making move forward require a modifier and it worked as expected.

@autoforge autoforge bot removed the Triage This request requires the active attention of the Triage Team. Requires labelling or reviews. label Nov 28, 2023
@djnifos
Copy link
Author

djnifos commented Nov 28, 2023

Should I make a video? I did open achievements with Alt-G Ctrl-G and Shift-G. Just G will open it

@LexManos
Copy link
Member

LexManos commented Nov 29, 2023

Ah, so this only effects things that use the keybindings in a certain way. Which happens to be the way the advancement screen uses it, but not the way the movement keys use it. Which explains why my tests didn't work. Which is also why explicit instructions are appreciated.

@djnifos
Copy link
Author

djnifos commented Nov 29, 2023

I had verified quite a few different ones in a full pack, then only the advancements with just forge installed. I will be more explicit in the future, should I need to file any more. Thanks!

@Eragonawesome
Copy link

I am having this same issue in version 47.2.20 in the All The Mods 9 To The Skies pack version 1.0.6

To reproduce:
Bind shift to sprint
Bind E to inventory
Bind open wireless crafting grid to Shift + E
Press the E key on its own to open your inventory
Close your inventory
Press the shift key on its own
The wireless crafting grid opens if you have one equipped, or puts a message in chat telling you that you don't have one equipped

@LexManos
Copy link
Member

The key events work fine on our end.
There may be some mods who relied on broken behavior, so you can ask them. I would bet that whatever provides the 'wireless crafting grid' is doing their conflict resolver incorrectly. Not an issue on my end.

@ninjomoe
Copy link

im just gonna post it here just in case it helps, this bug is still happening on all refined storage versions for 1.20.1 with forge version 47.3.5

It didnt happen on 1.18 or 1.19
And it doesnt happen on Forge Version 47.1.22

@ninjomoe
Copy link

ok so i tested it on every forge version on 1.20.1.

The Ctrl + E interaction for the refined storage wireless terminal works as usual on forge version 47.2.8 and doesnt work for 47.2.14 and any version above.

@ninjomoe
Copy link

I FOUND A MOD THAT FIXES THIS!!!

https://www.curseforge.com/minecraft/mc-mods/key-binding-patch

This works on 1.20.1 on the newest forge version 47.3.5 and the newest refined storage version 1.12.4

@LexManos
Copy link
Member

For future note, the few cases that were brought up here, the issue is the modder and their mod having invalid conflict contexts. Most cases they just blindly set their context to always inactive, so obviously Forge never calls their keybinding.

@ajaxnotmeta
Copy link

ajaxnotmeta commented Dec 7, 2024

For future note, the few cases that were brought up here, the issue is the modder and their mod having invalid conflict contexts. Most cases they just blindly set their context to always inactive, so obviously Forge never calls their keybinding.

may i ask? why every mods on Forge hapen to have this issue? this drives me nuts! heck Even Project Architect 2 and ATM9 have this same issue
edit : i might need to dig deeper as it seems more strange, using same mods(sophisticated Backpacks, Curios and "On a Stick") i het the same bug when directly running the game on 47.3.12(Currently lastest i think?) but when i tried to run the game on 47.1.22 then 47.2.14 and lastly on 47.3.12 the issue was gone.
*do keep in mind im not developer, but im tryng to find a way to replicate the issue here

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

No branches or pull requests

5 participants