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

iOS icon has black border - can be polished #2604

Closed
duarteocarmo opened this issue Feb 28, 2020 · 9 comments
Closed

iOS icon has black border - can be polished #2604

duarteocarmo opened this issue Feb 28, 2020 · 9 comments
Labels
bug It's a bug high High priority issues

Comments

@duarteocarmo
Copy link

The iOS icon shows a weird black border as you can see in the following screenshot:

image

Environment

Joplin version: 10.0.43
Platform: iOS
OS specifics: iPhone 11 pro max on iOS 13.3.1

Steps to reproduce

Not applicable.

Describe what you expected to happen

Expect to not see black borders.

Appreciation Note

Great work guys! Keep it up!

@duarteocarmo duarteocarmo added the bug It's a bug label Feb 28, 2020
@duarteocarmo
Copy link
Author

I’m gonna go ahead and say it’s probably because of some icon asset having transparency.

Reference: https://stackoverflow.com/questions/26014461/black-border-on-my-ios-icon

@duarteocarmo
Copy link
Author

@duarteocarmo
Copy link
Author

Would love to pull request if someone can confirm issue!

@vernvandamn
Copy link

I don’t know if you're looking for confirmation from another user but I do see the same issue on my phone.
PNG image

@duarteocarmo
Copy link
Author

@laurent22 should I upload new icons?

@laurent22 laurent22 added the high High priority issues label Mar 9, 2020
@amandamcg
Copy link
Contributor

amandamcg commented Mar 10, 2020

I noticed this also plus on Mac I also noticed that the app icon is larger than normal and doesn't have all the sizes in the .icns file.

If someone can point me to the original icon file, I can provide the assets in all the various sizes.

@amandamcg
Copy link
Contributor

amandamcg commented Mar 10, 2020

In digging in further, I was able to find the icons in the assets folder. I am not able to build / test the iOS and Mac apps, so I don't feel confident creating a pull request, but I used the same naming convention that was already in use to create the following:

iOS icons

  • Changed to be square instead of roundrect, which was getting cut off
  • I also noticed that "29x3.png" is currently 96px x 96px. My version resizes it to 87px since I think the other was an oversight.
    Example:
    60x3

Mac icons

  • Changed to add a little bit of transparent margin and use a tighter corner radius, modeled off of the Finder icon
    Example:
    icon_128x128

Zip of icons

MacAndiOSIcons.zip


Hope this is helpful, I'm a very happy recent Joplin convert. Thanks for the hard work!

@duarteocarmo
Copy link
Author

duarteocarmo commented Mar 10, 2020

@amandamcg nice find, thanks for the update. Same here, also not able to build source.

The maintainer has marked this as high priority so it hopefully should be solved soon!

@duarteocarmo
Copy link
Author

Issue follow up

Icons are still not completely there in mainly two places.

iOS

  • Still shows black borders
  • iPhone 11 pro max
  • Running 13.3.1
  • Joplin Version 10.0.46
  • Database v28

Example Screenshot:
IMG_02FB6AD48910-1

MacOS

Icon is large compared to other icons.

  • MacOS version: Catalina 10.15.3
  • Joplin Version: 1.0.197 (prod, darwin)
    Example Screenshot:
    Screen Shot 2020-04-06 at 09 30 16

@lock lock bot locked and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug It's a bug high High priority issues
Projects
None yet
Development

No branches or pull requests

4 participants