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

Show if the client is locally muted in the window title #476

Closed
dingodoppelt opened this issue Jul 26, 2020 · 22 comments
Closed

Show if the client is locally muted in the window title #476

dingodoppelt opened this issue Jul 26, 2020 · 22 comments
Labels
feature request Feature request

Comments

@dingodoppelt
Copy link
Contributor

Sometimes I forget to unmute myself, especially when the client window is not visible or I sometimes just want to know, if I am muted or not without bringing the client window to the front to be able to check.
I've created a quick fix for this and created a pull request for everybody to review.
cheers, nils

@corrados
Copy link
Contributor

I do not think putting this information in the server title is a good idea. We already have the recording information put there.

@dingodoppelt
Copy link
Contributor Author

Hi,
it is shown in the window title, not the server title. That way you can see if you are muted even if the client window is not visible and while you are playing your instrument you cannot reach for your mouse to bring the window to the front. You can see it in the taskbar.
selfmute

@corrados
Copy link
Contributor

corrados commented Aug 3, 2020

Thanks for the clarification. We had a discussion about the mute state before, see: #187 (comment)
WolfganP suggests something blinking which would be more obvious. Maybe color code your own fader to make it more visible that you are muted.

We had a very lengthy discussion about the name "Mute Myself", see #187. In your implementation you simply use the word "MUTED". But this is not exactly what is happening. Since you hear your own signal even if MUTED is displayed. What actually is done is that your stream to the server is stopped when you activate Mute Myself.

I would prefer some color coding but this would mean some more coding. The advantage of your solution is that its implementation is fairly simple. Please give me some more time to think about this feature and do some more brainstorming and looking at the code. I'll get back in this thread sometime soon.

@corrados corrados added the feature request Feature request label Aug 3, 2020
@storeilly
Copy link

storeilly commented Aug 3, 2020 via email

@corrados
Copy link
Contributor

corrados commented Aug 3, 2020

Thanks for your suggestion. But the intended use case of Mute Myself is that if a session is ongoing and you want to find out, e.g., which chords are played, you want to hear the others and your instrument but you do not want your signal to be sent to the server. See the original request from Simon: #75 and also this post: #75 (comment)

@dingodoppelt
Copy link
Contributor Author

dingodoppelt commented Aug 3, 2020

hi,
thanks for your response. I would love to see this issue addressed, since it is still an issue for me not knowing outside of the client window if I am muted or not. (with my implementation I do, that's why I'm using it already ;) That is why I put the Information in the window title so it shows up in the taskbar. I'd love to see flashing lights in the taskbar (maybe trayicon/floating overlay?) and so on, but I couldn't figure out how to do that on all platforms since Qt doesn't offer this functionality, I think. The most important point for me is that I get the info about my mute status even when I'm looking at sheet music or a zoom feed and the jamulus window itself isn't visible.
Since my implementation is far from elegant and the code isn't consistent with your design I already know what I will be improving, once I know how to call functions from a different namespace :)
thanks again volker, and keep up the great work!

EDIT: I used the term "MUTED" since regular users of jamulus generally refer to this behaviour as "being muted". we've had those situations on the world jam (people being told "you're muted" after not being heard for a good amount of the piece being performed) and in daily use as well.

@corrados
Copy link
Contributor

corrados commented Aug 9, 2020

I have a different proposal:
grafik

That is why I put the Information in the window title so it shows up in the taskbar.

This you will not get with my implementation. Do you often have the situation that the client window is not shown but you have set yourself to muted? I think this is a special case. Most users will have the Jamulus client window up-front when they jam and when they use the Mute Myself button.

I think the red bar clearly shows the state. And we have the possibility to add some descriptive text (as I do in my example) to make it clearer what Mute Myself does (I am referring to again to #187). @gilgongo I think that red info bar with the additional text would solve the confusion, what do you think?

@gilgongo
Copy link
Member

gilgongo commented Aug 9, 2020

Yes, seems pretty clear. I'd suggest, "MUTED (You are not sending any audio to the server)"

BTW does this message override the server recording one? It seems to be shown in the same place.

@corrados
Copy link
Contributor

corrados commented Aug 9, 2020

No, but we could move the information about the recording also in that label.

@pljones
Copy link
Collaborator

pljones commented Aug 9, 2020

Well, recording is what the server is doing to you, whereas muted is what you're doing to yourself, so it seems sensible to keep the recording info in the server group box border to me.

@corrados
Copy link
Contributor

corrados commented Aug 9, 2020

Agreed.

@corrados
Copy link
Contributor

corrados commented Aug 9, 2020

I have just added the code of the new implementation to the Git repo.

@dingodoppelt I did not get any feedback from you, yet. I think with the implemented solution we have improved the situation at least for the Jamulus users which have the main window visible (which, I guess, are the most of them). For your special case, you already have implemented the support for showing the muted state in the Window title so I guess you can live with using your special version of Jamulus.

So I will close this Issue and also your pull request now. If you disagree, please write your opinion in this Issue and/or re-open the pull-request/Issue again if you think that is necessary.

@corrados corrados closed this as completed Aug 9, 2020
@Hk1020
Copy link
Contributor

Hk1020 commented Aug 10, 2020

Oops, I am too late, sorry. I’m not using Jamulus anymore, we went back to physical rehearsals. Anyway, why not say in the red text You aren’t heard by other participants or something like that. Who cares about the technology, say what’s the effect. Otherwise I like your change.

@corrados
Copy link
Contributor

Good idea. What about "The others will not hear me"?
@gilgongo What's your opinion?

@gilgongo
Copy link
Member

gilgongo commented Aug 10, 2020

Yes that's perhaps even better because it's the practical effect of muting yourself rather than the technical function of it. I guess there will always be some some possible confusion that then relates to the level meters still moving, but I guess we can't have everything :-)

@corrados
Copy link
Contributor

@gilgongo You are the native speaker. Would be "The others cannot hear me" better?
Or maybe "The others cannot hear you"?
But since we have "Mute Myself", maybe "me" is better than "you".

@jerogee
Copy link
Contributor

jerogee commented Aug 10, 2020

@corrados, @gilgongo, I really support to describe the effect as @Hk1020 suggested. Both variants are possible and fine imho. I will just go with "Others cannot hear you" in Dutch as this seems more common (in English and Dutch at least).

@Hk1020
Copy link
Contributor

Hk1020 commented Aug 10, 2020

One other wording that stuck in my mind: Other people won’t hear you.

@gilgongo
Copy link
Member

"You" vs "Me" in this context is really just stylistic and usually depends on the style used in other messages in the app. "Me" is sometimes said to be possibly unclear if the reader thinks the app (Jamulus) is referring to itself, so for this message perhaps "you" is clearer even though we have "mute myself"?

@corrados
Copy link
Contributor

Thanks, so let's use "you".
Last question (to the native speakers): Should we use:
"Other people won’t hear you"
or
"Other people don’t hear you"
?

@storeilly
Copy link

storeilly commented Aug 11, 2020 via email

@corrados
Copy link
Contributor

Ok, thanks. I'll change it now.

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

Successfully merging a pull request may close this issue.

7 participants