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

Places: Refresh mailbox icons #1314

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

newhoa
Copy link
Collaborator

@newhoa newhoa commented Sep 23, 2024

  • Use green and purple arrows for mailbox- inbox and outbox, consistent with the recently updated colors to mail send/receive (also matches document send, browser download colors and new rounded-arrow style).
  • Use slightly more front-facing perspective.
  • Add 24px icons. Honestly not sure if the 24px ones are used anywhere, but I was going to add them to elementary-xfce so thought I'd put them in here too. If they aren't needed I can remove them.
  • Fixed an issue from last mail PR, removed leftover arrow layer from mail-send-receive-symbolic.

Symbolic icons use the more top-down perspective. Is this okay? I couldn't come up with a better design there and think they look nice and clear as-is, even if it is a little inconsistent.

Any feedback would be appreciated. Thanks!

Using Evolution as an example:

Current:
current

Proposed:
propPR2

Use green and purple arrows for `mailbox-` inbox and outbox,
consistent with use of same colors in mail send/receive.

Use more forward-facing perspective.

Add 24px icons.

Fix: Delete duplicated arrow from `mail-send-receive-symbolic`.
@newhoa
Copy link
Collaborator Author

newhoa commented Sep 27, 2024

Played with the perspective of the 24px one a little more, let me know if these alternatives are preferred and I can update the PR:

Current PR:
mail-inbox
mail-mailbox
mail-outbox

Alternatives:
mail-inbox00
mail-mailbox00
mail-outbox00


16px for reference:
mail-inbox
mail-mailbox
mail-outbox

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.

1 participant