YunoHost / issues

General issue tracker for the YunoHost project
72 stars 8 forks source link

Enforce size/dimension limits for apps logos #2438

Open selfhoster1312 opened 2 months ago

selfhoster1312 commented 2 months ago

The apps repos is way too big and app logos vary wildly... It's README currently mentions to keep the logo « as small as possible » and « almost square ».

However, it's not clear:

Illustrating the problem

For example:

Solutions

I'm happy to help with this if given specific guidelines to enforce.

Salamandar commented 2 months ago

We shall also follow Android logo guidelines such as margins around logo : image

ericgaspar commented 2 months ago

Most of my logo additions are set to 300px square. Following Android logo guidelines will involve a significant effort to gain 84px

Salamandar commented 2 months ago

I'm sorry, what I meant only was about the margins. See for example Immich vs Jellyfin, Immich is really not pretty here. image

Android suggests that the actual icon makes up for 75% of the outer dimensions.

ericgaspar commented 2 months ago

I yes, my bad

ericgaspar commented 2 months ago

I also think that we shoul optimise the file size of PNGs (personally, I am using ImageOptim to shrink the file size)

selfhoster1312 commented 2 months ago

Personal opinion: if we apply transformations on upstream logos, it's much better if these are done by a script so they can be edited/reproduced without graphics editing skills, either when the upstream logo changes, or when our logo guidelines change (for example, if we allow more sizes). Something like:

logo_source = "https://foo.bar/baz.png"
logo_transform = "convert -i \"$1\" .........."