Review Request 127484: applet: Add streams tab to manage applications volume

[Thomas Pfeiffer]

> On March 24, 2016, 4:53 p.m., Thomas Pfeiffer wrote:
> > Great that you're bringing streams back to the applet, much appreciated!
> > 
> > I agree with Kai that we should hide the technical stream names, they are of no use to anybody not deeply interested in PulseAudio. If an application has more than one stream, it only makes sense to show anything besides the applicaiton name if it has any meaning for the user. Distinguishing multiple streams based on some purely technical identifier gives zero useful information. If we have no human-readable information to distinguish streams from the same application, we might as well fall back to simply numbering them.
> > 
> > As for the redirection of streams to devices: What I'd like to see from a user perspective is simply dragging and dropping. So users could grab a stream, drag it over to the devices tab and then drop it on a device. That would be kind of intuitive and wouldn't clutter the UI with dropdown arrows. I don't know if that is technically feasible, of course.
> 
> David Rosca wrote:
>     > I agree with Kai that we should hide the technical stream names, they are of no use to anybody not deeply interested in PulseAudio....
>     
>     Alright, but that's for another patch. Currently it only shows name that exposes model.
>     
>     > As for the redirection of streams to devices: What I'd like to see from a user perspective is simply dragging and dropping. So users could grab a stream, drag it over to the devices tab and then drop it on a device. That would be kind of intuitive and wouldn't clutter the UI with dropdown arrows. I don't know if that is technically feasible, of course.
>     
>     It is possible, but as the streams are now displayed in separate tab, I think it would be extra awkward. First you will need to change the tab by dragging the stream on the "Devices" tab and then you'll need to find the right device. As the systemtray popup is small, if you have more devices then you would need to also scroll the view.
>     It is also much faster with the combobox than dragging.
>     
>     In other applets (print, network, bluetooth, ..), we already have the pattern of showing additional options when clicking on item. So maybe remove the arrow and indicate the additional actions with hover highlight (as in network applet)?
>     
>     Oh and also currently there is issue with scrolling of the view with mouse wheel that will be interrupted by volume sliders (or even may accidentally change volume). So I'm thinking of completely disabling scrolling with mouse wheel (as there doesn't seem to be better fix while having the option to change volume with wheel on sliders), which would make the scrollbar and flick with left mouse the only way to scroll it. And that would make the dragging impossible.

> In other applets (print, network, bluetooth, ..), we already have the pattern of showing additional options when clicking on item. So maybe remove the arrow and indicate the additional actions with hover highlight (as in network applet)?

Yes, that makes sense!


- Thomas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127484/#review93937
-----------------------------------------------------------


On March 24, 2016, 2:28 p.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127484/
> -----------------------------------------------------------
> 
> (Updated March 24, 2016, 2:28 p.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-pa
> 
> 
> Description
> -------
> 
> Applet now have two tabs: Devices and Streams.
> If there are more audo devices, streams can be moved between them with combobox that is visible after clicking on stream item.
> 
> BUG: 352042
> BUG: 353232
> 
> 
> Diffs
> -----
> 
>   applet/contents/ui/ListItemBase.qml 2e31eeb 
>   applet/contents/ui/SinkInputListItem.qml 3867007 
>   applet/contents/ui/SourceOutputListItem.qml 3b684fc 
>   applet/contents/ui/StreamListItemBase.qml 7c0af12 
>   applet/contents/ui/main.qml bf6f683 
>   src/pulseaudio.h 9fc9656 
>   src/pulseaudio.cpp 1488784 
> 
> Diff: https://git.reviewboard.kde.org/r/127484/diff/
> 
> 
> Testing
> -------
> 
> Everything seems to work fine.
> 
> 
> File Attachments
> ----------------
> 
> applet-devices.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/03/24/d1f444cd-c69b-4888-a0c0-24bc4b6b8904__applet-devices.png
> applet-streams.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/03/24/30a3cdfb-b6b5-41c7-bb15-37c6b87604a4__applet-streams.png
> applet-streams-more-devices.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/03/24/82f0eb13-95a5-4d0c-b50b-682cf7c3b384__applet-streams-more-devices.png
> applet-streams-more-devices.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/03/24/b2dafc91-4bde-42dc-bfd5-036ee001b984__applet-streams-more-devices.png
> 
> 
> Thanks,
> 
> David Rosca
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160324/ae8b359d/attachment.html>