Do not disable the slider if it's muted. It would be more convinient if
it's possible to change muted item volume, just like other mixers. Also
unmute the device/stream if it's muted at the same time.
Unfortunately, I didn't find a way to pretend to be visually disabled,
thus here we use opacity to get the similar visual feedback.
test on my desktop
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| src/kcm/package/contents/ui/VolumeSlider.qml | ||
|---|---|---|
| 44–45 | It should also change opacity when muted here. | |
How is this presented to the user, visually? I mean when using shortcuts the pop-up (early morning the oms/cms/abbreviation-thing) that shows you sound is muted, or lowered/raised is on screen for just a flicker - which for most are enough since the information is so clear and if you raise/lower it becomes a string of images creating an animation.
Wont adding a third effect sort of make that too complex?
Then the slider: the issue with a slider thats not opaque is that it tend to grey out, a greyed out object is by all norms something you can't interact with.
Not saying that its wrong - just wondering if you guys would want suggestions for alternative ways to present it from us in the VDG?
| src/kcm/package/contents/ui/VolumeSlider.qml | ||
|---|---|---|
| 44–45 | I'd rather not, because this is widget style, and disable style is actually "grey out". | |
This is for sliders in ui (applet/kcm), not in OSD as you describe.
I'd rather not, because this is widget style, and disable style is actually "grey out".
The same is in applet, disabled slider is greyed out. And Jens has a point here, it would be better if we come up with a different way to visualize muted stream without changing opacity of sliders (if we want to make them interactive when muted).
Actually now I looked at the one in windows, probably opacity makes sense. Shall we do it this way (set opacity) for now, since this is not only about visual but also functionality
Shall we do it this way (set opacity) for now
I'd say let's go with it for now, unless VDG has better idea? In any case, this should go into master and not 5.7.