<div dir="ltr"><div dir="ltr"><div>My other patch uses that drop-down to populate this setting. But in the Advanced Preferences, yeah, it shows up as a plain text box. I guess you could argue that it is "advanced", but I wouldn't argue if you say that's bad UX. And I understand the macOS doesn't use the Qt interface, and I have no experience there, so I don't know if there's an equivalent drop-down box there already.</div><div><br></div><div>I've been following the discussion but not commenting since I know I don't know the code base very well. I agree that the enumerating the screens and setting/using the value is dependent on the window provider, but my rationale was that every window provider would be able to enumerate the screens and set and use the value. So no matter which OS you're on, it is possible to put a value in this setting.</div><div><br></div><div>So Rémi, the alternative is for each window provider to have its own setting? Was Thomas right when saying:</div><div><br></div><div>    have this option in win32/window.c</div><div><br></div><div>      vlc_module_begin()<br>        set_shortname(N_("Win32 window"))<br>        set_description(N_("Win32 window"))<br>        set_category(CAT_VIDEO)<br>        set_subcategory(SUBCAT_VIDEO_VOUT)<br>        set_capability("vout window", 10)<br>        set_callback(Open)<br><div>    vlc_module_end()</div></div></div><div><br></div><div>I'm happy to make that change. However, then I'm not sure how to use the setting. vout_ChangeFullscreen accepts an id parameter, which is supposed to be the "output identifier", which I assume means the ID of the monitor? (correct me if I'm wrong) So how do we find that value when calling 

vout_ChangeFullscreen if only the window provider knows? Or am I misunderstanding something there?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Oct 17, 2019 at 2:18 AM Steve Lhomme <<a href="mailto:robux4@ycbcr.xyz">robux4@ycbcr.xyz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2019-10-16 21:58, Rémi Denis-Courmont wrote:<br>
> Meanwhile, you did not explain how a core setting is going to work. You can't<br>
> seriously be suggesting that users type in the monitor identifier in the VLC<br>
> preferences... so much for being "compatible with a correct UX".<br>
<br>
There's already a setting listing the screen to use when going in <br>
fullscreen. You can't type it manually though.<br>
<br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div></div>