[0.17.0][keybind] Picking up items from ground doesn't work (F bound to pickup/enter/exit)

This subforum contains all the issues which we already resolved.
kovarex
Factorio Staff
Factorio Staff
Posts: 8078
Joined: Wed Feb 06, 2013 12:00 am
Contact:

Re: [0.17.0][keybind] Picking up items from ground doesn't work (F bound to pickup/enter/exit)

Post by kovarex »

ManaUser wrote:
Thu Mar 14, 2019 7:00 pm
Twinsen wrote:
Thu Mar 14, 2019 4:07 pm
Fixed in Version: 0.17.12.
Thanks.
Out of curiosity, since I'm the kind of guy who likes to customize controls significantly sometimes, can you provide any more details on how this is handled now? Like what sort of actions I can expect to be able to combine safely?
It was a general change that key action doesn't block other key action with exactly the same control.

In other words, Control + C block later usage of C (as before), but only becuase Control + C control is actually different. If Control +C was also set to shooting, it should activate both.

ManaUser
Fast Inserter
Fast Inserter
Posts: 225
Joined: Sat Aug 12, 2017 9:41 pm
Contact:

Re: [0.17.0][keybind] Picking up items from ground doesn't work (F bound to pickup/enter/exit)

Post by ManaUser »

kovarex wrote:
Thu Mar 14, 2019 9:55 pm
It was a general change that key action doesn't block other key action with exactly the same control.

In other words, Control + C block later usage of C (as before), but only becuase Control + C control is actually different. If Control +C was also set to shooting, it should activate both.
I see! Thanks for the explanation.

I'm sure you've given this more thought that I have, but that seems like a good change. As long as the default controls are sensible, it's probably better to allow stupid combinations (like copy and shooting) rather than get in the way of reasonable ones (like pick-up and enter vehicle). I expect people who remap controls generally know what they're doing, or at least understand the risks. So letting them do whatever they want even if it might not be a good idea is the way to go, IMHO.

Twinsen
Factorio Staff
Factorio Staff
Posts: 1330
Joined: Tue Sep 23, 2014 7:10 am
Contact:

Re: [0.17.0][keybind] Picking up items from ground doesn't work (F bound to pickup/enter/exit)

Post by Twinsen »

ManaUser wrote:
Thu Mar 14, 2019 7:00 pm
Out of curiosity, since I'm the kind of guy who likes to customize controls significantly sometimes, can you provide any more details on how this is handled now? Like what sort of actions I can expect to be able to combine safely?
Honestly, it's so complicated not even we understand sometimes. :)
We have 3 main mechanisms:
1. input actions triggered by input events. When an event is triggered, we go through a priority list.
If an item in the list is valid for the input event and the event is valid for the game state, the event is consumed and the actions below are ignored.
I attached a text file with the code we define the list.
Now there are some rules.
- Sometimes it's hard to detect if an input is valid. For example the event for Enter Vehicle will always be consumed, since at this part of the code we don't search for entities to see if the player is near a vehicle.
- If the key combination is the same as a key combination that is used in the GUI, and the mouse is over a GUI, then the action will not be triggered.
2. GUI actions. These are shortcuts that are used in the GUI such are inventory transfer shortcuts, craft X, cancel craft, open item. If the mouse is over a gui and the corect key combination is active, these action trigger.
3. input state based on the state of the keyboard/mouse(e.g. W is pressed down so we walk up).
This includes mining, building, walking, picking up items, shooting, repairing. These don't generally conflict with any of the above shortcuts.

There are way more rules but hopefully this makes sense.
Attachments
input.txt
(6.98 KiB) Downloaded 60 times

RenKuro
Manual Inserter
Manual Inserter
Posts: 4
Joined: Sun Aug 12, 2018 7:54 pm
Contact:

Re: [0.17.0][keybind] Picking up items from ground doesn't work (F bound to pickup/enter/exit)

Post by RenKuro »

Thanks for fixing the issue :)

Post Reply

Return to “Resolved Problems and Bugs”