Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

App icon refresh #145

Closed
wants to merge 1 commit into from
Closed

App icon refresh #145

wants to merge 1 commit into from

Conversation

JSEHV
Copy link

@JSEHV JSEHV commented Jun 28, 2021

App icon refresh

App icon refresh
@mentaljam
Copy link
Collaborator

Launched a vote for accepting this PR

@Olf0
Copy link
Member

Olf0 commented Jul 9, 2021

@JSEHV, I like the new icon a lot, but IMO the SailfishOS logo should be slightly bigger to become better visible / recognisable. It also looks / appears / is a bit off centre, like being slightly shifted to the lower left corner on that side of the package.

TL;DR: The SailfishOS logo scaled to 110% to 120% of its current size and moved a bit to the upper right corner of the side it is "printed" on might look better.

Other ideas:

  • Increase the size of the white label.
  • Put something on the white label, e.g. SFOS or Storeman
  • Rotate the package a bit to the right (i.e., anti-clockwise), so it is not displayed in an exactly symmetrical position, in order to create a more 3D looking impression. This would also expose the side with the SailfishOS logo better.
    Or rotate it to the other direction (i.e., left / clockwise) to expose the label better: This might be useful, if you consider to put Storeman on the label, as this is a long word, which may be too tiny to read otherwise.
    Take a look at the old icon, which achieves that 3D-look nicely by being tilted and a bit rotated, i.e., its "flight lines" (best visible at the top lines of the icon with transparent background) meet at an infinite far point somewhere slightly to the left.
  • Utilise the space available, i.e., scale the whole package slightly up and move it slightly down. Again, take a look at the current icon which does that "to the max.", but still well and well looking.
  • Simulate a light source at the upper left, like done for the extant icon, resulting in colour gradients on the sides of the package and a drop shadow being cast to the lower right (see current icon).

@mentaljam
Copy link
Collaborator

@JSEHV, thanks for the icon. I'll merge the PR, but I want to see your opinion on some points:

  • About increasing the contrast as @direc85 suggested
  • About @Olf0's suggestions
  • About the current cover background image which is made from the old icon

Also I ask you to remove all the PNG files from the PR and leave only SVG

@Olf0
Copy link
Member

Olf0 commented Jul 13, 2022

Closing this PR, because there was no reaction from the original author of this PR for more than a year.

Furthermore the requirements have changed: Now the SVG plus an export in certain PNG sizes are needed, but more sizes than before.

I also took the time to analyse a couple of aspects of the current icon by laurent_c, such as perspective, shading, shadows etc., which let me comprehend that the draft of this new icon is not up to the standards of the current one.

This PR can be reopened at any time, if a reworked version of the new icon is submitted. I suggest to submit an SVG first, so its style and properties can be discussed and only export the PNGs when we have agreed on an SVG version as final.

@Olf0 Olf0 closed this Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants