Comment 5 for bug 1239605

Revision history for this message
David Henningsson (diwic) wrote :

Thanks for the design update. I have a few questions/comments:

1) About "headphones/speakers" - I'd prefer if we left "speakers" out of it for now. The reason is that external powered speakers would assume a line level output, where as a headphone would require the headphone amplifier (integrated into the codec) to be activated instead. Putting external speakers on a headphone output works, but the sound quality is not optimal.
There is a use case where we would want to switch between headphones and speaker output to get the best sound quality, but I don't want to add support for that now. Ok to change "headphones/speakers" to "headphones"?

2) The dialog as designed includes some icons. Should we try to use the icons supplied by the current theme (audio-input-microphone, audio-headset, audio-headphones), or should I request somebody draw them for me?

3) I'm not sure exactly where you want me to add the "When an unknown audio device is plugged in" line. I can't find any place in the current design where we have a "Show sound volume in the menu bar either", btw.

4) The drawing shows a text "unknown devices in future". Should this be "in the future" (missing a "the")? I'm not a native English speaker, but the latter wording sounded more natural to me.