Review Request 127492: KCM: Add possibility to move stream to other devices

David Rosca nowrep at gmail.com
Fri Apr 1 06:31:06 UTC 2016



> On March 25, 2016, 7:24 p.m., Thomas Pfeiffer wrote:
> > Two questions:
> > 1. If there is only one device, are the comoboxes hidden (they should be)?
> > 2. If I change the default device, are all streams automatically switched to that device?
> 
> David Rosca wrote:
>     1. Yes
>     2. No, it just sets the pulseaudio default sink/source. It is then used as fallback device (when stream's device disappears, it will move to default device) and probably also for new streams.
>     
>     I also plan to make the global shortcut to change/mute volume and osd to only work with default device.
> 
> Thomas Pfeiffer wrote:
>     1. Good
>     2. A bit of a problem. Or rather, it means we have to re-think some things. The problem is that currently we make the more common scenario (just switching all audio to a different device) more complicated to do than the more advanced scenario (moving individual streams).
>     
>     I know that in theory, PulseAudio should be smart enough to just switch all audio to a newly plugged-in or switched-on device, but from my own experience I can say that this automatism doesn't always work.
>     So for example if I have my TV connected to HDMI, I want all audio to go through my speakers when the TV is off. When I switch the TV on, though, it's more likely that I just want all audio to go through it instead of having to switch individual streams.
>     Switching individual streams is a nice feature, but only useful for rather advanced setups. So we should keep it, but we need a way to move all output to a different device easier.
> 
> David Rosca wrote:
>     Sure, rerouting all streams to one device can be added. Any idea where to expose it in ui?
> 
> Xuetian Weng wrote:
>     I also wonder, is it possible to route the sound to multiple device at the same time? If so maybe combobox is not enough.
>     
>     Also I feel this feature is not a commonly used feature. I wonder if it is possible to hide it until user click on it, just like when you click on connection in plasma-nm, you'll see connection details in plasma-nm.
> 
> Kai Uwe Broulik wrote:
>     > I also wonder, is it possible to route the sound to multiple device at the same time? If so maybe combobox is not enough.
>     
>     Oh, I would love that :) We could use a Button with an attached Menu with checkable items instead, UI-wise should be least of our problems I guess.

One stream can be only on one device, so the combobox with one choice is correct. If you want to play one stream on multiple devices, you need to configure virtual devices afaik.

> I wonder if it is possible to hide it until user click on it, just like when you click on connection in plasma-nm, you'll see connection details in plasma-nm.

Maybe later, I plan to add more features so it will probably be needed anyway.


- David


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


On March 25, 2016, 4:48 p.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127492/
> -----------------------------------------------------------
> 
> (Updated March 25, 2016, 4:48 p.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-pa
> 
> 
> Description
> -------
> 
> Add combobox next to mute button with available devices.
> 
> 
> Diffs
> -----
> 
>   src/kcm/package/contents/ui/DeviceComboBox.qml PRE-CREATION 
>   src/kcm/package/contents/ui/SinkInputView.qml PRE-CREATION 
>   src/kcm/package/contents/ui/SourceOutputView.qml PRE-CREATION 
>   src/kcm/package/contents/ui/StreamListItem.qml 7f1d69e 
>   src/kcm/package/contents/ui/StreamView.qml b08c87a 
>   src/kcm/package/contents/ui/main.qml 3360201 
> 
> Diff: https://git.reviewboard.kde.org/r/127492/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> Screenshot_20160325_142736.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/03/25/06b2aafd-f05d-4140-933a-89fbc8ed7c5c__Screenshot_20160325_142736.png
> 
> 
> Thanks,
> 
> David Rosca
> 
>

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


More information about the Plasma-devel mailing list