Bug 294626

Summary:	kmix icon doesnt change when sound is muted
Product:	[Applications] kmix	Reporter:	Zé <mmodem00>
Component:	KMix Panel Docking	Assignee:	Christian Esken <esken>
Status:	RESOLVED FIXED
Severity:	normal	CC:	esken
Priority:	NOR
Version:	4
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Zé 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 Zé 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 Zé 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 Zé 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 Zé 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