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: Refresh and add color to session icons #1283

Merged
merged 1 commit into from
May 28, 2024

Conversation

newhoa
Copy link
Collaborator

@newhoa newhoa commented May 18, 2024

We did these in elementary-xfce, was wondering if you all would be interested in them?

The dark icons were a little close to disabled-button looking and adding color helps with that and makes them easier to see on dark backgrounds. Also reboot uses rounded arrow.

Questions:

  1. Only 32px and 48px exist, do you all want 24px and 16px added?
  2. system-restart and system-suspend are non-FDO names, remove these or think about it in another PR? Nevermind, after checking the code base these are uses in elementary.
  3. We also did suspend and hibernate, and switch user if you all want them (none are FDO, but are commonly used) (also, suspend was a different design than current, didn't want to change it without asking):

session-prop

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.

These are great, much nicer looking!

@danirabbit
Copy link
Member

@newhoa I would totally merge sleep and hibernate ones. The snowflake looks great and I love the color there.

We're using a moon metaphor in quick-settings also, but we're using stars instead of "z"s since that seems like a locale specific thing:

Screenshot from 2024-05-28 11 56 04

@danirabbit danirabbit merged commit fa22659 into elementary:main May 28, 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