riksweeney / edgar

A 2D platform game with a persistent world. When Edgar's father fails to return home after venturing out one dark and stormy night, Edgar fears the worst: he has been captured by the evil sorcerer who lives in a fortress beyond the forbidden swamp.
https://www.parallelrealities.co.uk/games/edgar
116 stars 28 forks source link

Wrong controls after upgrading #37

Open musuruan opened 5 years ago

musuruan commented 5 years ago

After upgrading from v1.29 to v1.31 the configuration of the following controls are wrong: up, down, left, right, attack, block. Other controls seem fine.

riksweeney commented 5 years ago

Thanks, I suspect it's something to do with the upgrade to SDL2. I'll see if there's a way to update the controls.

lesleyrs commented 1 year ago

This is still a problem in master on Linux, using default US keyboard layout. Those 6 controls were using accented characters.

riksweeney commented 1 year ago

I cannot reproduce this, I'm afraid. What are the accented characters? In your config file ($HOME/.parallelrealities/edgar/config) what is the value of the number proceeding "UP" (should be on line 4)

lesleyrs commented 1 year ago

I realize what the problem is, the edgar build from apt creates a broken config file. So if you remove it and build master you still have the broken config.

CONTROLS LEFT 276 RIGHT 275 UP 273 DOWN 274 JUMP 32 ATTACK 306 BLOCK 308 INVENTORY 122 PREVIOUS 44 NEXT 46 ACTIVATE 97 INTERACT 99 PAUSE 112 DEAD_ZONE 6000 GAME_SETTINGS AUDIO 1 SFX_VOLUME 10 MUSIC_VOLUME 8 HINTS 1 FULLSCREEN 0 AUDIO_QUALITY 22050 FONT SMALL_FONT_SIZE 14 LARGE_FONT_SIZE 26

riksweeney commented 1 year ago

Sorry, I don't follow. Is the apt build one of the binaries you can download from here, or is it one of the build scripts in the repository?

lesleyrs commented 1 year ago

I installed the edgar package from the apt repository, which created the invalid config file above (it still uses sdl 1.2).

After that I removed it and built it from master here, and it didn't update the incorrect config keys.

If you change the keys manually or remove the config it will work in the master branch version, but I couldn't get the keys in the older version from apt to work at all though.

riksweeney commented 1 year ago

OK, thanks. I understand the problem now.