[pve-devel] [PATCH manager 0/2 v3] Add GUI for audio device

Aaron Lauterer a.lauterer at proxmox.com
Mon Jul 22 18:08:39 CEST 2019



On 7/22/19 10:06 AM, Thomas Lamprecht wrote:
> On 7/19/19 3:32 PM, Aaron Lauterer wrote:
>> This series adds the GUI part for the audio device.
>>
>> It's a partial v3 to [0] (the GUI part).
>>
>> The option for the audio backend is fixed to `spice` in accordance with
>> the latest patch to qemu-server [1].
>>
>> [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-July/038273.html
>> [1]: https://pve.proxmox.com/pipermail/pve-devel/2019-July/038333.html
> 
> from a quick look: looks OK.
> 
> Single things I'm not yet fully sure (sorry) is UX, i.e., would it make sense
> to have this in the Display field when selecting spice? I mean:
> 
> Pros
>   + Makes only sense to have if one has spice
>   + Most people using spice probably want this too, so shortcut to get to it
>     (if I'd even have it default to on/add)
> 
> Cons
>   - Could seem a bit strange to have audio in a display component
>   - if more audio stuff comes we'd need to split it up again, or have the
>     possibility to add spice audio two different ways (non-ideal)
> 

When thinking were to put this in the GUI we might want to think about 
possible other options for spice that we might implement like an option 
for better video streaming or folder sharing through the client.

But that depends on how far in the future we want to plan this :)

> @Tim, @Domninik: what do you think?
> 




More information about the pve-devel mailing list