[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: [Issue N23227] Menu key doesn't work
From:       qt-bugs () trolltech ! com
Date:       2003-05-28 13:27:44
[Download RAW message or body]

On Wednesday, 28. Mai 2003 00:06 André Wöbbeking wrote:
> On Saturday 24 May 2003 16:48, André Wöbbeking wrote:
>
> > I'm using KDE and Qt from CVS HEAD. If I press the Menu key (i.e in
> > konqueror) no context menu is opened. Does anyone know why?
>
> Ok, Trolltech changed the inline ctor of QContextMenuEvent which is
> used in
> qapplication_x11.cpp:
>
> @@ -400,7 +400,7 @@ class Q_EXPORT QContextMenuEvent : publi
>  public:
>      enum Reason { Mouse, Keyboard, Other };
>      QContextMenuEvent( Reason reason, const QPoint &pos, const QPoint
>      &globalPos, int state )
> -       : QEvent( ContextMenu ), p( pos ), gp( globalPos ), accpt(
> FALSE ), consum(FALSE),
> +       : QEvent( ContextMenu ), p( pos ), gp( globalPos ), accpt(
> TRUE ), consum( TRUE ),
>         reas( reason ), s((ushort)state) {}
>      QContextMenuEvent( Reason reason, const QPoint &pos, int state );
>
> the result is that the menu key is not propagated anymore and
> KListView
> can't emit the signal contextMenu().
>
> BTW: In qapplication.cpp in line 2422
>
>   if ( res || cevent->isConsumed() || w->testWFlags(
>   WNoMousePropagation ) )
>
> the propagation of the event can be disabled by the window flag
> WNoMousePropagation. Is this correct for Key_Menu?
>
> @ KMail:
>
> You use the signal QListView::contextMenuRequested() so you ignore the
> KDE settings for the menu shortcut.
>
>
> Cheers,
> André

Hi André,

the bug was that the consumed flag is also still TRUE; this should be
FALSE in the constructor. I've fixed that for the next beta, thanks for
pointing it out!

Best Regards,
Volker

--
Volker Hilsheimer, Support Manager
Trolltech AS, Waldemar Thranes gate 98, NO-0175 Oslo, Norway

_______________________________________________
KMail Developers mailing list
kmail@mail.kde.org
http://mail.kde.org/mailman/listinfo/kmail
[prev in list] [next in list] [prev in thread] [next in thread]