Project

General

Profile

Actions
Copy link

Bug #4905

closed
AT RS

When SetSeriesSelectionEnable, setZoomEnable, and setPanEnable was enabled altogether, performing zoom/pan gesture(touch) can accidentally end up selecting curve.

Bug #4905: When SetSeriesSelectionEnable, setZoomEnable, and setPanEnable was enabled altogether, performing zoom/pan gesture(touch) can accidentally end up selecting curve.

Added by Apivan Tuntakurn over 10 years ago. Updated about 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Roel Standaert
Target version:
3.3.6
Start date:
05/10/2016
Due date:
% Done:

0%

Estimated time:

Description

According to the comment in the SetSeriesSelectionEnable, the series selected should be triggered by long touch.
However, when I enable all SetSeriesSelectionEnable, setZoomEnable, and setPanEnable, doing zoom could also trigger the series selected.

Is it possible to improve these touch gestures classification?

Long touch: touch for a while and not move (SeriesSelection)
Zoom gesture: multi touch (Zoom)
Pan gesture: touch and move (Pan)

Or, perhaps it would be better to ask my client to redesign such use case.
I would like to hear your opinion.

I have attached the test project here.

Files

chart2_touch.tar (50 KB) chart2_touch.tar Apivan Tuntakurn, 05/10/2016 04:17 AM

KD Updated by Koen Deforche over 10 years ago Actions
Copy link
 #1

  • Status changed from New to InProgress
  • Assignee set to Roel Standaert

KD Updated by Koen Deforche over 10 years ago Actions
Copy link
 #2

  • Target version set to 3.3.6

RS Updated by Roel Standaert over 10 years ago Actions
Copy link
 #3

  • Tracker changed from Support to Bug
  • Status changed from InProgress to Implemented @Emweb

Yeah, that was a bug. The series selection wasn't properly canceled when transitioning from a single touch to multiple touches.

AT Updated by Apivan Tuntakurn over 10 years ago Actions
Copy link
 #4

Thank you for the quick response and help. I appreciate it.

KD Updated by Koen Deforche over 10 years ago Actions
Copy link
 #5

  • Status changed from Implemented @Emweb to Resolved

KD Updated by Koen Deforche about 10 years ago Actions
Copy link
 #6

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: PDF Atom