L/R shoulder buttons not working to jump letters

mitu

@zerobyzero The L/R buttons don't work right away after mapping - which is probably a bug - but they work after a restart. That's a 'feature' - probably because the es_input.cfg is read only on start, but the mapping - in-memory - doesn't have the L/R enabled.

zerobyzero

I can confirm that's not working for me, @mitu. I removed the configuration and I started ES from scratch so I got:

<input name="leftshoulder" type="button" id="4" value="1" />
<input name="rightshoulder" type="button" id="5" value="1" />

By renaming the buttons to pageup and pagedown everything works as expected.

mitu

@zerobyzero Thanks for reporting - let's see if we can find a better solution.

zerobyzero

@mitu It seems simple to fix in the code. I'd say that, to be consistent, the "pageup"/"pagedown" texts should be changed to "leftshoulder"/"rightshoulder". They don't appear in too many files. I'll try to modify the code this weekend, check it works and submit the patch to github.

Regards.

EDIT: Patched and tested, it works. Now I need to figure out how to push to ES git, I'm getting an error 403. Do I need to be verified or something like that to be able to push patches?

mitu

@zerobyzero said in L/R shoulder buttons not working to jump letters:

Do I need to be verified or something like that to be able to push patches?

Modifications are usually sent as Pull Requests to the RetroPie's Emulationstation Github repository - https://github.com/retroPie/emulationstation/pulls.
You'll need to fork the repository, add your modifications (commit) to your repository, then create a Pull Request with the modifications - GitHub will propose you to do so when it detects you pushed your commits.

zerobyzero

Thanks, done.

mitu

@zerobyzero I've taken a look at the PR, but it doesn't seem correct. The .cfg file is not produced by Emulationstation, it's only read by it. The .cfg file is produced - externally - by this script, which already maps the shoulder buttons to 'pageUp/pageDown' (see here).

Moreover, as others pointed out, the mapping appears to be correctly generated for other controllers - my DS4 input configuration has - correctly - pageup/pagedown in the .cfg file, so as my other controllers which have shoulder buttons.

I'm thinking to have something like 'isMappedLike' which handles both D-Pad and Joysticks for movement, to map both types of inputs to the same action.

ExarKunIv

im on build 4.4.12 and using a 8bitdo NES30 and the shoulder buttons work fine

just giving info, thats all :)

zerobyzero

@mitu, I'll check the script but to me the fact that the source code of ES refers to the buttons by two different names is incorrect.

Anyway, I have to say that I didn't download the full package at first. I mean I only downloaded the source code of ES to compile, then when I realised I missed the themes I installed the configuration script. I need to do more testing though.

Regards

mitu

@mitu, I'll check the script but to me the fact that the source code of ES refers to the buttons by two different names is incorrect.

Emulationstation works with more than 1 type of controller and/or keyboard. The fact that it works fine for a lot of other people contradicts your observation.

Anyway, I have to say that I didn't download the full package at first. I mean I only downloaded the source code of ES to compile, then when I realised I missed the themes I installed the configuration script. I need to do more testing though.

If you didn't install the package correctly, then you'll get incorrect results. Install the package from the RetroPie-Setup package - the stable version - then re-configure your controller and see if the .cfg file is correctly generated.