[pulseaudio-discuss] [PATCH pavucontrol] mainwindow: Show the availability of the ports and profiles.

Tanu Kaskinen tanuk at iki.fi
Fri Nov 9 11:26:59 PST 2012


On Fri, 2012-11-09 at 19:29 +0100, David Henningsson wrote:
> On 11/09/2012 07:17 PM, Tanu Kaskinen wrote:
> > On Thu, 2012-11-08 at 23:38 +0100, poljar (Damir Jelić) wrote:
> >> +        if (hasNo && !hasYes && !hasOther)
> >> +            desc += " (unplugged)";
> >> +        else if (hasYes && !hasNo && !hasOther)
> >> +            desc += " (plugged in)";
> >
> > I think profiles should be "plugged in" if any port is available.
> 
> The reason I did it for profiles in the first place, were just to make 
> the user get a hint about which one of the four HDMIs to select on the 
> configuration tab, and there is just one port per profile in that scenario.
> 
> I also think that if one port is available and another is not, we 
> shouldn't say that the entire profile is "plugged in". It really only 
> makes sense to write something when all of the ports have the same 
> value. I e, the current code is correct IMO.

What about laptop speakers that are always available (let's ignore the
fact that it looks like the analog-output-speaker port never reports
itself as "available", only "unknown" or "unavailable")? If headphones
are not plugged in, I think the output:analog-stereo profile is still
very much available ("plugged in" is not the best phrase in this case).

I think the distinction between "all ports available" and "some ports
available" is confusing to the user, and it would be better to drop that
distinction altogether. I would actually prefer not using any status
specifier for profiles, except if all ports are unavailable, in which
case I think "unplugged" is fine to signal to the user that this profile
is pretty useless.

-- 
Tanu



More information about the pulseaudio-discuss mailing list