Thanks Anna, changes now merged and checked into OSG master. Cheers,
Robert.
On 7 August 2017 at 22:05, Anna Sokol wrote:
> Robert,
>
> In the attached GraphicsWindowWin32.cpp file, I changed order of the
> "else-if" for the TOUCHEVENTF_UP and TOUCHEVENTF_MOVE, I made it less
> intrusive then changing them into "if" events. I tested this on Windows 7
> 64-bit with a touch screen and a Windows 10 64-bit tablet.
>
> Separately in the same file, I also needed to address the fact that the
> close button would not react on touch so I added to the top of the
> "handleNativeWindowingEvent" close button handling in case of touch events.
> Again this was tested on the same 2 devices.
>
> Regards,
> Anna Sokol
>
>
>
> On Mon, Aug 7, 2017 at 6:42 AM, Robert Osfield
> wrote:
>
>> Hi Anna,
>>
>> I'm just doing a review of your changes and am curious how the order
>> change affects the behvaior. What happens before and after these changes?
>>
>> It seems like if the dwFlags can have TOUCHEVENTF_UP and TOUCHEVENTF_MOVE
>> at the same time changing the order will just ensure that move events get
>> handled correctly, but will mean that up would be missed.
>>
>> I'm now a Windows developer and haven't worked with touch under Windows
>> so can't provide much specific guidance, so I'm just trying to sort out why
>> the changes are required and how they might effect things down the line - I
>> don't want to have one usage case fixed but another broken by merging this
>> change.
>>
>> Thanks,
>> Robert.
>>
>>
>> On 27 June 2017 at 02:31, Anna Sokol wrote:
>>
>>> Hi Robert,
>>>
>>> I made a slight adjustment by keeping the "else if" but by switching the
>>> order of "move" and "up" touch events.
>>>
>>> Anna Sokol
>>>
>>>
>>>
>>> On Mon, Jun 26, 2017 at 7:01 PM, Anna Sokol wrote:
>>>
Hi Robert,
This is a fix for win32 touch events.
The issue is that a TOUCHEVENTTF_MOVE and a TOUCHEVENTTF_UP can occur
on the same input. In the original code there was an "else if" checking
first for "move" and then for "up". Since both can occur at the same time
the "up" was being ignored. In the attached GraphicsWindowWin32.cpp file
the "else if" is changed to an "if" check instead so that both the "move"
and "up" touch events are registered.
Anna Sokol
>>>
>>>
>>> ___
>>> 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
>
>
___
osg-submissions mailing list
osg-submissions@lists.openscenegraph.org
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org