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

[4.0 a1] Input map keys wiped when upgrading from Godot 3 #57311

Closed
Zireael07 opened this issue Jan 27, 2022 · 5 comments · Fixed by #71753
Closed

[4.0 a1] Input map keys wiped when upgrading from Godot 3 #57311

Zireael07 opened this issue Jan 27, 2022 · 5 comments · Fixed by #71753

Comments

@Zireael07
Copy link
Contributor

Godot version

4.0 alpha 1

System information

Linux Manjaro, Vulkan, Intel Kaby Lake

Issue description

Title.
They're set to "Physical" with no key associated.

Steps to reproduce

Import a project from Godot 3, I guess? (Running it once may or may not be required)

Minimal reproduction project

N/A

@Calinou
Copy link
Member

Calinou commented Jan 27, 2022

I can confirm this on master fc09d78.

@Zireael07
Copy link
Contributor Author

Same thing applies to Shortcut property of the Button (as in, a Control type, not in a physical button on the keyboard :P )

The bigger the project, the more annoying this is going to be.

@EricEzaM
Copy link
Contributor

EricEzaM commented Feb 9, 2022

Yeah, InputEventKey changed slightly in 4.0, as did Shortcut.

Should this really be handled by some engine code or should some modification be made to one of those project converters floating around?

See #51273, #43662 and a couple of others to do with implementation of the "Physical" key support in InputEventKey.

For InputEventKey, it should just be setting the store_command property to true, plus possibly some other minor changes. For Shortcut, it will just be putting the existing event from 3.X into an array of events for 4.0, as shortcuts can now have more than one event assigned.

@Zireael07
Copy link
Contributor Author

I'd go for modifying one of the project converters, AND shipping said converter with the next alpha.

@KoBeWi
Copy link
Member

KoBeWi commented May 1, 2022

For InputEventKey, it should just be setting the store_command property to true, plus possibly some other minor changes. For Shortcut, it will just be putting the existing event from 3.X into an array of events for 4.0, as shortcuts can now have more than one event assigned.

Also it needs to change scancode to keycode.

I wouldn't bother with this though. As already said, this should be handled by converters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants