8000 Replaced the old Atom document icon with the Big Sur icon by andylin2004 · Pull Request #22421 · atom/atom · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Mar 3, 2023. It is now read-only.

Replaced the old Atom document icon with the Big Sur icon #22421

Merged
merged 3 commits into from
May 19, 2021

Conversation

andylin2004
Copy link
Contributor

Issue or RFC Endorsed by Atom's Maintainers

#21128 (for parity purposes)

Description of the Change

I replaced the older default icon for Atom with the Big Sur-style icon for Atom for Macs only. Windows icons is still the same.

Alternate Designs

Keeping the icon as is: this would fragment the Atom icon use on macOS (between the old and new icons)
Adding the "Use system generated icons" variable and setting to YES (to load the Atom icon as the document icon automatically): this would not replicate the previous file icon behavior where the Atom icon appeared on the title bar and in the filesystem. In addition, I don't know how far back OS-wise this workaround will go back, as the lowest version Atom supports is OS X 10.10.
Using the new layered icon technique as depicted in https://developer.apple.com/news/?id=5i6jlf4d : adding new Document Type Identifiers is not worth the time, as Atom uses a single file icon and to repeat this process for many file types would be time consuming. I am open to doing this as a pet project on the side. In addition, I don't know how far back OS-wise this technique will go, as the lowest version Atom supports is OS X 10.10.

Possible Drawbacks

The document icon is based off the one in macOS Big Sur, meaning that any user using macOS Catalina or lower will see the newer style document icon, instead of the one used in macOS Catalina and lower.

Verification Process

The verification process is not needed, as this is just an icon swap with the same file extension (between the icon being replaced and replacement icon).

Release Notes

The document icon for files openable by Atom and on the title bar on macOS has been changed to the new app icon to reflect the style in macOS Big Sur.

@ghost
Copy link
ghost commented May 13, 2021 via email

Copy link
Contributor
@ThatXliner ThatXliner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but why did you delete so much in the package-lock.json?

@andylin2004
Copy link
Contributor Author

That was unintentional. I have committed the old package lock back

@ThatXliner
Copy link
Contributor
ThatXliner commented May 13, 2021

That was unintentional. I have committed the old package lock back

The easiest way is to uncommon those, remove the changes on package-lock.json, commit and force-push. That way, I can be sure it's the same lock file. Icon looks good, though!

…at issue and reverts the correct package lock.
@ThatXliner
Copy link
Contributor

Yep, that seems about right

Copy link
Contributor
@ThatXliner ThatXliner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ghost
Copy link
ghost commented May 16, 2021

Hi, I think it would be better to use the system generated icons.

Otherwise your icon is blurry on my system and the icon background is not fully transparent.

@ThatXliner
Copy link
Contributor

Hi, I think it would be better to use the system generated icons.

Otherwise your icon is blurry on my system and the icon background is not fully transparent.

Can you show us a screenshot?

@ghost
Copy link
ghost commented May 16, 2021

The icon is a bit blurry, especially at the edges:

screenshot

If you zoom in, we can see that the background of the icon is slightly gray (this is also notable in the capture above):

screenshot 2

Moreover, depending on the size of the icon, we have different styles (3 styles, which is a lot):

icon

Wouldn't it be better to use the files icons that are automatically generated by the system? (as does Sketch, Pixelmator and soon Transmission)

Since the current icon is simple, there is no reason to create a specific one:

https://developer.apple.com/news/?id=5i6jlf4d
By default, if you don’t specify a document icon for a file type in your app, macOS will automatically create one for you by compositing your app’s icon together with the correct extension name. This is a common pattern for imported document types or non-proprietary file formats that your application can open such as MP3, JPG or PNG.

@ThatXliner
Copy link
Contributor

The description of this PR says

Adding the "Use system generated icons" variable and setting to YES (to load the Atom icon as the document icon automatically): this would not replicate the previous file icon behavior where the Atom icon appeared on the title bar and in the filesystem. In addition, I don't know how far back OS-wise this workaround will go back, as the lowest version Atom supports is OS X 10.10.

but ok

Copy link
Contributor
@sadick254 sadick254 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@sadick254 sadick254 merged commit e417208 into atom:master May 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0