Long standing ES bugs

[ruckage]

@lilbud
It's already added (the height at least) <selectorScale>. The width is the width of the game list.

[lilbud]

@ruckage Could the width be changed with a separate element?

[jdrassa]

https://ci.appveyor.com/api/buildjobs/q63ax46bxlr37917/artifacts/EmulationStation-Win32.zip

Here is an updated Windows build for anyone interested.

<selectorScale> has been swapped out for `<selectorHeight>' which now functions like other sizes and is a ration of the screen height.

I also updated THEMES.md to document the new options.
https://github.com/jrassa/EmulationStation/blob/gamelist-fixes/THEMES.md#textlist

@ruckage I haven't seen any issues with video scaling.

@lilbud It is certainly possible, but I would need to be convinced it would be worth the effort. If you want the selector image to be wider than the text, you can use horizontalMargin. I can't think of a use case for having the image be narrower, but it could be achieved by having transparent space on the sides of the image.

[Syhles]

@jdrassa
I'll test out the new build later today.

[ruckage]

@jdrassa
Thanks. I tried the new build and <selectorHeight> works perfectly.

Regarding the video scaling, I'm not sure if it's affecting the videos themselves - I'm using Video view but without videos so it shows just an image preview instead.

You can see below, the preview images are squashed vertically, however in the latest standard version of ES it displays normally.

[jdrassa]

@ruckage I was able to reproduce the issue with the image scaling. Looks like it was an existing issue. My branch was a few commits behind the RetroPie repository. I updated it to pull in the latest changes and it is scaling properly now.

[Syhles]

@jdrassa
I haven't checked the build where you added the video fix, but I tried the one before that with all of the bug fixes, seemed to be working without any issues along with the selector image and scale, couldn't get anything else to act incorrectly as I don't have videos.

Do I need to check the video fix build as well? Or was that the only change?

[jdrassa]

@Syhles No need to check. The only change was to sync up with the RetroPie repository.

[Syhles]

@jdrassa
Then everything seems to be in working order from my testing.

[Syhles]

@ruckage
These changes have been merged to main RetroPie branch as of today.

[ruckage]

@Syhles said in Long standing ES bugs:

@ruckage
These changes have been merged to main RetroPie branch as of today.

Thanks, that's great. I'll get my themes updated then.

[Syhles]

@ruckage
Just curious but are you going to use the selector image function on your theme?

[ruckage]

@Syhles
Most likely, probably something simple though like very slightly rounded corners (corner pixels removed) to match the rest of the layout. I may do a little bit more on certain systems that would suit it. I'll be experimenting to see what would look good.

[Syhles]

@ruckage
Sounds reasonable, I was just thinking that you could do all kinds of different things for selector image using pixel art.

[ruckage]

@Syhles said in Long standing ES bugs:

@ruckage
Sounds reasonable, I was just thinking that you could do all kinds of different things for selector image using pixel art.

Definately. I don't want over do it but certain systems would benefit from more elaborate selector bars.

[Syhles]

@ruckage
I understand sometimes there can be too much on screen, though I feel like arcade, virtual boy, and game boy could end up with some interesting selectors.

[ruckage]

@jdrassa
Hi. Would it be possible to get an up to date windows build with the video fix as it helps with testing. Thanks.

[jdrassa]

@ruckage Here you go.
https://ci.appveyor.com/api/buildjobs/rd05i2jbp8oonvh6/artifacts/EmulationStation-Win32.zip

[ruckage]

@jdrassa said in Long standing ES bugs:

@ruckage Here you go.
https://ci.appveyor.com/api/buildjobs/rd05i2jbp8oonvh6/artifacts/EmulationStation-Win32.zip

You're a star, thank you very much.