<div dir="ltr">Hi Fabian,<div><br></div><div>The patch I sent in actually fixes the issue, I would not send in an untested patch.</div><div><br></div><div>I have just done some more digging, and you are right yes. </div><div>If it wasn't working, then the systray would be showing, as 0x1 is less than whatever resolution you are using.</div><div><br></div><div>Just about to submit a v2 patch, which solves this and is actually smaller by just calling GetSystemMetrics with SM_CXSCREEN, SM_CYSCREEN, SM_CXFULLSCREEN & SM_CYFULLSCREEN.</div><div><br></div><div>Thanks in Advance,</div><div>Hamish</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, Dec 30, 2018 at 8:51 AM Fabian Maurer <<a href="mailto:dark.shadow4@web.de">dark.shadow4@web.de</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"><u></u>
<div style="font-family:"Noto Sans";font-size:9pt;font-weight:400;font-style:normal">
<p style="margin:0px;text-indent:0px">Hi Hamish,</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">SM_CXSCREEN and SM_CYSCREEN are constants, defined as 0 respective 1.</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">So basically your code is</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">>  if ( !( window.right >= 0 && window.bottom >= 1 ) )</p>
<p style="margin:0px;text-indent:0px">>          do_show_systray();</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">And this is not correct. Those values are just flags to use in the GetSystemMetrics function.</p>
<p style="margin:0px;text-indent:0px">Did you check that the patch you sent in actually fixes the issue? Or did you only test with a previous version of your patch? Because I don't see how this would help.</p>
<p style="margin:0px;text-indent:0px"> </p>
<p style="margin:0px;text-indent:0px">Regards,</p>
<p style="margin:0px;text-indent:0px">Fabian Maurer</p></div></blockquote></div>