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

Fixed key names so the key set menu now works. #5

Closed
wants to merge 9 commits into from
Closed

Fixed key names so the key set menu now works. #5

wants to merge 9 commits into from

Conversation

adamnew123456
Copy link

Somebody forgot to update the other half of the names in keycode.cpp. I corrected to their proper Irrlicht names.

@adamnew123456
Copy link
Author

@TMinusPaperclip Thanks for that - I replaced some other oversights as well.

@dannydark
Copy link
Contributor

Hi Adam, Thanks for this commit fixed a few issues I was having on my windows system when changing controls.

Also I think you missed KEY_RETURN ;)

@adamnew123456
Copy link
Author

@dannydark Thanks for the notice - that appears to be all of them.

@celeron55
Copy link
Member

Is this patch still valid and useful?

@dannydark
Copy link
Contributor

Is for my windows system (not had the same problems with key binding on linux) I merged this with the latest dev build a few days ago and seems to fix some key binding issues for me.

@TMinusPaperclip
Copy link

Yes, it's still useful.

@adamnew123456
Copy link
Author

Should be - it'll probably be better to copy https://github.com/adamnew123456/minetest/blob/master/src/keycode.cpp over the head and commit that than to try to merge the whole thing.

This was referenced Feb 17, 2016
leonardoguidoni pushed a commit to leonardoguidoni/minetest that referenced this pull request Nov 13, 2019
evanslify pushed a commit to uccuhacker/minetest that referenced this pull request Sep 5, 2020
@gpcf gpcf mentioned this pull request Feb 10, 2021
JosiahWI pushed a commit to JosiahWI/minetest that referenced this pull request Jan 12, 2023
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.

None yet

4 participants