Bug 294626 - kmix icon doesnt change when sound is muted
Summary: kmix icon doesnt change when sound is muted
Status: RESOLVED FIXED
Alias: None
Product: kmix
Classification: Applications
Component: KMix Panel Docking (show other bugs)
Version: 4
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Christian Esken
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-22 17:30 UTC by
Modified: 2012-02-28 00:02 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description 2012-02-22 17:30:22 UTC
Version:           4 (using Devel) 
OS:                Linux


The kmix dock icon when clicking and selecting to mute, the icon doesnt change to show that the sound is muted, the icon continues the same as if the sound was on.

Reproducible: Always

Steps to Reproduce:
Select kmix icon from panel and click to mute sound.

Actual Results:  
The icon doesnt change when clicking to mute sound.

Expected Results:  
The icon should change when muting sound.

OS: Linux (x86_64) release 3.2.2-1.1-desktop
Compiler: gcc
Comment 1 Colin Guthrie 2012-02-22 20:41:21 UTC
Can't reproduce here - it works fine and the icon properly changes.

I suspect it's because you've changed your master to a playback stream as per # 294625 but then muted the sink, and not the stream and thus the icon is following what the stream mute status is and not the sink. In other words it's behaving as expected.
Comment 2 2012-02-23 03:36:45 UTC
I have created a new user account, and the bug continues existing.
Are you a on recent master?
I also "svn co " kdemultimedia today.
Comment 3 2012-02-23 03:51:45 UTC
Just to clarify, the icon im referinto, is when you click in the kmix in the plasma panel, the icon from that small window in wich apepars the master channel.

If you call the Mixer window, the icons there do change if you click to mute.
Comment 4 Colin Guthrie 2012-02-23 09:05:22 UTC
No using 4.8 here.

Can you reproduce with PA support disabled? KMIX_PULSAUDIO_DISABLE=1 kmix --nofork?

I doubt anything specific to the PA backend is to blame here as that icon handling and mute status stuff is abstracted.
Comment 5 2012-02-23 16:51:42 UTC
So you need to first use master to be able to reproduce and be able ti fix it.
And running without pulseaudio backend and doesnt change the icon when mutting, but i didnt said tis was caused by using pulseaudio backend.
Comment 6 Colin Guthrie 2012-02-23 16:56:50 UTC
(In reply to comment #5)
> but i didnt said tis was caused by using pulseaudio backend.

This bug is reported against the "Component: Backend: PulseAudio", so you kinda did say it is.

As you've confirmed it's nothing to do with this componenet, I'll re-assign it as it won't be me looking into the issue. 

All the best.
Comment 7 2012-02-23 17:25:25 UTC
Yes i reported like that because i was using pulseaudio backend :)
Comment 8 Christian Esken 2012-02-27 22:59:01 UTC
Yes. I confirm  that it is broken, as the code is currently undergoing cleanups and maintenance.
Comment 9 Christian Esken 2012-02-27 23:14:26 UTC
SVN commit 1282604 by esken:

Fix mute state in DockIcon

 M  +3 -3      kmixdockwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1282604
Comment 10 Christian Esken 2012-02-28 00:02:13 UTC
SVN commit 1282607 by esken:

Fix slider in Tray Popup to follow volume changes again (trunk)
Fix mute state for WheelEvent in DockIcon
Related: bug 290863

 M  +2 -1      kmixdockwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1282607