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

Actions: Make mail-message-new uniform, add 16px #1312

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

newhoa
Copy link
Collaborator

@newhoa newhoa commented Sep 18, 2024

mail-message-new symbolic did not match the 24px version (which was updated more recently in 367c69b). So updated the symbolic icon to match.

Also added a 16px version.


I worked on a 16px mail-send-receive to match the symbolic version. Wasn't sure whether to include it in this PR. If it's wanted I can add it here or in another PR, just let me know.

Currently, no fullcolor version of this icon exists. I tried 3 options:

  1. Orange and Green based on this older version, which elementary-xfce still uses.
  2. Purple and Green based on document-send and browser-download
  3. Purple and Blue based on import, export (although mail does already use these colors for forward and reply)

mail-send-receive mail-send-receive-send-download mail-send-receive

mail-send-receive-prop

Anyway, let me know if you all want this one and if so what version and whether to include it in this PR or another.

Thanks!

Copy link
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

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

Thanks for keeping these consistent!

And yeah totally interested in the color mail-send-receive! I think the purple and green one to be consistent with upload/download looks really good personally!

@danirabbit danirabbit merged commit e77d29b into elementary:main Sep 18, 2024
3 checks passed
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.

2 participants