Chicuelo Theme

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!

chicuelo

@movisman said in Chicuelo Theme:

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:

I do prefer a invisible help text, so I disabled in my build, but I think its useful so the color is ok. I think the original position on the in game menu fits better.

For all the other changes I think they are great so yo can raise the PR!

chicuelo

@movisman Yes, I overrides those images because they are not maded by me, so I used the sega cd and the genesis to replace them. I thought redesigning them but it will take a lot of time, maybe in a near future, but by now those two are ok!

movisman

@chicuelo said in Chicuelo Theme:

@movisman Yes, I overrides those images because they are not maded by me, so I used the sega cd and the genesis to replace them. I thought redesigning them but it will take a lot of time, maybe in a near future, but by now those two are ok!

Ah right, so it was deliberate changing Mega CD to be the 32x image, and Sega CD to be the Genesis image? Fair enough. I didn't realise the Sega CD images were not yours. Are you not allowed or do you not wish to use them?

As I run the main Sega systems, including 32x, Sega CD and Megadrive/Genesis, I prefer to have a unique image for each system, so I will keep those other Sega CD images to one side and use them in my build.

Otherwise, when scrolling through the systems there is a duplicate.

Thanks