-
Notifications
You must be signed in to change notification settings - Fork 218
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
Muted Speaker Icon update #1691
Muted Speaker Icon update #1691
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also need to add src/res/mutediconorange.png
to Jamulus.pro
, somewhere around line 752.
6481b14
to
1ef7caa
Compare
@softins , updated as requested |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on Arch Linux. Looks good and is way better than before (as it was just an orange square for me, probably due to missing font glyphs).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
When revisiting this after the release, to change to the ear icon, we should clarify the What's This text: the icon showing just means the client cannot hear you. It might not be because they have you muted (which is what the text says), because it also shows if you turn your own fader right down.
The 'muted speaker' in jamulus-3.8.0rc1looks good and works good. Please keep it this way. |
Fixes #1684