Chicuelo Theme

chicuelo

@movisman
Here are the updates! I scaled all the controllers into a 548x450 px png so the scaling process make all of them look good.
The math I do was a maximum width size of 28,5% and max height of 41,6% (if 1920 px is 100%, 548 px is 28,5%, and 1080 px = 100% so 450 px are 41,6%)
All the controllers are placed at bottom right as expected:

I also created a correct (hope so!) Amiga keyboard!

I did not updated the repo, its better if I send you the files and you download and work from your side so when we are sure its working fine we could merge all.

https://www.dropbox.com/s/dzah56pmv3gy4j0/controllers.zip?dl=0

Thanks!

movisman

@chicuelo

Hi,

Yes! This is much more like it :)

I have downloaded the files and they all look like I was expecting (all images within the same canvas size). The only file in the zip which doesn't adhere to the fixed canvas is megadrive (I guess you forgot to edit that one), but as it's the same as Genesis we can simply copy Genesis and rename.

I am at work at the moment so will not be able to test properly on my local machine until later. Once I do so I will show you a video of how it looks in 1080p and 720p.

Regarding the scaling percentages, when I did tests it's best to only scale on the horizontal axis and leave the vertical as 0 (this way the vertical size will be automatically calculated in accordance with the image's aspect ratio and thus takes care of itself). We also use the size parameter rather than 'maxSize', this then forces the size to be the percentage to what we configure in the XML, and if we pick the correct percentage it means no scaling up or down in 1080p, but it will scale downwards for 720p, etc.

The only thing I might ask, is your dimensions chosen do not calculate to be an exact/rounded percentage for 1080p (548px is 28.541 recurring, 450px is 41.666 recurring). I'm not sure how many decimal places the XML will work to before it rounds up or down. It might cause the tiniest amount of scaling in 1080p, but i'm not sure until I try it. It may not even be noticeable and/or the XML might let me use enough decimal places. I could probably work it out.

To make sure there is definitely zero scaling though, is there any chance you can adjust the overall canvas size to be something like 528x432? (this equates to more precise (non recurring) percentages we can use in the XML for 1080p - so 27.5% and 40% exactly).

Would that be ok and does that make sense? The vertical canvas size doesn't matter too much, as we are not inputting that percentage into the XML (we are only scaling on the horizontal axis, vertical will sort itself out and keep the aspect ratio), so technically you could leave this 'as is'. However if you want to keep the same scaling ratio for your images it would probably be worth adjusting this too.

Personally I would have the canvas at 528x432 if you can, and just scale the necessary images inside down slightly to fit within the boundary. The images will be a little bit smaller but only a very small amount.

Then in the XML we would use 0.275 (27.5% exactly) as the size for horizontal. Vertical takes care of itself.

Would this be ok and does that make sense? So basically exactly as your updates above, but at a canvas size of 528x432.

If you don't want to make further changes, I can try later to see and use these longer percentages in the XML, and see if it causes any weird single pixel scaling as a result of rounding up. I don't know how many decimal points the XML can adhere to. However if it's a quick change I would just go for 528x432 and we then know that will be perfect.

Thanks!

movisman

@chicuelo

PS. The Amiga image is perfect - an A1200 which is a standard and much loved model :)

Cheers!

chicuelo

@movisman

No problem! I will adjust all the images to that canvas, is it necessary to adjust the size of the controllers too or only the canvas?

movisman

@chicuelo

Cool!
Well, because the proposed canvas size is a little bit smaller than before, I guess you will have no choice but to reduce some of the images slightly to suit, otherwise they will go off the edge of the reduced canvas. Know what I mean?

Cheers!

chicuelo

@movisman
Yes I notice but its not too much!, here are the files on that size!

https://www.dropbox.com/s/gu0nbb47namydqn/controllers 528x432.zip?dl=0

cosmo0

@chicuelo Yay! the new arcade controllers look great :) thanks !

movisman

@chicuelo

Awesome, thank you so much!

When I get home I will take a look and test on my local install and let you know how it goes!! :)

Cheers

movisman

@chicuelo

Fantastic! These look really great!

1080p:
https://drive.google.com/open?id=13r84uApa4eX-N7i7jCq_sK6YXiVpkFiM

720p:
https://drive.google.com/open?id=1xGvaOxO2e2vwkm9uw2nhtf0LB7ouUxfy

Scaling all working exactly as I hoped. Obviously I don't have all the systems, but I can't see why the others wouldn't display nicely.

If you think there are any which in the end, you think are too large/small, it would just be a case of editing the image inside the SVG you made to the set canvas and re-saving I guess.

The 'only' one which doesn't look as sharp as the others on first glance is the Dreamcast controller, and maybe it could do with being a touch bigger (closer to gamecube/n64). If your source is already at maximum size without scaling up, we don't need to worry about this for now.

Since I made the video I also made a 'very' slight change to the positioning to make the margins between the controller and the edge of the screen exact and consistent with each other :)

How do you want to put the changes live?

Do you want me to upload the controllers to my fork along with the modified theme.xml and raise a pull request? I synced with your latest repo and made the changes to my local version. Or do you prefer another way? Not sure if you need or want to make any further corrections at this time? Otherwise I can just raise a PR and you can merge?

Thanks!

chicuelo

@movisman
It looks awesome!
If you want make a PR with the controllers and the xml update and I merge.
I lately done a mame and Neo geo controllers update, but I think you downloaded that versions, if not I replace them after.
there were minor changes on the darkness of the controller

chicuelo

@movisman
Before make the PR update the repo because I just added wonder swan color!

movisman

@chicuelo

Hi,

Yep, I freshly updated from your master this evening before making the changes.

I have raised a PR now, as follows:

theme.xml
Updates to controller sizes and position

Controllers
Update all controllers to set canvas size for scaling purposes

Readme
Quick update and tidy up, add in new blisters

Hope you don't mind me tidying up the readme a little bit, I didn't change much just some corrections and additions here and there. Apologies there are three commits for it, I realised it needed further edits each time I committed to my fork. I assume you can merge all the commits into a single one anyway for all my changes.

When complete, i'll update my main version from RetroPie Setup and test all is working ok!

Thanks a lot for your work on rescaling and reworking the controllers to my preferred size.

Cheers

movisman

@chicuelo

Ooooh! Ha I raised the PR pretty much at exactly the same time as you wrote this.

Hope this does not cause issue for you. It should still be fine if you merge in my changes as your updates are all new files so no conflicts.

Once you have merged my changes in (including my readme edits), you could then update the readme to add WonderSwan as supported.

movisman

@chicuelo

BTW, I noticed there is a standard (non color) Wonderswan theme.xml and Logo already in place, but no controller or background. I have added this to the 'coming soon' on the readme and updated my PR, or is full support for WonderSwan not planned? At the same time I have added WonderSwan Color to the readme as fully supported now. At the same time I finished reordering all the systems so they are in alphabetical order.

Thanks!

chicuelo

@movisman
Great I will merge now.
As wonder swan monochrome there are a lot of system without the character and controller like colecovision, vectrex, neogeopocket ande NGP color , those will be updated soon! we already have the xml and the logo but it remains the character and controller!

movisman

@chicuelo

No worries! Well I updated the readme so all the system info is nice and up to date now, I also added WonderSwan to coming soon and WonderSwan color into the fully supported system list.

Hope ok.

Thanks!

chicuelo

@movisman
Done the PR! and thanks for upgrading my gramar! I not very good at English :)
Next step is completing the missing system list!

Thank you very much!

movisman

@chicuelo

No worries at all! I hope you didn't mind me tweaking the Readme file to tidy some things up and rearrange the systems. Your English is actually very, very good btw!!!

I will make some further adjustments to the readme a bit later - just did some quicker bits for now. I think we can expand on some of the 'features' as well to let people know about the blisters, screenshot and video support and other things like that.

I updated my main theme from RetroPie-Setup, and all works perfectly! Great stuff :)

chicuelo

@movisman
Sounds great! When @cosmo0 finish with the genre files I will ad to the preview images on the readme so people can have a better idea of that!

movisman

@chicuelo

Excellent! I was going to mention that, certainly would be worth doing once those are completed.

Cheers!