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

CHANGE(ui): Make lock icons fit with other icons #5455

Merged
merged 1 commit into from Feb 6, 2022
Merged

CHANGE(ui): Make lock icons fit with other icons #5455

merged 1 commit into from Feb 6, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jan 18, 2022

The current channel lock icons introduced for 1.4.x use a line-art
style. This stands out against the other icons that show in the
channel tree (user mute status, comments, friend hearts, etc.) because
they are all filled in.

The included changes to the icons are:

  • Switch to a filled-in style
  • Set opacity to 75% to match other icons
  • Slightly shrink them, as they look somewhat large compared to the
    channel comment icon.

Checks

The current channel lock icons introduced for 1.4.x use a line-art
style. This stands out against the other icons that show in the
channel tree (user mute status, comments, friend hearts, etc.) because
they are all filled in.

The included changes to the icons are:
 - Switch to a filled-in style
 - Set opacity to 75% to match other icons
 - Slightly shrink them, as they look somewhat large compared to the
   channel comment icon.
@Krzmbrzl
Copy link
Member

Could you provide a screenshot of your changes?

@ghost
Copy link
Author

ghost commented Jan 18, 2022

Light theme:
image

Dark theme:
image

@Krzmbrzl Krzmbrzl merged commit 71ecd12 into mumble-voip:master Feb 6, 2022
@Krzmbrzl
Copy link
Member

Krzmbrzl commented Feb 6, 2022

Thanks for these changes as well 👍

@Krzmbrzl
Copy link
Member

Krzmbrzl commented May 8, 2022

Turns out this can't be easily backported without adding it to the old mumble-theme repository. However, I don't think that it is worth going down that route. Thus, this will simply be shipped once we release 1.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants