Opened 7 years ago

Last modified 5 years ago

#4060 new defect

Bad behaviour of gestures inside a ScrolledWindow

Reported by: humitos Owned by: garnacho
Priority: Normal Milestone: 0.102.0
Component: Sugar Version: 0.97.x
Severity: Major Keywords: gtk
Cc: humitos, godiard Distribution/OS: OLPC
Bug Status: New

Description

Working on ImageViewer I found that touch gestures are not recognized by SugarGestures when a ScrolledWindow is showing its scrollbars.

If I try to make a gesture inside a ScrolledWindow, the window starts scrolling over the image instead of recognizing my gesture.

Attachments (2)

0001-scrolledwindow-Stop-scrolling-when-a-second-touch-po.patch (4.9 KB) - added by garnacho 7 years ago.
updated patch
0001-scrolledwindow-Stop-scrolling-when-a-second-touch-po.2.patch (5.4 KB) - added by garnacho 7 years ago.
Updated patch, fixes extraneous behavior when the contained widget doesn't do touch events

Download all attachments as: .zip

Change History (10)

comment:1 Changed 7 years ago by erikos

  • Bug Status changed from Unconfirmed to Assigned
  • Component changed from untriaged to sugar-toolkit-gtk3
  • Severity changed from Unspecified to Major
  • Version changed from Unspecified to 0.97.x

This was the initial patch that Carlos and myself came up with: http://dev.laptop.org/~erikos/gtk3_patches/deprecated/0005-Stop-scrolling-when-a-second-touch-point-is-detected.patch

It does not work yet, but that should be the approach.

Changed 7 years ago by garnacho

updated patch

comment:2 Changed 7 years ago by garnacho

This patch makes scrolledwindow concede touch handling to the contained child if a second touch is received, but if scrolling has started already, the captured button event will be outdated and at the wrong coordinates, so further touches while scrolling is ongoing are silently ignored.

Changed 7 years ago by garnacho

Updated patch, fixes extraneous behavior when the contained widget doesn't do touch events

comment:3 Changed 7 years ago by garnacho

This new patch fixes some issues, bear in mind that the stated in comment 2 still applies

comment:4 Changed 6 years ago by dnarvaez

  • Component changed from sugar-toolkit-gtk3 to Sugar

comment:5 Changed 6 years ago by dnarvaez

  • Milestone changed from 0.98 to Unspecified

comment:6 Changed 6 years ago by dnarvaez

  • Bug Status changed from Assigned to Unconfirmed

comment:7 Changed 6 years ago by quozl

  • Bug Status changed from Unconfirmed to New
  • Milestone changed from Unspecified to 0.102.0

(triage meeting: bug is verified, some work done)

comment:8 Changed 5 years ago by manuq

  • Priority changed from High to Normal
Note: See TracTickets for help on using tickets.