Skip to content

feat: update icon for macOS#838

Merged
ouuan merged 4 commits intocpeditor:masterfrom
swiftqwq:update-macos-icon
May 23, 2021
Merged

feat: update icon for macOS#838
ouuan merged 4 commits intocpeditor:masterfrom
swiftqwq:update-macos-icon

Conversation

@swiftqwq
Copy link
Copy Markdown
Member

Description

Update icon for macOS.

Related Issues / Pull Requests

Closes #834

Motivation and Context

Make the icon follow Apple's Human Interface Guidelines.

How Has This Been Tested?

Screenshots (if appropriate)

macos-icon

Checklist

  • If the key of a setting is changed, the old attribute is updated or it is resolved in SettingsUpdater.
  • If there are changes of the text displayed in the UI, they are wrapped in tr() or QCoreApplication::translate().
  • If needed, I have opened a pull request or an issue to update the documentation.
  • If these changes are notable, they are documented in CHANGELOG.md.

Additional text

@coder3101 coder3101 requested review from coder3101 and ouuan May 22, 2021 05:53
@coder3101
Copy link
Copy Markdown
Member

This icon sure looks more consistent with macOS and apple design, the question is, should we have different icon for different OS? What is your opinion @ouuan on this?

@ouuan
Copy link
Copy Markdown
Member

ouuan commented May 22, 2021

The app icon in the Dock looks good, but do we need to change the in-app icon (which is displayed on the home page of the preferences window and some other places)? @swift-zym What do you think?

@coder3101
Copy link
Copy Markdown
Member

The app icon in the Dock looks good, but do we need to change the in-app icon (which is displayed on the home page of the preferences window and some other places)? @swift-zym What do you think?

In my opinion changing launcher icon is enough. This will also remove those conditional OS macros from code which makes it looks like we have different icon for different OS.

Please share your opinion @swift-zym ?

@swiftqwq
Copy link
Copy Markdown
Member Author

@coder3101 @ouuan I agree that we needn't change the in-app icon,but I think we should keep the code in appwindow.cpp and main.cpp, it will change the dock icon when running.

@ouuan ouuan force-pushed the update-macos-icon branch from 102abec to 7516c8e Compare May 23, 2021 02:10
@ouuan ouuan force-pushed the update-macos-icon branch from 7516c8e to 137a89b Compare May 23, 2021 02:16
Copy link
Copy Markdown
Member

@ouuan ouuan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@ouuan
Copy link
Copy Markdown
Member

ouuan commented May 23, 2021

@allcontributors add @swift-zym as a contributor for code and design.

@ouuan ouuan enabled auto-merge (squash) May 23, 2021 02:21
@ouuan ouuan merged commit d7bc62b into cpeditor:master May 23, 2021
@swiftqwq swiftqwq deleted the update-macos-icon branch May 23, 2021 05:06
@coder3101
Copy link
Copy Markdown
Member

Does the old icon not at all used by macOS now?

@ouuan
Copy link
Copy Markdown
Member

ouuan commented May 23, 2021

Does the old icon not at all used by macOS now?

5fb109e

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.

Update icon for macOS

3 participants