Bug 356030 - 15.11.80: track speaker icon coloring is ambiguous and inconsistent with other track icons
Summary: 15.11.80: track speaker icon coloring is ambiguous and inconsistent with othe...
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (other bugs)
Version First Reported In: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-28 16:05 UTC by Wegwerf
Modified: 2016-01-02 11:49 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 15.12.1
Sentry Crash Report:


Attachments
dk_audio-volume-muted.svgz (1.42 KB, image/svg+xml)
2015-12-09 21:11 UTC, Wegwerf
Details
dk_kdenlive-hide-audio.svgz (1.42 KB, image/svg+xml)
2015-12-09 21:11 UTC, Wegwerf
Details
lt_audio-volume-muted.svgz (1.17 KB, image/svg+xml)
2015-12-09 21:12 UTC, Wegwerf
Details
lt_kdenlive-hide-audio.svgz (1.17 KB, image/svg+xml)
2015-12-09 21:15 UTC, Wegwerf
Details

Note You need to log in before you can comment on or make changes to this bug.
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!