<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 22, 2016 at 2:28 AM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Le 2016-03-19 19:58, Darrell Walisser a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The bug that I am attempting to fix is the one where if you make the<br>
window fullscreen, when it returns to normal its size is now the size<br>
of the screen and not the original size.<br>
</blockquote>
<br></span>
Picking the correct size when going from fullscreen to normal window mode is the job of the window manager. Don't force a size; problem solved.<br>
<br></blockquote><div>This is precisely what the patch does - it does not allow our code to change the window size when switching to/from fullscreen, it is qt's job to manage that.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As I already wrote, the UI should not even care whether it's fullscreen or normal mode (a normal window of the size of the screen should behave the same as a fullscreen window).</blockquote><div><br></div><div>Are you suggesting that every call to check window state (isFullscreen(), isMaximized(), ...) be removed, since the UI should not care about it?</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
-- <br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><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><br>
</div></div></blockquote></div><br></div></div>