<html><head></head><body>Can't we set the Qt UI on top only when the Qt vout window gets the flag set via set_state callback? I don't understand why you need to elaborate on top of the existing hack instead of just removing it.<br><br><div class="gmail_quote">Le 2 avril 2019 13:45:52 GMT+03:00, Steve Lhomme <robux4@ycbcr.xyz> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">On 4/2/2019 10:40 AM, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">I don't really see how you can enforce embedded video on Windows or <br>any Qt-based platform. You still have the headless (e.g. HTTP <br>interface, VLM, RC), Skin engine and multiple track scenarii that <br>cannot use the embedded Qt window provider.<br></blockquote><br>I agree. The proper vout_window_t for "non-embedded" for Windows is <br>coming very soon (probably today).<br><br>The problem is that if you start VLC with --no-embedded-video and <br>--video-on-top you might end up having the Qt interface on top of the <br>video and that should not be the case (I suppose the same issue exists <br>on other OSes already).<br><br>><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> But that's beside the point: there are no reasons so far to make a <br> hack when a proper interface already exists.<br><br> Le 2 avril 2019 09:19:57 GMT+03:00, Steve Lhomme <robux4@ycbcr.xyz> a <br> écrit :<br><br> On 4/1/2019 5:26 PM, Rémi Denis-Courmont wrote:<br><br> Le maanantaina 1. huhtikuuta 2019, 13.41.22 EEST Steve Lhomme<br> a écrit :<br><br> The doc of video-on-top is:<br> Alwaysplacethevideowindowontopofotherwindows. That sounds<br> like a feature of the standalone video window rather than<br> the UI. It corresponds to the VOUT_WINDOW_STATE_ABOVE<br> state that is already handled by many of them. The UI may<br> use the flag if a standalone window is not used. But if<br> there is one, IMO it shouldn't. <br><br> Yes, that seems to match the documentation. Still, I think Qt<br> should allow the window state, rather than second guess from<br> the configuration leading to unnecessary error corner cases. <br><br><br> It still does after this patch but only if we know we won't use an<br> external window (which is coming for Windows).<hr> vlc-devel mailing list<br> To unsubscribe or modify your subscription options:<br> <a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br><br><br> -- <br> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez <br> excuser ma brièveté.<hr> vlc-devel mailing list<br> To unsubscribe or modify your subscription options:<br> <a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>