Input needed: ES theming improvements

mattrixk

@Zigurana This works great! Well done mate.

Notes:

I can only find one small issue. The pop-in of the right logo is gone, unless you set the carousel to display more systems than you have. I have 5 systems and have no pop-in if the carousel is set to 5 or fewer, but if I set it to 6 or more, then the right-hand logo will pop-in. Not a major problem, and relatively easily fixed for anyone that runs into it (just set fewer systems).
<forceUppercase> doesn't work on <text name="systemInfo">, but it also doesn't break the theme if it's used.
<alignment> does work on <text name="systemInfo">and that makes me happy.

Requests:

I'm curious how hard it would be to change the alignment of the logos... At the moment, all the logos are aligned along the center-line of the carousel. I wonder how hard it would be to give them the ability to align to the top or bottom of the carousel, then we could have them running along the top/bottom of the screen and have the active logo higher/lower than the rest. Just thinking out loud.
It would be great if we could use the <alignment> tag on the <helpsystem name="help"> element. It would make it a bit more versatile.

Again, love your work!

lilbud

@mattrixk I can only think of maybe 5 themes that could do something extraordinary with a themeable carousel. I got nothing. Thoughts?

mattrixk

@lilbud I have the feeling we won't know until someone comes out of nowhere and blows us away with an awesome idea, totally different to what we've seen before.

lilbud

@mattrixk I had the idea for a kodi confluence theme

mattrixk

@lilbud I'm a big fan of the various Kodi layouts and themes. If we're ever able to work with multiple images (box art, fan art, logos, screenshots, etc), and when Grid View is finished, we should be able to make some really interesting stuff.

jdrassa

@Zigurana I have been running this on my linux box and my pi. So far I have had no issues. The only additional suggestion I would make would be to apply the color theme element to all of the logos vs just the selected one.

EctoOne

Uhm, how can I get this on my Pi? I thought there was a guide somewhere in this thread but I couldn't find it. And it's been awhile since I've used Linux as OS. So I'm kinda out of practice. I just made a backup of my SD because of this and now I'm stuck.

Edit: Nvm, i made a copy of this script from the Grid View Mod Thread and changed it into this. It's building right now.

Zigurana

I've submitted a PR for the SystemCarousel theming.
If anyone fancies having a look at the code, please have a go.

EctoOne

After i've managed it to get this on my Pi, i did some testing over the past few days. I can't spot any massive Bugs but there are two small things i noticed.

1: It has to be
<text name = "systemInfo">
Whereas every other tag doesn't need to have the spaces before and after the equal sign.
2: It has to be

<view name="system">
  <image name="background_image" extra="true">

Maybe it needs to be an extra image, but i would prefer it if i could simplify my theme by using something like this:

 <view name="system">
   <image name="background_image" extra="true">
	<path>./_inc/system.png</path>
   </image>
  <view name="system, basic, detailed, video">
   <image name="background">
	<path>./_inc/background.png</path>
	<origin>0 0</origin>
	<pos>0 0</pos>
	<size>1 1</size>
   </image>
</system

instead of:

  <view name="system">
   <image name="background_image" extra="true">
	<path>./_inc/system.png</path>
	<origin>0 0</origin>
	<pos>0 0</pos>
	<size>1 1</size>
   </image>
</system>
<view name="system, basic, detailed, video">
   <image name="background">
	<path>./_inc/background.png</path>
	<origin>0 0</origin>
	<pos>0 0</pos>
	<size>1 1</size>
   </image>
</system

But maybe that's not possible, i just wanted to mention it. Great work anyways!

darknior

Hi

I'm really impressive and happy that you add this feature to ES :D
So cool, we can now make a skin like we want ;)

For my part i will adapt my xBox Skin to Pi with your new Carrousel option <3
It take me many month of my life to release it with my little hands, Cyberlink Power Director and Photoshop :)
And all in 1080p full HD !

It is an old preview and i don't know why Youtube remove my last one two years ago ...

My last version share by a fan :

But i have two request please to help me to release :

Can you update the ES Theme Wiki or post some where all the new XML we can use to do it ?
Like i know some one add the VIDEO MAXSIZE and not document it in the wiki :(

https://github.com/retropie/retropie-setup/wiki/Creating-Your-Own-EmulationStation-Theme

Can you add the video support to the SYSTEM SELECT SCREEN ?
Because like you can see on my skin i use it and it is really better i think :)

Thanks a lot

mattrixk

@darknior said in Input needed: ES theming improvements:

Can you update the ES Theme Wiki or post some where all the new XML we can use to do it ?
Like i know some one add the VIDEO MAXSIZE and not document it in the wiki :(

Once the Carousel Mod has been accepted into RetroPie Core, then I can add it to the Theme Wiki. I'll also add the Video Preview stuff when I get the chance. I'm not sure if <maxSize> is part of Core yet or not though.

jdrassa

@mattrixk <maxSize> for video was accepted yesterday and should be included if you update from source.

FYI, I did update the theme documentation within EmulationStation with details for the video element and video view. The pull request for the carousel will include corresponding updates as well.

https://github.com/RetroPie/EmulationStation/blob/master/THEMES.md

darknior

@mattrixk @jdrassa Thanks guys you are the best :)

I see it this afternoon for MAXSIZE ;)
https://github.com/RetroPie/EmulationStation/pull/104

mattrixk

@jdrassa Great news about <maxSize>. Unfortunately I'm still using Fieldofcow's older Windows build of ES, so it won't have <maxSize> available. If anyone wants to build a fresh Windows ES with all the new features I wouldn't mind.

Nismo

@mattrixk +1 for that.

jdrassa

@mattrixk @Nismo I have been trying to setup a CI build for Windows using appveyor.com, but I am still working through some issues. Not sure if I will get it working, but I am hopeful.

mattrixk

@jdrassa said in Input needed: ES theming improvements:

but I am hopeful.

So are we :D

Not to mention appreciative!

EctoOne

Um, have there been any major changes been made? I had to rebuild it from source today and it doesn't work anymore on my Pi.

Edit: Nvm, build it again, rebooted and it works again.

masteryoer

Perhaps the most useful feature would be an actual templating engine. For example, like twig or blade. One where each window, dialogue box, system can be themed completely independent of each other.

I emailed the original es author and asked about working with him on it, but didn't really get a positive response.

Would be nice to set up individual templates that are easily created or modified (for example an HTML file), with placeholders for all of the data you want to place %gamesList% for example. Then you could position the elements anywhere on the screen, any way you wanted to, and would provide a vast increase in possibilities for theming.

mattrixk

@masteryoer Sorry mate, but I'm having trouble seeing how what you've suggested is really any different from how theme making is currently done.

Also, what are twig and blade?