[vlc-devel] [PATCH] Send fullscreen video to specific monitor

[Steve Lhomme]

On 2019-10-03 21:03, Gabriel Luci wrote:
> Can you point me to that proper API? I certainly don't mind using a 
> different method to do it if it makes more sense.

I think Remi is talking about the vout_window_SetFullScreen() which can 
tell which output-identifier should be used. In other words that would 
be the name of the monitor.

Right now we always pass NULL all the time, via 
vout_window_SetFullScreen(). That's probably where the 
"qt-fullscreen-screennumber" should be passed, although the name 
suggests it's Qt specific. It seems that the "screen id" would rather be 
OS/window manager specific than relying on Qt. It seems the Qt option is 
already handled fully by Qt, setting in the fullscreen window in the 
right window. But it may be broken in the current UI.

> But the reason I went this route is because the SetFullscreen function 
> in /modules/video_output/win32/window.c already uses an OS-specific way 
> to select which monitor to put the video output window on 
> (MonitorFromWindow). So I assume that code must change if it's going to 
> put the window on a different monitor. It just needs to be told which 
> monitor somehow - I just don't know what that "somehow" would be besides 
> a new configuration setting.
> 
> 
> On Thu, Oct 3, 2019 at 2:39 PM Rémi Denis-Courmont <remi at remlab.net 
> <mailto:remi at remlab.net>> wrote:
> 
>     Hello,
> 
>     There is already a proper API defined for this, so clearly adding an
>     plugin-
>     specific OS-specific setting on top is not the way to go.
> 
>     -- 
>     雷米‧德尼-库尔蒙
>     http://www.remlab.net/
> 
> 
> 
>     _______________________________________________
>     vlc-devel mailing list
>     To unsubscribe or modify your subscription options:
>     https://mailman.videolan.org/listinfo/vlc-devel
> 
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
>