Chicuelo Theme

movisman

Hey :) hope you are good!
I think I might have spotted an issue (or perhaps it's deliberate) with a commit you made 11 days ago regarding 'assets update':
https://github.com/chicueloarcade/es-theme-Chicuelo/commit/7c33a4f1f07a50b5f9a7d8906d328ebeca953b9c

On first glance, I noticed Amiga is no longer Zool but the character you were using previously, FBA is now the Simpsons character as opposed to the Sunset Riders character, ZX Spectrum is the old initial blocky rendition of the Dizzy character (which was really low res) as opposed to the reworked one you made later (which was a lot better). There are a few more changes on this commit to various other characters and i'm not sure what is correct - I think a few of these assets have been replaced with older versions?

Just wanted to let you know so you can take a look? Let me know if I can help. I have not updated my copy - I was about to as I have a few minor improvements to the main theme.xml to share with you, but I have held back for now, as something doesn't seem quite right to me with a few of the images.

Thanks as always!

chicuelo

@rand0m said in Chicuelo Theme:

we can set missing artwork placeholders which are shown when image/ artwork is not present. That's why the questionmark sign in clouds (snap has too much detail, likely to change it to something which fits in better).

If you don't have any artwork it would not display. But for you can opt for using a blank png with the name of the system. I used that way for RetroPie setup, favorites, and auto collections. Instead of having a controller I used a blank png file so the theme displays something and there are no errors.

Try using this one :)

chicuelo

@movisman

Ohh, I think there is a problem with those assets. I only wanted to replace the bg for a new one with a subtle texture, and the Neo Geo controller for a more accurate one. All the other stuff is wrong.
If you want to take a look go ahead!

thanks for the adivce!

chicuelo

@movisman
I updated those one who where missed, now you can pull all the media again and upload your xml.
I have some pull request with assets changes but I closed all, I will only accept xml or display improvements.
Thanks again!

movisman

@chicuelo

Thanks for looking at this so quickly. I see you have restored those assets.

There were quite a few updates made, do you think there are any other images which could be incorrect or do you think everything is as it should be now?

PS. Did you make new updates to the global ingame bg and system ingame bg? I notice the arrows and BG are slightly different now.

movisman

@chicuelo

Also, just to check, did you deliberately overwrite the ports image to go back to the 'Doom' guy? Originally ports was the Doom guy, then it was changed to the Cacodemon up until 15 days ago, now it is back to Doom guy again.

I don't use ports but just another image I noticed which has totally changed? If this is deliberate no worries.

Thanks!

chicuelo

@movisman
I think the character assets are ok now so that's complete.
I changed slightly the global and in game backgounds, make them a little bit darker and added a texture so yes, they are new

chicuelo

@movisman Damn, it has to be the cacodemon, I will replace it now again. I think someone send a pull request and I accepted and changed all my stuff. If you see another asset messed up please tell!

movisman

@chicuelo

I see another, maybe deliberate but did you change the Sega CD image?
https://github.com/chicueloarcade/es-theme-Chicuelo/blob/master/_assets/characters/segacd.jpg

If so, perhaps you want to update the Mega CD image to match:
https://github.com/chicueloarcade/es-theme-Chicuelo/blob/master/_assets/characters/megacd.jpg

I will keep looking to see if I spot any more changes. The problem is i'm not sure which changes are deliberate or not :)

Cheers

movisman

@chicuelo

Maybe this will help if you want to check all your files. Here is a list of all files changed in those last two recent commits (not including your fixes today).

Perhaps you can check all the below files and ensure they are all the correct revision:

changed on 25th Jan
_assets/characters/neogeo.jpg
_assets/characters/nes.jpg

changed on 21st Jan
_assets/characters/amiga.jpg
_assets/characters/atarist.jpg
_assets/characters/auto-favorites-2.jpg
_assets/characters/auto-favorites.jpg
_assets/characters/fba.jpg
_assets/characters/gbc.jpg
_assets/characters/genesis.jpg
_assets/characters/mastersystem.jpg
_assets/characters/megadrive.jpg
_assets/characters/ports.jpg
_assets/characters/segacd.jpg
_assets/characters/zxspectrum.jpg
_assets/controllers/gb.png
_assets/controllers/gba.png
_assets/controllers/megacd.png
_assets/controllers/megadrive.png
_assets/controllers/neogeo.png
_assets/controllers/nes.png
_assets/controllers/sega32x.png
_assets/controllers/segacd.png
_assets/ingame-global-bg.jpg
_assets/system-global-bg.jpg

I agree you should only accept PR's for small changes, it looks like someone has raised a PR based on a much older version of the theme and it's overwritten a load of your images.

movisman

Once we have got your master github code completely up to date and correct, I will share my minor XML tweaks (which may or may not be of use). They are only small sizing improvements to preview images, a slide transition fix if enabled and improvements to visibility of the help icons if you have them enabled.

But lets not do that now until the code is all fixed up on your github master. If you need any files let me know because my copy is from November 2018.

Thanks

chicuelo

@movisman
The controllers were updated for a more accurate ones, with the logo or some info, so that's ok. Ive noticed that the genesis one and another didn't have the sega logo so I will update them again.

I think now we have all the setup fine again

movisman

@chicuelo

Great, no worries. Don't forget about the Mega CD and Sega CD characters - now they are different. Sega CD is a different Sonic image to Mega CD - I assume the newer Sega CD one is correct/preferable (?) or was this accidentally overwritten too?

Either way, I think they need to be the same, like Genesis and Megadrive are?

chicuelo

@movisman Thats the only character I have left to modify, I will upload it asap so you could update your script and change the xml.
I think the slide transition will be awesome

movisman

@chicuelo

Great, no worries. So once you have fixed that image, these are the only character images left which were modified recently:

_assets/characters/neogeo.jpg
_assets/characters/nes.jpg
_assets/characters/auto-favorites-2.jpg
_assets/characters/auto-favorites.jpg
_assets/characters/gbc.jpg
_assets/characters/genesis.jpg
_assets/characters/mastersystem.jpg
_assets/characters/megadrive.jpg

Are these ones all correct? To me, I think they are but they are clearly modified from previous.

movisman

@chicuelo

Regarding slide transition, it's not too exciting - it's just that the logo no longer slides in front of the text, it will slide behind it which looks a little nicer.

chicuelo

@movisman Yes, all those characters are correct. they have minor adjustments so all of them are ok.
Now you can update your files!

rand0m

@chicuelo Thanks, this controller icon is much better.

movisman

@chicuelo

Great, I will take a backup and update my theme now to match with your master. I will then check my theme.xml changes.

These are my proposed amendments:

improved carousel slide transitions if enabled - now the logo does not overlap the text (it goes behind which is nicer)
adjusted size, position and colour of help icons if enabled
increase maximum size of image and video slightly, so that media appears at 800x600 instead of slightly smaller. This eliminates the need for any scaling to take place when ES renders it on screen if running at 1080p
change delay of image>video from 2 to 1.5 to make the transition just slightly snappier

I cannot show you the slide improvement on an image, but here are some pictures of the help icon realignment/colour and the slightly larger in-game preview images:
.
SYSTEM BEFORE:
System (before, can barely see help icons):

.
SYSTEM AFTER:
System (after, lined up text more accurately, reduced font size and used a grey colour which is readable):

.
IN-GAME BEFORE:
In-game (before, help icons not visible, preview is resized to be less than 800x600, help icons in a different place which although looks good, is slightly erratic when swapping between in game and system):

.
IN-GAME AFTER:
In-game (after, help icons identical to system mode, preview image/video is slightly larger and should measure 800x600 so no resize needed by ES):

What do you think?

If happy, I will raise a PR for the changes, which will also include the slide improvement if you have that transition enabled. The XML changes are only minor, no major rework of anything.

Thanks

movisman

@chicuelo

Oh no! Your last commit for Sega / Mega CD is messed up:
https://github.com/chicueloarcade/es-theme-Chicuelo/commit/a369e850983a3269a3aff1bb53817481344d19fe#diff-fb9fbe25026bd918b37846b027e57d46

Mega CD is using the 32X image, and Sega CD is using the Megadrive/Genesis image!

Surely it should be either this or this:
https://github.com/chicueloarcade/es-theme-Chicuelo/blob/7c33a4f1f07a50b5f9a7d8906d328ebeca953b9c/_assets/characters/segacd.jpg
https://github.com/chicueloarcade/es-theme-Chicuelo/blob/91b44f59ece3232cb1f6464adfab4baf664d154f/_assets/characters/segacd.jpg

I don't know which you prefer to use, I don't know which is newer or the one which you want. But currently those assets are broken, as above.

Thanks!