Bug 356030

Summary: 15.11.80: track speaker icon coloring is ambiguous and inconsistent with other track icons
Product: [Applications] kdenlive Reporter: Wegwerf <wegwerf-1-2-3>
Component: User Interface & MiscellaneousAssignee: Jean-Baptiste Mardelle <jb>
Status: RESOLVED FIXED    
Severity: normal CC: null, wegwerf-1-2-3
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In: 15.12.1
Sentry Crash Report:
Attachments: dk_audio-volume-muted.svgz
dk_kdenlive-hide-audio.svgz
lt_audio-volume-muted.svgz
lt_kdenlive-hide-audio.svgz

Description Wegwerf 2015-11-28 16:05:10 UTC
Each track currently can show up to four track icons:
1. lock (locked/unlocked)
2. speaker (muted/unmuted)
3. film (video enabled/disabled)
4. "box" (composite/opaque)

While track icons 1, 3, and 4 are b/w only, the speaker icon instead changes its color. To me, the color red doesn't convey a clear meaning: does this mean that this track has audio? Surely that's not the intended meaning, but that is what I was thinking when I first saw a red speaker icon.

A consistent scheme would be to cross out the speaker icon #2 in the same way as it is done with the film strip icon #3. In the same way the color change should be removed, or alternatively, all icons #1, #2, and #3 should conform to the same color change indication.

Reproducible: Always

Steps to Reproduce:
1. Create new Kdenlive project.
2. Click on the speaker icon on the topmost track.
3. Click on the film strip icon on the topmost track.

Actual Results:  
Speaker icon becomes red but still shows a speaker emitting sound. The film strip becomes crossed out.

Expected Results:  
Speaker icon becomes crossed out as does the film strip icon.
Comment 1 Jean-Baptiste Mardelle 2015-12-08 08:48:38 UTC
I totally agree that the current state is incoherent. 
We need some new icons there, that behave consistently
Comment 2 Wegwerf 2015-12-08 17:41:45 UTC
I can make a suitable icon for "disabled audio" following the same meme as for "disabled video"; both for breeze light & dark.
Comment 3 Unknown 2015-12-09 16:44:27 UTC
Good eye, Wegwerf. I agree.
Comment 4 Wegwerf 2015-12-09 21:09:14 UTC
Sent new disable audio icons for dark/light breeze theme to the developers.
Comment 5 Wegwerf 2015-12-09 21:11:14 UTC
Created attachment 95959 [details]
dk_audio-volume-muted.svgz
Comment 6 Wegwerf 2015-12-09 21:11:39 UTC
Created attachment 95960 [details]
dk_kdenlive-hide-audio.svgz
Comment 7 Wegwerf 2015-12-09 21:12:31 UTC
Created attachment 95961 [details]
lt_audio-volume-muted.svgz
Comment 8 Wegwerf 2015-12-09 21:15:04 UTC
Created attachment 95962 [details]
lt_kdenlive-hide-audio.svgz
Comment 9 Wegwerf 2015-12-19 20:25:00 UTC
Fixed in 15.12.0. Many thanks!