Closed n-peugnet closed 2 years ago
So, what do you think about this @christophgysin ?
What happens if you decrease the volume all the way down? In that case you would need the minimum of the two channels, otherwise you will never reach zero?
It seems there is no good solution to this. The icon can not correctly show two different volumes.
Here is what happens with my patch (max):
And here is what happens with the previous code (avg):
Maybe the notification with the sound volume will better show what I mean. Here is the icon with my patch (max):
And here is the icon with the previous code (avg):
See how the icon shows a low volume even if we reached the maximum allowed by pasystray
Thanks, this looks good :+1:
Following up #141, given that pasystray will stop increasing the volume once one of the channels arrived to max, it seems logical to index the volume icon on the maximum value of all channels.
This might be a controversial change and I am open to discussion, but IMO this is usually what people expect from this icon.
Indeed, with avg, if one of the channels is set a lot higher than the other ones, the icon could show a low volume while the maximum is reached. When using channels volume as a balance setting for instance this is misguiding as one could think there is some volume left to increase when in fact it is set to the maximum.