[vlc-devel] [PATCH 1/4] libvlc: Add "fullscreen-monitor" setting

[Alexandre Janniaux]

On Wed, Oct 16, 2019 at 10:11:33PM +0300, Rémi Denis-Courmont wrote:
> Le keskiviikkona 16. lokakuuta 2019, 21.46.26 EEST Alexandre Janniaux a écrit
> :
> It's not a UX patch. This specific patch out of the series is touching the
> core, in a way that contradicts the existing Wayland code (XDG-shell), the
> reference model (i.e. audio output device) and the agreed upon design of the
> affected feature.

It adds a parameter that the user can directly provide, I
hardly see how this can be closer to UX.

>
> > My arguments are on the previous-previous post, but the TL;DR is: how the
> > interface can determine what configuration variable to set in the general
> > way ?
>
> Just like audio output devices. Also did I mention it's just like audio output
> devices?

I brought examples on why it wasn't compatible, because you
don't select the fullscreen device from a menu during the
playback and the underlying audio multiplexer already has
the correct preference most of the time, while the video
compositor usually doesn't and you set the setting for the
most of the time usage. You didn't bring any argument
against this, not even an UX use case which would improve it
to work your way.

If your way is not compatible with a correct UX, I don't
think that would be the way I'd like to follow.

Regards,
--
Alexandre Janniaux
Videolabs