Grid Updates - emulationstation-dev

paffley

@jdrassa that's brilliant thank you! :)

sergioad

@jdrassa ok, thank you very much

paffley

Hi @jdrassa quick question...

With the <centerSelection> set to true, would that mean, on first load, the last boxart/game would be displayed to the left of the 1st centered boxart/game?

Does that make sense?

jdrassa

@paffley It would not. Say you had your grid configured with 1 row and 5 columns. On first load, there would only be 3 grid items displayed, the center selected one and 2 more to the right.

        [1] [2] [3]

However, there is an additionaly option, scrollLoop, that works with centerSelection. This will create a seemless never ending loop and will on first load display the last grid item(s) to the left of the centered selected item.

[n-1] [n] [1] [2] [3]

paffley

@jdrassa said in Grid Updates - emulationstation-dev:

@paffley It would not. Say you had your grid configured with 1 row and 5 columns. On first load, there would only be 3 grid items displayed, the center selected one and 2 more to the right.
        [1] [2] [3]
However, there is an additionaly option, scrollLoop, that works with centerSelection. This will create a seemless never ending loop and will on first load display the last grid item(s) to the left of the centered selected item.
[n-1] [n] [1] [2] [3]

Absolutly brilliant @jdrassa love it! The scrollLoop is a fantastic addition! looking forward to testing these, do you have a timeframe on when these will be added to es-dev? no worries if not, I totally understand the dev side.

paffley

Hi @jdrassa just updated es-dev and I see you've made some updates.

I have added the below to my theme and all seems to work great:

<imagegrid name="gamegrid">
	<centerSelection>true</centerSelection>
	<scrollLoop>true</scrollLoop>
	<pos>0.00375 0.20765</pos>
	<size>1.0 0.527</size>
	<margin>0.00368 0.00368</margin>
	<scrollDirection>horizontal</scrollDirection>
</imagegrid>

The only issue im having now is that it is not picking up the <backgroundImage> tag in the default or selected grid section.

This is what my section looks like:

<view name="grid">

<imagegrid name="gamegrid">
	<centerSelection>true</centerSelection>
	<scrollLoop>true</scrollLoop>
	<pos>0.00375 0.20765</pos>
	<size>1.0 0.527</size>
	<margin>0.00368 0.00368</margin>
	<scrollDirection>horizontal</scrollDirection>
</imagegrid>

<gridtile name="default">
	<imageColor>ffffff</imageColor>
	<size>0.195577 0.527</size>
	<padding>16.0 90.5</padding>
	<backgroundColor>ffffff</backgroundColor>
	<backgroundImage>./_inc/images/select.png</backgroundImage>
</gridtile>

<gridtile name="selected">
	<size>0.1957 0.527</size>
	<backgroundColor>ffffff</backgroundColor>
	<backgroundImage>./_inc/images/selected.png</backgroundImage>
</gridtile>

</view>

In fact, I don't think its picking up any of the settings in the gridtile default and selected. Hope this helps.

paffley

@jdrassa in theory majority of the new updates you are doing could be adapted within the <imagegrid> tag and make the <gridtile> tags redundant? is that your thinking or plan?

The majority of your updates make the <gridtile> tags unnecessary imo.

paffley

Just been having another play with it and this thing is looking awesome! I know this is a WIP so keep up the great work @jdrassa looking forward to running some more tests :)

The animation is great and also the loop feature! The boxart that is highlighted in the above image is the one selected and if you press left on the controller it selects the left hand one then keeps going left. So basically it selects 2 - the right one for going right and the left one for going left. (If that all makes sense)

@ruckage you may want to watch this post also ;)

jdrassa

@paffley Would you mind posting the selected image you are trying to use?

jdrassa

@paffley said in Grid Updates - emulationstation-dev:

In fact, I don't think its picking up any of the settings in the gridtile default and selected. Hope this helps.

There is a bug in the assignment of the gridtile properties. I am preparing a PR and it should be merged within the next day or two.

@paffley said in Grid Updates - emulationstation-dev:

@jdrassa in theory majority of the new updates you are doing could be adapted within the <imagegrid> tag and make the <gridtile> tags redundant? is that your thinking or plan?
The majority of your updates make the <gridtile> tags unnecessary imo.

Some of the new options like autoLayout and autoLayoutSelectedZoom simplify the need to set specific sizes, but there are no plans to remove <gridtile>

sergioad

@jdrassa could / might you restore the slide minimal transitions to ES? Some time ago it was added but then deprecated and was my favorite transitions, it animated the game lists when you switch system not leaving the gamelists and when launching a game and the transition from the carrousel to the gamelists but it did not animated the carrousel background (not the carrousel itself), it was elegant and perfect for themes like NES mini, SNES mini, Switch and many other themes

paffley

@jdrassa said in Grid Updates - emulationstation-dev:

There is a bug in the assignment of the gridtile properties. I am preparing a PR and it should be merged within the next day or two.

Thank you :) I'll keep an eye out and run some more tests.

Do you still need the selected image from your post above?

The autoLayout and autoLayoutSelectedZoom are great additions and will make life a lot easier for boxart. I had a previous discussion with @ruckage about this on one of this themes (regarding specific grid sizes for specific systems etc for boxart for the end-user - this solves that issue).

jdrassa

@paffley The fix should now be in emulationstaiton-dev.

paffley

@jdrassa said in Grid Updates - emulationstation-dev:

@paffley The fix should now be in emulationstaiton-dev.

Perfect - all working great Thanks @jdrassa

This has changed the game completely for the grid view! All possibilities now!

paffley

Found a bug @jdrassa

<animate> tag does not work with either

<animate>true</animate>

or

<animate>false</animate>

It crashes the theme when inputted into the <imagegrid> section.

Hope this helps.

jdrassa

@paffley This should be fixed now. animate got left of the list of supported theme elements.

paffley

@jdrassa said in Grid Updates - emulationstation-dev:

@paffley This should be fixed now. animate got left of the list of supported theme elements.

Super, thanks @jdrassa i'll have a play with it shortly - loving your work, thank you :)

paffley

@jdrassa all seems to work fine with true and false for <animate>, your a star! :)

I'm not asking for straight away but when you get some free time would you be able to look at adding a <scrollSound> to the imagegrid please, I believe then it would be a complete view of its own :)

I believe someone was looking at this a while back but it never materialised and think they have disappeared now.

Thanks again for the updates, really appreciate it. Looking forward to releasing some grid themes!

paffley

Hi @jdrassa I've ran vigorous tests with the new updates you've done on the grid and all seems to work fine! It's ten times better than the previous grid release. Thanks again for all your hard work :)

Do you think it would be merged to the official retropie release?

sergioad

@sergioad said in Grid Updates - emulationstation-dev:

@jdrassa could / might you restore the slide minimal transitions to ES? Some time ago it was added but then deprecated and was my favorite transitions, it animated the game lists when you switch system not leaving the gamelists and when launching a game and the transition from the carrousel to the gamelists but it did not animated the carrousel background (not the carrousel itself), it was elegant and perfect for themes like NES mini, SNES mini, Switch and many other themes

@jdrassa could you?