1807626 - Consider an outline speaker/sound icon

Status: RESOLVED FIXED

(Firefox :: Theme, enhancement)

Description

[Karl Tomlinson (:karlt)]

The icon toolkit/themes/shared/media/audio.svg, which looks like a speaker with sound radiating, is used in video controls and to indicate that a tab is playing audio. e.g. https://philips.s3.eu-central-1.amazonaws.com/Rudolf/VfE_html5.mp4.
With pref "media.setsinkid.enabled" set to true, the same icon is also used in prompts for selectAudioOutput(), in the same way as images of a microphone or camera are used in prompts for getUserMedia(). e.g. https://bugzilla.mozilla.org/attachment.cgi?id=9303828

The microphone and camera icons, browser/themes/shared/notification-icons/microphone.svg and browser/themes/shared/notification-icons/camera.svg, are rendered as outlines, but the speaker in audio.svg is filled.
For consistency, perhaps the speaker also should be rendered as an outline, or perhaps a distinct outline speaker icon should be used with selectAudioOutput() prompts.

Comment 1

[Karl Tomlinson (:karlt)]

Attachment: audio.svg in about:preferences#privacy

Comment 2

[Karl Tomlinson (:karlt)]

As shown in the attachment, the inconsistency will be more apparent if/when speaker selection permissions are added to about:preferences#privacy.

Comment 3

[Karl Tomlinson (:karlt)]

The microphone icon has been outline style since the proton refresh.

Comment 4

[Karl Tomlinson (:karlt)]

Attachment: Bug 1807626 use an outline icon for speaker selection permissions r?pbz

The icon was designed by Crystal Wong <[e-mail]>.

Comment 5

[Karl Tomlinson (:karlt)]

Attachment: with outline speaker icon

Comment 6

[Sam Foster [:sfoster] (he/him)]

Looks good. We might want to rename the existing audio.svg to speaker-fill.svg to match our naming conventions and clarify the relationship between the 2 icon variations. I've filed bug 1841331.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/712427656ebd
use an outline icon for speaker selection permissions r=pbz,desktop-theme-reviewers,dao

Comment 8

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/712427656ebd