Long standing ES bugs

Syhles · 6 Mar 2017, 22:27

@jdrassa
I'll probably install this branch tomorrow sometime. I'll post here how it goes.

I look forward to these changes making it to the main branch and the variables as well.

ruckage · 3 Jun 2017, 21:43

@ruckage Sorry, poor testing on my part. The code is not properly handling the case where there isn't a selectorImageTile element. Just specify it as false and it should work. I will get that fixed soon.
<selectorImageTile>false</selectorImageTile>

Doh! - I nearly tried adding that in as well but thought that couldn't possibly be the problem.

Just did a very quick test and it works brilliantly - look at all those pretty colours :D

I have one suggestion regarding this, currently <selectorScale> is a ratio of the line/text height which isn't very intuitive. Would it be possible for this to be a ratio of the screen height instead (like most other elements) or would that cause problems?

One quick point unrelated to the selector bar - it seems that the video snapshot scaling is broken in this build (see screenshot above - it's squashed) , not sure if you realised.

lilbud · 4 Jun 2017, 15:18

@ruckage Has this been added to the Retropie ES?

Syhles · 6 Apr 2017, 16:28

@lilbud
No these features haven't been added to the main branch (RetroPie's ES), you can test them to speed along the process of them getting added though.

@jdrassa
Tested this build, everything seems to be working as intended. Had no issues during testing and the selector image seemed to work perfectly fine. Also I have no video snaps so I can'tcomment on them being broken.

lilbud · 4 Jun 2017, 15:55

@jdrassa Could Selector Size be added as an element for the selector bar?

ruckage · 4 Jun 2017, 16:11

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

lilbud · 4 Jun 2017, 16:17

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

jdrassa · 4 Jun 2017, 16:56

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 · 4 Jun 2017, 17:22

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

ruckage · 6 Apr 2017, 18:24

@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 · 5 Jun 2017, 03:38

@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 · 6 May 2017, 05:05

@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 · 5 Jun 2017, 22:27

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

Syhles · 5 Jun 2017, 22:39

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

Syhles · 7 Jun 2017, 03:33

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

ruckage · 7 Jun 2017, 08:11

@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 · 7 Jun 2017, 08:22

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

ruckage · 7 Jun 2017, 08:29

@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 · 7 Jun 2017, 08:36

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

ruckage · 7 Jun 2017, 08:41

@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.