[PATCH] Bug 156495: Drag and drop breadcrumb popup
rahman.duran at gmail.com
Mon Mar 23 20:42:42 GMT 2009
On Mon, Mar 23, 2009 at 10:28 PM, David Faure <faure at kde.org> wrote:
> Ah OK. But exec() is called on that menu, right? It feels weird (and
> fragile) to
> delete the menu while it's in exec().
> Would hide() work too? From the qmenu code, it seems to end the exec()
> which is all you need AFAICS.
> I see. From just reading the patch I hadn't realized that the added code
> that calls
> startPopupDelay() ends up creating m_dirsMenu and calling exec on it
> (correct me
> if I'm wrong).
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.
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 :)
How many apples fell on Newton's head before he took the hint!
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the kde-core-devel