Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#4228 closed defect (fixed)

Text selections should not be draggable via touch

Reported by: garycmartin Owned by: garnacho
Priority: Unspecified by Maintainer Milestone:
Component: Sugar Version: 0.97.x
Severity: Unspecified Keywords: 13.1.0
Cc: erikos, gonzalo Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

Testing 13.1.0 build 11

Text selections should not be draggable via touch, it is very easy to accidentally move a text selection accidentally due to their usually large hit area, especially multiline selections and when using touch selection handles.

Test Case:
1) Start the Write Activity
2) Type some text
3) Make a selection (any way, keyboard, mouse, or touch)
4) touch and drag the selection with the touch screen

Result:
Text is dragged by touch screen interaction

Expectations:
No text drag should trigger, if the text view is scrollable I'd expect the interaction to scroll the view.

Attachments (1)

abiword-touch-scrolling.diff (5.8 KB) - added by garnacho 8 years ago.
very rough patch

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by garnacho

very rough patch

comment:1 Changed 8 years ago by garnacho

I've attached a quite crude patch that implements a touch interaction pattern similar to GtkScrolledWindow's, letting button events go through:

  • if barely no motion happened in between
  • after a timeout
  • on a second tap around the same area

It's worth observing that *only* scrolling is implemented, there's no kinetic deceleration animation, nor overshooting. Text selection hasn't been disabled either, but just happens on the same terms than GtkTextViews under GtkScrolledWindows

comment:2 Changed 8 years ago by godiard

  • Resolution set to fixed
  • Status changed from new to closed

Included in olpc6, this issue is solved.

comment:3 Changed 8 years ago by dnarvaez

  • Milestone 0.98 deleted

Milestone 0.98 deleted

Note: See TracTickets for help on using tickets.