[EMAIL PROTECTED] wrote:

> Author: baum
> Date: Wed Jan 31 17:07:37 2007
> New Revision: 16981
> 
> URL: http://www.lyx.org/trac/changeset/16981
> Log:
> Fix bug 3107
> 
> Modified:
>     lyx-devel/trunk/src/frontends/qt4/GuiApplication.C
> 
> Modified: lyx-devel/trunk/src/frontends/qt4/GuiApplication.C
> URL:
>
http://www.lyx.org/trac/file/lyx-devel/trunk/src/frontends/qt4/GuiApplication.C?rev=16981
>
==============================================================================
> --- lyx-devel/trunk/src/frontends/qt4/GuiApplication.C (original) +++
> lyx-devel/trunk/src/frontends/qt4/GuiApplication.C Wed Jan 31 17:07:37
> 2007 @@ -212,7 +212,7 @@
>  return true;
>  }
>  default:
> -             return false;
> +             return QApplication::event(e);
>  }
>  }

I knew that this was no qt bug. Abdel, if there was any reason for returning
false here please refine this fix, but it is clear that we need to call
QApplication::event for all events we do not handel ourselves.

In general, as I also wrote on bugzilla: Please don't waist time to discuss
bug severity, rather try to fix the bug. Fixing this did cost me 15 minutes
of reading the QAction and QToolTip docs and debugging.


Georg

PS: I would not be surprised if more bugs like this where something went
wrong during the qt3->qt4 port are discovered.

PS2: Bennet and Jean-Marc, I also discovered something that might be useful
for OS X: http://doc.trolltech.com/4.2/qaction.html#MenuRole-enum

Reply via email to