Peter Kümmel wrote:
> Abdelrazak Younes wrote:
>> Richard Heck wrote:
>>> Abdelrazak Younes wrote:
>>>> Martin Vermeer wrote:
>>>>> I seem to recall that there was some code in the Qt X event handling
>>>>> code, precisely to handle this problem. Lars wrote this. But then it
>>>>> was decided that it could be removed... anyone remember? Something
>>>>> with coalescing of multiple keystroke events.
>>>> Yes there is a patch in bugzilla for _key_ events but this is
>>>> independent issue: Helge's problem is about the scrollbar.
>>>>
>>>> About Lars patch I hoped that someone on Linux (Richard?) would
>>>> continue with this patch...
>>> Point me in the right direction. I have a vague memory of this....
>> http://bugzilla.lyx.org/show_bug.cgi?id=3320
>>
>> There's a patch mostly ready for the key release problem. The only
>> problem with the patch is that it uses specific X11 event; it would be
>> better to do something similar with Qt only API. But this is not a big
>> problem because the bug is only visible on certain version of X11
>> apparently, not for Windows or Mac.
>>
>> Abdel.
>>
>
> I've found a simpler solution. See attached patch.
>
> It now stops immediately when the page up/down key is released
> the same for the mouse button.
> (seems the mouse button event is mapped to the page up/down key.)
> Tested under Linux and Windows.
>
> Peter
>
>
> waiting for a ok ...
but I forgot that it would also eat other key events. So when you
type text faster than the GUI could update, what should be possible
with ten fingers, some will be lost. We must check for the
page up/down key. I update the patch.
>
>
> ------------------------------------------------------------------------
>
> Index: qt4/GuiWorkArea.cpp
> ===================================================================
> --- qt4/GuiWorkArea.cpp (Revision 18371)
> +++ qt4/GuiWorkArea.cpp (Arbeitskopie)
> @@ -414,6 +414,15 @@
>
> void GuiWorkArea::keyPressEvent(QKeyEvent * e)
> {
> + // do nothing if there are other events
> + // (the auto repeated events come too fast)
> + if(QCoreApplication::hasPendingEvents()) {
> + LYXERR(Debug::KEY) << BOOST_CURRENT_FUNCTION
> + << endl << "key ignored" << endl;
> + e->ignore();
> + return;
> + }
> +
> LYXERR(Debug::KEY) << BOOST_CURRENT_FUNCTION
> << " count=" << e->count()
> << " text=" << fromqstr(e->text())
--
Peter Kümmel
