Pixel Theme

[hiulit]

Hello @Rookervik !
I'm working on fun facts splashscreens and I was using your theme as an example and while trying to parse pixel.xml with xmlstarlet I kept getting and error telling me that that xml was malformed.
Looking into it, I found a couple of comments that have one extra dash -, these ones:

https://github.com/ehettervik/es-theme-pixel/blob/6488c5426c2ede809d6b3aafeb6e6f78773ce11e/pixel.xml#L202-L203

<!--- This will enable Player Count under the screenshot --->
<!---<pos>0.686 0.573</pos>--->

and

https://github.com/ehettervik/es-theme-pixel/blob/6488c5426c2ede809d6b3aafeb6e6f78773ce11e/pixel.xml#L213-L214

<!--- This will enable Player Count under the screenshot.--->
<!---<pos>0.830 0.567</pos>--->

I removed them and now it's working! :D
Just wanted to let you know, because it might affect other projects using xmlstarlet to parse themes.

[meleu]

@hiulit I think Rook is aware of it:
https://github.com/ehettervik/es-theme-pixel/pull/3
:D

---

EDIT: also https://retropie.org.uk/forum/post/60107

[hiulit]

@meleu @Rookervik whoops, sorry! Should have checked for issues in the repo before :P

[Rookervik]

@hiulit Yeah I'm aware and I haven't updated the theme on purpose. LOL. Tho if I ever intend to use meleu's awesome splashscreen maker, I might fix it. But the reason I didn't fix it is exactly because of the splash screen maker :D Hahaha

[hiulit]

@rookervik Why wouldn't you want @meleu 's splashscreen maker to use your theme? :O

[meleu]

@hiulit Have you ever heard of the Luddite movement? :D

[hiulit]

@meleu x_D

[Rookervik]

Ok, starting to work on Pixel with Metadata. (gotta go make me some gamelists so I can even see the crap :P )

Anything you guys want fixed up or changed? Now's the time to ask. I don't use this theme so I don't care how it looks. Haha.

[Rookervik]

I think it's all working. I don't have gamelists to test. I can make a couple but I didn't move any positions of text.

Videos are in. And I think everything from Pixel-regular. Anyone wanna test it?

[Quackwalks]

@rookervik The fancy carousal at the bottom is about the only thing I would like to see in the Pixel metadata update. I'm sorry to say, I'm not a fan of the spaced out game list in Pixel. I appreciate your hard work anyway.

[Quackwalks]

@rookervik Should I be able to update it through updating retropie setup then updating the theme in es-themes? I tried and no update is available.

[Rookervik]

@quackwalks I was gonna mail it to whoever wanted to test it. Before I send it to the main download and potentially ruin everyones' pixel. LOL

[Rookervik]

@quackwalks Ah, the spacing was added to make the game selection bar fit in the background. You don't like how the games are spaced further now?

[Rookervik]

Screenshot with a video playing. Metadata shown. Everything seems to be working. Systems screen is the same as Pixel. Carousel moved and art repositioned.

[RetroNESguy]

Thanks for all the hard work, great theme!

[Quackwalks]

@rookervik In my opinion it's more important to show a good number of games per page than having them match the background. On my 1080p TV, towards the bottom, the list becomes unaligned with the background anyway. But hey, I'm just one user.

[Rookervik]

@quackwalks Nah, I also am a little put-off by not showing as many titles on the gamelist. I'm going to look into this and see about fixing it up.

[Rookervik]

If I don't find any errors in Pixel Metadata I will update the github repo so everyone can try it out and potentially break their systems. LOL.

In other news... coming up with a selector graphic is hard... especially since adding a color to the bar colors the entire bar.

[RetroNESguy]

@Rookervik can you just have the text change colors for the highlighted game?

[Rookervik]

@retronesguy Yeah, that's all it typically does. Easiest method.