No problem! Thanks for the great work you are doing with OSG.
Hope to submit more improvements in the future!

Cheers!
Juan M.


On Thu, Jun 20, 2013 at 7:13 AM, Robert Osfield <robert.osfi...@gmail.com>wrote:

> Hi Juan,
>
> Sorry for taking a while to reivew your change.  I've been juggling
> various tasks over the last weeks and only able to merge submissions
> in a bit of adhoc way - you submission was in my inbox awaiting a time
> when I had clear my desk enough to get back to clearly my submissions
> backlog.
>
> I have just done the review, and also modified osgkeyboard (that will
> be using X11 on my Linux system) to report event types when I keep the
> key down and this confirmed the behaviour your changes add to osgQt.
> All looks OK so I've gone ahead and merged.  Changes now checked into
> svn/trunk so will be part of the 3.1.8 dev release.
>
> Cheers,
> Robert.
>
> On 19 June 2013 18:40, Juan Manuel Alvarez <naici...@gmail.com> wrote:
> > Hi! I am resubmitting my first submission. I sent it some days ago and it
> > did not receive any response. If it is something wrong with the
> submission,
> > please tell me so I can correct it. Please forgive me for the repost!
> >
> > So here is the submission again:
> >
> > When handling keyboard events on osgQt, if a key is kept pressed,
> multiple
> > sequences of KEYDOWN -> KEYUP events are fired.
> >
> > This is because Qt auto repeats keyboard events, so multiple calls are
> made
> > to GLWidget::keyPressEvent and GLWidget::keyReleaseEvent by Qt, and
> > subsequently translated to OSG events.
> >
> > The way to solve this is ignoring key released auto repeated events (see
> > http://qt-project.org/doc/qt-4.8/qkeyevent.html#isAutoRepeat), so
> multiple
> > KEYDOWN events are fired, but only one KEYUP.
> >
> > I attach a modified osgQt/GraphicsWindowQt.cpp with this change.
> >
> > Thanks!
> >
> > _______________________________________________
> > osg-submissions mailing list
> > osg-submissions@lists.openscenegraph.org
> >
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
> >
> _______________________________________________
> osg-submissions mailing list
> osg-submissions@lists.openscenegraph.org
>
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
>
_______________________________________________
osg-submissions mailing list
osg-submissions@lists.openscenegraph.org
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to