-
Notifications
You must be signed in to change notification settings - Fork 804
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
Add SVG icon styled for macOS Big Sur #4631
Conversation
I'd still say the icon shouldn't have a shadow as it looks a bit off-brand, but the rest is great. :) Also cc @karlitschek @marcoambrosini cause you use MacOS. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good :)
but I agree about the shadow on the logo, could you please remove that @elsiehupp?
Fully flat color-on-color logos don't follow the macOS icon design guidelines. To quote my original Issue:
Indeed, looking at most third-party applications with icons on both iOS and macOS, one sees slightly more defined 3D on the macOS icons than on their iOS counterparts. For example, Element Messenger: Incidentally, as I mentioned in my Pull Request on Element Desktop from last year, the official Element Mac icon itself does not follow the Element brand guidelines (which actually specify a green circle inside a larger white shape on platforms with non-circular icons, furthermore with different proportions than those used in the official icon)—EDIT: I just checked, and the Element brand guidelines have been updated to match the Element iOS icon but still not the Element macOS one—and AFAIK these guidelines are actually significantly more prescriptive than Nextcloud's. If we don't follow Apple's design guidelines, what would be next, a full-color Nextcloud logo in the Finder sidebar? Incidentally, @claucambra mentioned liking the slightly tweaked Finder-sidebar icon in #4367, so there's that. While I am open to making additional tweaks to this Pull Request, more broadly I would like the final version to still follow Apple's macOS icon design guidelines. Also, in terms of resolving any conflict between Apple's icon design guidelines and Nextcloud's (apparently non-formalized) brand guidelines, I would be most happy to defer to the opinions of Nextcloud team members who regularly use Nextcloud Desktop for Mac (as @karlitschek and @marcoambrosini apparently do). |
Codecov Report
@@ Coverage Diff @@
## master #4631 +/- ##
==========================================
- Coverage 56.52% 56.42% -0.10%
==========================================
Files 138 138
Lines 17071 17071
==========================================
- Hits 9649 9632 -17
- Misses 7422 7439 +17
|
Here are the relevant bits from Apple's guidelines (bold text from original; italics added):
And here's the updated icon (left) compared to the previous version (right): It turned out that I had also not correctly set up the main drop shadow (but hadn't noticed this due to the dark background). If you'd like me to tweak any other parameters with the shadow (like the blur or the opacity), it should be extremely straightforward for me to do so. Also IMHO the background gradient could stand to be ever so slightly more pronounced, though this is the version of the gradient you selected from the set in the original Issue. Partially I think the softer shadow visually flattens the gradient behind it, and enhancing the gradient would counteract that effect. (The main complication with revisions is that I haven't squashed my commits because doing so would break the inline images, though I guess I could replace the embeds with versions mirrored as GitHub gists.) Also, yes, I noticed that I messed up my merge commit. Could you figure out fixing it for me? Thanks! |
@elsiehupp I cleaned up history and made it only one commit with your changes in it (even double checked that I was getting the proper icon result at the end) |
Signed-off-by: Elsie Hupp <[email protected]>
Kudos, SonarCloud Quality Gate passed! |
AppImage file: Nextcloud-PR-4631-30970547fc0c09da31303b40f84dfb9e29d8a3d5-x86_64.AppImage |
Okay, I guess you (@mgallien) liked it as-is! But everyone else feel free to let me know if you'd like tweaks to this or any other icon! (I could also with not too much difficulty make slight variations of this for the most popular Linux icon themes.) Note that this version is extremely "parametric" in a way that makes it extremely conducive to subtle tweaks, though I didn't annotate it quite as much as I could, and some aspects could probably be made less redundant. (I could always do another pull request adding a bunch of inline XML comments to help other people work with the SVG source.) |
Fixes #3294 (kind of: I say "kind of" because it still needs to be integrated into CMake)
This is a 100% vector icon that I largely wrote in the SVG markup itself, and I tried to format it so it's easier to understand and easier to tweak. I mention this because AFAIK Inkscape doesn't have a GUI for the sorts of filter effects I used, and editing this in Inkscape would probably wildly reformat the syntax. By contrast if you tweak individual parameters in the code itself, the Git diff should be much more legible.
The only really messy bit is the "squircle" outline itself, as SVG doesn't include a superellipse element, so it's just baked as a path instead.
If you'd like me to make additional platform-specific variations of this with outlines other than the macOS Big Sur "squircle", just let me know which ones you'd like me to do.
EDIT
Here's an embedded preview of the file in this pull request: