Opened 11 years ago
Last modified 10 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)
Change History (10)
comment:1 Changed 11 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
comment:2 Changed 11 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 11 years ago by garnacho
Updated patch, fixes extraneous behavior when the contained widget doesn't do touch events
comment:3 Changed 11 years ago by garnacho
This new patch fixes some issues, bear in mind that the stated in comment 2 still applies
comment:4 Changed 10 years ago by dnarvaez
- Component changed from sugar-toolkit-gtk3 to Sugar
comment:5 Changed 10 years ago by dnarvaez
- Milestone changed from 0.98 to Unspecified
comment:6 Changed 10 years ago by dnarvaez
- Bug Status changed from Assigned to Unconfirmed
comment:7 Changed 10 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 10 years ago by manuq
- Priority changed from High to Normal
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.