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

Update icon to macOS 11 style #1054

Merged
merged 3 commits into from
Nov 21, 2020
Merged

Update icon to macOS 11 style #1054

merged 3 commits into from
Nov 21, 2020

Conversation

jasonlong
Copy link
Contributor

@jasonlong jasonlong commented Jul 5, 2020

Way back in #1, I submitted the currently used app icon for the then-new Yosemite version of OS X. The new icon styling for Big Sur is a pretty big departure from the current styling and it makes sense to have MacVim continue to feel current going forward (once macOS 11 ships).

This PR includes a new MacVim.icns that looks like this:

Option 1

I didn't include any document-specific .icns files because I read the following on this page, but I'm happy to help generate them if I'm misinterpreting.

To compose a document icon, the system combines your app icon with the familiar folded-corner icon shape.

@jasonlong
Copy link
Contributor Author

jasonlong commented Jul 5, 2020

Here's an alternative version that aligns with the original icon better.

Option 2

@ychin
Copy link
Member

ychin commented Jul 5, 2020

Thanks for the update! Between the original and alt version is there one you prefer?

I appreciate the update but I may let this sit a little bit when macOS 11 Big Sur is still in beta to see if others have opinions and let me take a look at Apple's guidelines in the meantime.

@jasonlong
Copy link
Contributor Author

Sounds good @ychin. I think it makes sense to gather more feedback during the Bir Sur beta period so that everyone feels confident by the time the macOS 11 ships.

I initially liked the first version better, but after refining the second version more I prefer that one. Since it stays more true to the original Vim logo, that's probably the more sensible (and least controversial) direction.

@ychin
Copy link
Member

ychin commented Jul 6, 2020

Just some immediate feedback / thoughts: I like option 1 better. The second option aligns better with the original Vim logo but it looks… kind of off? Like we just introduced a white squircle border for no good reason with yet another solid color border inside it. It in a way reminds me of Vim in that we pile on new features but trying to keep backwards compatibility at all cost, but seems like at first glance at least the first icon seems cleaner.

The other thing that jumped at me was how strong the 3D shading shadow is. The drop shadow is fine, but the "V" almost feels too… 3D to me. I'm still not sure if I'm put off by it itself or just the new Big Sur aesthetics, and I'm going through the list of icons and the official recommendation to see how this fits in, and so far it seems… inconclusive to me and seems like some icons have this strong 3D shading effect (e.g. Messages, App Store) while others still look flat with a squircle around the icon (e.g. Apple TV, Apple Music). So I'm not sure yet.

Maybe will need to wait till I (and others) get a beta version installed and can try them side-by-side, and to let it sink in a bit first.

@jasonlong
Copy link
Contributor Author

Here's another variant on the first option, but with the shading dialed way down.

Option 3

@eirnym
Copy link
Contributor

eirnym commented Jul 10, 2020

I perfect the most Option 2. I like that macOS is going back to 3d effects in icons

@jasonlong
Copy link
Contributor Author

Option 4

@UncleBill
Copy link

👍 for Option 2 but with the shading dialed way down == Option 4 with white background color.

@jasonlong
Copy link
Contributor Author

Sorry for the delay - I was on vacation last week.

Option 5

@ychin
Copy link
Member

ychin commented Aug 18, 2020

Hi, @jasonlong , do you mind if you could provide all the different options' icn files? I am trying to look at them in Big Sur beta just to be able to tell how they look like relative to the other icons, and it would be easier if you could provide the icns either in individual commits or just another way to attach them. Thanks!

@ychin
Copy link
Member

ychin commented Sep 8, 2020

Just pinging again. @jasonlong , do you mind providing the different options as icn files? I would like to try them out in macOS 11 beta to see what they look like if possible.

@jasonlong
Copy link
Contributor Author

@ychin Sorry for the delay. Here's a link to the different .icns options:

https://macvim-icns-riselanjk.vercel.app/

@ychin
Copy link
Member

ychin commented Oct 4, 2020

Just trying out to see how the 5 options look relative to the other icons in the Big Sur beta:

Option 1:

MacVim Dock icn 1

MacVim Dock 2 icn 1


Option 2:

MacVim Dock icn 2

MacVim Dock 2 icn 2


Option 3:

MacVim Dock icn 3

MacVim Dock 2 icn 3


Option 4:

MacVim Dock icn 4

MacVim Dock 2 icn 4


Option 5:

MacVim Dock icn 5

MacVim Dock 2 icn 5


After seeing it side-by-side, I think I'm still not a huge fan of Option 5 (even though it seems to be popular with the limited voting we have). In particular, most default icons seems to have a simple "background color + thing in the middle" design, where as Option 2/5 is a thing within a thing within a thing. But I feel like Option 5 looks a little better in the 2nd screenshot (next to the Home app) compared to the first one. The white background also makes the V's contrast hard to see. In that sense I actually like Option 2 more than 5.

I think Option 3 remains my favorite, whereas I think Option 4 is pretty bad.

Also, Apple seems to not be terribly consistent in their 3D shading effect. App Store / Messages' icons use a fairly strong shading effect, but Music / News look pretty flat. The official HIG guidelines only suggests using it so 🤷 .

@jasonlong what do you think? Which one do you personally think look the best?


Going to do a second round of voting below. Feel free to both vote up or down on more than one icon! Kind of doing a poor-man's approval/ranked voting this way.

@ychin
Copy link
Member

ychin commented Oct 4, 2020

Vote Option 1:

MacVim Dock icn 1

MacVim Dock 2 icn 1

@ychin
Copy link
Member

ychin commented Oct 4, 2020

Vote option 2

MacVim Dock icn 2

MacVim Dock 2 icn 2

@ychin
Copy link
Member

ychin commented Oct 4, 2020

Vote Option 3

MacVim Dock icn 3

MacVim Dock 2 icn 3

@ychin
Copy link
Member

ychin commented Oct 4, 2020

Vote Option 4

MacVim Dock icn 4

MacVim Dock 2 icn 4

@ychin
Copy link
Member

ychin commented Oct 4, 2020

Vote Option 5

MacVim Dock icn 5

MacVim Dock 2 icn 5

@jasonlong
Copy link
Contributor Author

Thanks for putting these comparisons together @ychin.

Apple seems to not be terribly consistent in their 3D shading effect.

Agreed, this has made these explorations challenging trying to align the treatment w/ the default icons.

I think Option 3 remains my favorite

Seeing them all in context, I tend to agree with you. Sometimes I feel like that one is too flat, but I feel like that one looks better than the really shaded Option 1. If we want to try some shading options in-between, I'm happy to build some.

@bingobongo
Copy link

The V a little further to the right?

v3-bit-right

@jasonlong
Copy link
Contributor Author

@ychin With Big Sur out, can I help at all getting the icon update ready to ship? I can update this PR to use Option 3 above if you like.

@ychin
Copy link
Member

ychin commented Nov 18, 2020

@jasonlong Yes please update to Option 3. I was a little busy with work and got a little behind on making sure MacVim is ready for Big Sur!

Just to make sure #1054 (comment) is addressed. I personally think the alignment in the original Option 3 looks fine, but it's because of how the "V" looks in the font that it looks a little offset to the left. I think it's fine to leave it as is, but if you want to shift it to the right I'm fine with that too.

@jasonlong
Copy link
Contributor Author

jasonlong commented Nov 19, 2020

Ok, the branch has been updated here and I tried to better optically center the mark in the container. The final version looks like this:

MacVim logo

@ychin ychin added this to the snapshot-167 milestone Nov 21, 2020
@ychin ychin merged commit f3b2d6c into macvim-dev:master Nov 21, 2020
@ychin
Copy link
Member

ychin commented Nov 21, 2020

Merged! Thanks for the update.

@jasonlong jasonlong deleted the big-sur-icon branch November 21, 2020 21:29
ychin added a commit that referenced this pull request Dec 11, 2020
Updated to Vim 8.2.2127.

*Note*: This release doesn't natively support Apple Silicon / M1 yet, but
does work under Rosetta. See below.

Features
====================

Big Sur / macOS 11
--------------------

- MacVim now has an updated app icon (#1054), and preference pane / toolbars
  have been updated to match Big Sur's interface guidelines. (#1128)
- Fixed Touch Bar warnings when launching MacVim from the terminal.
  #1114
- SF Symbol characters will show up properly as double-width as most of
  these icons would take up more than one column. Note that these
  characters are specific to macOS and would not work in other
  platforms. #1129

Renderer / scrolling performance improvements
--------------------

The Core Text renderer has been rewritten and is now much faster!
Scrolling should not stutter and lag like before and generally it should
feel a lot smoother now. Thanks to Sidney San Martín (@s4y) for the
contribution. #858

With this change, the non-Core-Text renderer is now considered
deprecated. The old renderer is accessible either through the Preference
Pane (under Advanced) or by setting the defaults "MMRenderer" to 0. It
works for now, but it will be removed in a future update as it has known
bugs.

Menu Localization
--------------------

Menus are now localized, see `:h langmenu` for how Vim menu localization
works. You can use `set langmenu=none` to turn it off if you would like. #1099

There still exists a few menu items that are not localized, and the
general MacVim GUI is not localized as well. If you would like to help,
please use #1102 to coordinate with MacVim dev team.

Getting help / Help menu
--------------------

- Help menu's search bar now searches Vim documentation as well! See
  #1095.
- Vimtutor is now bundled with MacVim, and you can access vimtutor from
  the Help menu (#1096). There is also a link to the latest release
  notes as well (#1131).

General
====================

- This release does not contain a native universal app for Apple Silicon
  / M1 Macs yet. The release binary will still work under Rosetta, which
  should provide enough performance, but if you use Python/etc plugins,
  you need to make sure you have x86 versions of Python/etc installed
  (which is still the default for Homebrew as of this release).

  MacVim is buildable under Apple Silicon, so if you need a native
  binary, you could build it yourself by downloading the source from the
  Github repository. See #1136 for progress on releasing a universal app
  for Apple Silicon.

- MacVim has enabled the Github Discussions feature, which serves as a
  good spot for general discussions and questions. See
  #1130 and check it
  out!

Fixes
====================

- Launching MacVim from the Dock with locales that use "," for decimal
  separators now works correctly. #11 (Vim 8.2.1738)
- `WinBar` menus (which are used by plugins like vimspector) now work
  properly and don't create dummy menu items. #918
- Using `:browse tabnew` no longer crashes MacVim in terminal mode.
  #1107 (Vim 8.2.1842)

Misc
====================

- Scripting languages versions:
    - Python is now built against 3.9, up from 3.8.
    - Lua is now built against 5.4, up from 5.3.

Compatibility
====================

Requires macOS 10.9 or above.

Script interfaces have compatibility with these versions:

- Lua 5.4
- Perl 5.18
- Python2 2.7
- Python3 3.9
- Ruby 2.7
ychin added a commit that referenced this pull request Dec 12, 2020
Updated to Vim 8.2.2127.

*Note*: This release doesn't natively support Apple Silicon / M1 yet, but
does work under Rosetta. See below.

Features
====================

Big Sur / macOS 11
--------------------

- MacVim now has an updated app icon (#1054), and preference pane / toolbars
  have been updated to match Big Sur's interface guidelines. (#1128)
- Fixed Touch Bar warnings when launching MacVim from the terminal.
  #1114
- SF Symbol characters will show up properly as double-width as most of
  these icons would take up more than one column. Note that these
  characters are specific to macOS and would not work in other
  platforms. #1129

Renderer / scrolling performance improvements
--------------------

The Core Text renderer has been rewritten and is now much faster!
Scrolling should not stutter and lag like before and generally it should
feel a lot smoother now. Thanks to Sidney San Martín (@s4y) for the
contribution. #858

With this change, the non-Core-Text renderer is now considered
deprecated. The old renderer is accessible either through the Preference
Pane (under Advanced) or by setting the defaults "MMRenderer" to 0. It
works for now, but it will be removed in a future update as it has known
bugs.

Menu Localization
--------------------

Menus are now localized, see `:h langmenu` for how Vim menu localization
works. You can use `set langmenu=none` to turn it off if you would like. #1099

There still exists a few menu items that are not localized, and the
general MacVim GUI is not localized as well. If you would like to help,
please use #1102 to coordinate with MacVim dev team.

Getting help / Help menu
--------------------

- Help menu's search bar now searches Vim documentation as well! See
  #1095.
- Vimtutor is now bundled with MacVim, and you can access vimtutor from
  the Help menu (#1096). There is also a link to the latest release
  notes as well (#1131).

General
====================

- This release does not contain a native universal app for Apple Silicon
  / M1 Macs yet. The release binary will still work under Rosetta, which
  should provide enough performance, but if you use Python/etc plugins,
  you need to make sure you have x86 versions of Python/etc installed
  (which is still the default for Homebrew as of this release).

  MacVim is buildable under Apple Silicon, so if you need a native
  binary, you could build it yourself by downloading the source from the
  Github repository. See #1136 for progress on releasing a universal app
  for Apple Silicon.

- MacVim has enabled the Github Discussions feature, which serves as a
  good spot for general discussions and questions. See
  #1130 and check it
  out!

Fixes
====================

- Launching MacVim from the Dock with locales that use "," for decimal
  separators now works correctly. #11 (Vim 8.2.1738)
- `WinBar` menus (which are used by plugins like vimspector) now work
  properly and don't create dummy menu items. #918
- Using `:browse tabnew` no longer crashes MacVim in terminal mode.
  #1107 (Vim 8.2.1842)

Misc
====================

- Scripting languages versions:
    - Python is now built against 3.9, up from 3.8.
    - Lua is now built against 5.4, up from 5.3.

Compatibility
====================

Requires macOS 10.9 or above.

Script interfaces have compatibility with these versions:

- Lua 5.4
- Perl 5.18
- Python2 2.7
- Python3 3.9
- Ruby 2.7
@diogocampos
Copy link

I know this issue is closed, but I'd like to suggest something like the "overhang" versions proposed for VS Code in this comment, maybe with the bottom and/or the right tip of the V sticking out a bit from the squircle.

Sadly I don't have the design skills to mock that up. Would anyone care to take a shot at it?

@ychin
Copy link
Member

ychin commented Dec 14, 2020

Hi, I appreciate the comment but I'm not sure I'm buying the overhang design. Also, the Vim's "V" has always been level at the top, so to make it overhang you would have to change the iconography of it as well to have some part poke out. I think the overhang is usually designed for an object laid on top of the squircle, not for a letter (https://developer.apple.com/design/human-interface-guidelines/macos/icons-and-images/app-icon).

If you have more thoughts though feel free to discuss at #1140 since people love to discuss design changes :) .

@eirnym
Copy link
Contributor

eirnym commented Dec 15, 2020

the default icon is highly new for me and an icon in docker looks very different than on a GitHub page.

I love the diamond was before and new icon looks flat to me.

Is it possible in macOS to change an icon from app settings like I can do it in iOS?

If yes, is it possible to add all icons and let user decide which icon to show? I somewhat want an option, to decide.

@ychin
Copy link
Member

ychin commented Dec 15, 2020

@eirnym See the discussion on #1140 (we should focus on using that thread to discuss icon change). No, macOS doesn't allow for multiple icons for an app so it has to be one. You could change it yourself in Finder and the OS will remember it, but we don't have a way for MacVim to bundle multiple icons and let you choose, unfortunately.

With most things icon or interface change related, I think it takes time to adapt. I'm open to see how the new icon is received in a couple more weeks at least to see if people got used to it.

@eirnym
Copy link
Contributor

eirnym commented Dec 15, 2020

@ychin Thank you for a link and an information. Here I wanted to ask a technical question, not discuss my personal preferences, and you gave me a good answer.

@timoxley
Copy link
Contributor

timoxley commented Jan 5, 2021

For anyone arriving in this thread wanting to say that they hate the new icon

I hated it 24 hours ago, but now? Looks fine. Whatever. Just give it a few days.

@samusz
Copy link

samusz commented Jan 29, 2021

@ychin I use brew to install macvim. (as a formula in HomeBrew parlance/terminology) with brew install macvim
(and update to the last version with brew upgrade macvim )

Here I won't discuss homebrew's merit.
I just wanted to point out that it may be possible to have an option in brew to get alternate icons as I believe have seen this for another app in brew... (can't remember which) This could be a workaround to MacOS 11 BigSur limitations ?

I'll try to get some time to investigate this (as I am not a brew wizard)

@ychin
Copy link
Member

ychin commented Feb 8, 2021

Homebrew doesn't build from source by default. It builds the formulas into bottles (cached binaries) that you download from their server when you install a package, so this would only be an option if you build MacVim from source using Homebrew, and I think that's just too niche an option.

@eirnym
Copy link
Contributor

eirnym commented Feb 9, 2021

@ychin this option exists in MacPorts and I'd like to see it. In home-brew it was cutoff as they have a policy to support only one configuration.

PS: I have questions on MacVim to provide better support in MacPorts

@ychin
Copy link
Member

ychin commented Feb 11, 2021

What would MacVim need to do to support MacPorts better? I used to use MacPorts but moved away from it, and it seems like MacPorts usually falls behind MacVim releases by a couple versions (https://ports.macports.org/port/MacVim/summary). Looking at metrics on that (which to be fair I don't know how accurate that i), it also appears the usage is quite low, which is why I'm not sure if I want to spend too much time supporting it (homebrew-core's MacVim formula is maintained by Homebrew devs and volunteers themselves).

Anyway let's make a new issue if you want to add more support for MacPorts, which I'm not against. I do question if it's worth spending effort to make the icon customizable just for MacPorts though. It's a pretty niche within niche use case.

@eirnym
Copy link
Contributor

eirnym commented Feb 11, 2021

I want to take maintainership of MacVim in MacPorts

@ychin
Copy link
Member

ychin commented Mar 8, 2021

macvim-dev doesn't own the MacPorts definition. You may want to go to https://ports.macports.org/port/MacVim/summary and ask the maintainer instead.

FWIW I don't particular support having customizable icons for the reasons I already discussed (majority of users would not be installing MacVim in a way that supports icon customization, and also I don't really think that's good for help identifying the app if we give too many icons to choose from).

@oseiberts11
Copy link

"it seems like MacPorts usually falls behind MacVim releases by a couple versions"

maybe that explains why I just recently got the new icon...
I'm not sure why a new icon was needed, but now it looks imho totally out of place with the Vim icons on my other computers. It hardly looks like Vim at all, really. Where did the "im" go? Was that really needed to drop it?

@jasonlong
Copy link
Contributor Author

jasonlong commented Apr 21, 2021

@oseiberts11 The "im" hasn't been part of the MacVim logo for a long time (at least 6 years).

@oseiberts11
Copy link

Interesting, you're right! I guess that the change somehow made it more "in my face" :)

@ychin
Copy link
Member

ychin commented Apr 22, 2021

Also, are you using macOS 11 Big Sur @oseiberts11 ? If you are, I would be surprised if you find the old icon preferable to the new one. If you aren't, unfortunately, as has been discussed on this thread already, there isn't an easy way to support multiple icons, and as a project we should be supporting the latest macOS version as top priority while remaining backwards compatible with older versions.

@oseiberts11
Copy link

oseiberts11 commented Apr 22, 2021

@ychin No, I don't (yet). Company policy has not approved it yet, I think... So this is on my work computer, that's why I notice that the icon differs so much from other Vim versions, because for my personal use I use the generic Unix version of Vim with GTK GUI.
Edited to add: Personally, to me it isn't that much of a priority that the icon fits with the other icons of the operating system. For me it should be recognizable as Vim, because I use Vim on other operating systems as well, and so the icon should resemble the icon of the Unix/X version (is there still an Amiga version? If so, that icon would surely also be similar). And with the green background being rotated from before, it doesn't. And it leaves so much "empty green space" below the V that it becomes much more obvious that the "im" is missing, which is much less obvious in the previous icon.

@eirnym
Copy link
Contributor

eirnym commented Mar 22, 2022

@oseiberts11 you can use your own icon locally by copying and pasting it into information window.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants