<br><br><div class="gmail_quote">On Mon, Mar 23, 2009 at 10:28 PM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org">faure@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">
<br>
</div>Ah OK. But exec() is called on that menu, right? It feels weird (and fragile) to<br>
delete the menu while it's in exec().<br>
<br>
Would hide() work too? From the qmenu code, it seems to end the exec() eventloop,<br>
which is all you need AFAICS.<br><div class="im">
<br>
</div>I see. From just reading the patch I hadn't realized that the added code that calls<br>
startPopupDelay() ends up creating m_dirsMenu and calling exec on it (correct me<br>
if I'm wrong).<br>
<div><div></div><div class="h5">
</div></div></blockquote></div><br>Don't know if hide() should work, But I know it will need more code. Because I just call the startPopupDelay() in dragmove event. thus same code runs for clicking event and dragging event. <br>
If I use hide() Then I will need to use show() for the hiden menu manage this menu seperate from startPopupDelay() aproach. Ofcourse I'm open to suggestions here :)<br clear="all"><br>-- <br>Rahman Duran<br><br>Software Engineer<br>
Turkey<br><br>How many apples fell on Newton's head before he took the hint! <br><br>