Grid tile content (image, text, or more ...)

A12C4

Ok I added a scrollDirection parameter to the theming syntax, where its value is vertical by default, but can be set to horizontal if needed.

Was kinda easy to do actually.

jdrassa

@a12c4 said in The Grid View need your opinion:

Ultimately, I'm doing this on my free time, I'm not being paid for doing that, so nobody should be able to order me to do something I don't want to. I do it because I have fun programing complex stuff and learning, if it can help the RetroPie community along the way, then it's even better, but I won't prioritize the second over the first.
I don't like this last argument, but well, you triggered me with the "Maybe we should have rejected some of your PRs for the same reasoning" and " This just seems like a wierd arbitrary point to decide to draw the line." ahahahah

No worries. I recognize we all are contributing in our free time. Hopefully, I didn't trigger you too much. I was a bit worried it may come across as harsh, which wasn't my intent. I appreciate the effort that you have put into this and especially the effort to break things down into smaller chunks which definately helps with reviews and testing.

A12C4

@jdrassa No problem, let's keep moving ahead together.

If I can't figure out, in a reasonable amount of time, how to properly refactor the game views into one single, modular game view, then I will add all the missing metadata elements to all the views.

But for now, please let me try my solution first.

I don't like to stupidly copy/paste stuff around, I feel like a bad IT student who just discovered stackoverflow.

A12C4

@jdrassa Btw, what's @BuZz policy for stable and master branches ? We are getting close to the version 1 of the grid view, so I wanted to know how I can plan ahead.

jdrassa

@a12c4 I wouldn't presume to speak for Buzz, but my understanding is that he will move changes to stable when he determines them to be stable. The stable/master split is a relatively new thing, and came about after a few contributions required fixes shortly after being accepted.

That being said, I would expect grid view to live on the master branch for a while before being moved to the stable.

PittStone

@jdrassa I have a Problem with Custom Collections if i Use Portable Emulationstation with the portable.bat Can you help me? https://retropie.org.uk/forum/topic/9094/updated-emulationstation-for-windows/97

A12C4

@ectoone Ok I added a md_title element to the theming syntax for all the views with metadata (grid, detailed and video).

By default it is positioned offscreen, beside that, I kept the name md_title and the default parameters (font size, color, ...) used by the grid view of @jacobfk20 to ensure maximum compatibility with existing themes.

I lost so much IQ points by copy/pasting stuff around that I want to start the hashtag #makeESgreatAgain

A12C4

@EctoOne @lilbud @Shadowron @herb_fargus @chicuelo @Clyde @afiction @paffley @jdrassa

We are getting close to the version 1 of the grid view, it only miss a few things (default image, folder image, dynamic texture loader so the grid can handle big game collections), so it's time to open the gates to more testers.

You can test out latest version of the grid view, by cloning my git repository, I will keep the master branch updated with all the latest changes as soon as they are stable enough.

https://github.com/Koerty/EmulationStation

CAUTION : after trying my version of EmulationStation, if you go back to the stable version of EmulationStation of RetroPie, the "Gamelist view style" option in "UI settings" will be empty, set it back to either "Basic", "Detailed", "Video" or "Automatic". If you enter the "UI settings" menu and leave it while "Gamelist view style" is empty, ES will crash. It doesn't know about the "Grid" view yet.

Hex

@A12C4 Currently it works as shown in the screenshot (names bellow images). The initial goal is to get it running. Prettifying can be done later.

A12C4

@hex What do you mean ? You want the game names below the images ?

It run right now, beside the dynamic image loader all the core features are here.

Hex

Get it running in the hands of users, themers and devs.

Ideally i would like the image to be shown and a dark (low alpha) background and text on top. I am searching for the eg. image that i did long ago.

paffley

@A12C4 great work.

If themed correctly, can this display the md_marquee tag vertically in a single column? Basically like the current System Carousel?

A12C4

@paffley No marquee nor thumbnails, only images for now, idk for the future I need to take a look at how marquee and thumbnails work first.

But yes, you are be able to do a single column grid if you want to.

paffley

@a12c4 said in The Grid View need your opinion:

@paffley No marquee nor thumbnails, only images for now, idk for the future I need to take a look at how marquee and thumbnails work first.

But yes, you are be able to do a single column grid if you want to.

That's great thank you! I can just move the marquee images into the image folder no problem for now. :)

Keep it up!

paffley

@A12C4 one more little question, if the grid view is enabled, can we still use the video tag elsewhere on the page. so basically you would have the image grid displayed and the video displayed in a different section (but the video is not part of the grid), if that makes sense?

I'm just thinking from a themers point of view. I can draw you up a example if you like me to?

A12C4

@paffley No video support as already stated previously and no plan for video support in the future. I need to take a deeper look into the code of the video view to see possible solutions.

paffley

@a12c4 said in Grid tile content (image, text, or more ...):

@paffley No video support as already stated previously and no plan for video support in the future. I need to take a deeper look into the code of the video view to see possible solutions.

No problem! thanks for the reply. :)

A12C4

@lilbud @Ectoone @chicuelo I copy/pasted the following from the related pull request on github, as you are more active on the forum :

I will, if you are ok with it, rename "md_title" to "md_name", to be consistent with how this element is named internally (in the gamelist.xml for example).

That's how I wanted to name it at first, but I chose "md_title" so theme makers who originally created themes for Jacobfk20's grid view have less modifications to make it run with the new grid view syntax.

As there is little to no theme that supported grid view at this time, and as we already stated when I created my first grid view PR that it isn't that important to don't break compatibility with themes developed for features we never officially supported, I'm willing to make this change, as I find consistency more important.

EctoOne

@a12c4 Yeah, I always thought at first that md_title was something related to the tiles when it was mentioned. md_name makes more sense to me.

Btw, can you tell me which one is your main branch for all of the new stuff? Last night I've downloaded a Rasbian VirtualBox image and installed Retropie on it and with the es-tester scripts from @meleu it looks like a good way to test things directly on my Windows PC.

A12C4

@ectoone I don't have one, I used to have one but it was way too much effort to constantly keep it updated due to the numerous PR I did for the grid view.

You can find my latest changes on RetroPie's EmulationStation's master branch, minus the ongoing PR and some experiments.

You can install EmulationStation-dev on your pie or clone the git repository.

Note that you need to enable the grid view manually : how to enable the grid view