<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 7, 2016 at 8:30 AM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 28 Feb, Darrell Walisser wrote :<br>
<span class="">> +        /* The original widget width() and height() won't be realized<br>
> +         * immediately (maybe a bug in Qt). Using a timer won't<br>
> +         * work because it is unknown how long this could take.*/<br>
> +         while (fullScreenSize == size())<br>
<br>
</span>This is really too ugly.<br>
<br></blockquote><div><br></div><div>Yes, it's ugly, but it is working around a bug in Qt, and a more serious issue with the UI. There is the windowStateChanged() signal/event in Qt but I bet it has the same problem.</div><div><br></div><div>I did not describe the issue this is trying to address. Which is, if you have minimal mode disabled, and then go fullscreen, when you come out of fullscreen, the window size will be the size of the screen instead of what it started at. The reason is because setMinimalView used the wrong window size because it hasn't updated yet (due to qt bug).</div><div><br></div><div>The alternative is to add much more code, it seems. One idea, keep track of window size ourselves and maybe roll our own fullscreen code. Easier to remove the height adjustment patch to setMinimalView() since it is what introduced the problem.</div><div><br></div></div></div></div>