Long press timeout 'feels' longer than 0.5 seconds

[erikos]

Compare:

• do a long press in the Home View on one of the activity icons

• do a long press in the Journal list view on an activity icon

For the latter you have to hold the finger down longer for the Palette to appear. The timeout in the code is the same.

Seen with 0.97.9

[humitos]

Trying to reproduce this bug I found this traceback:

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 1444, in __long_pressed_event_cb
    self.notify_right_click()
  File "/usr/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 967, in notify_right_click
    self._ensure_palette_exists()
  File "/usr/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 955, in _ensure_palette_exists
    palette = self.parent.create_palette()
  File "/usr/lib/python2.7/site-packages/jarabe/journal/listview.py", line 646, in create_palette
    metadata = tree_model.get_metadata(self.props.palette_invoker.path)
  File "/usr/lib/python2.7/site-packages/jarabe/journal/listmodel.py", line 103, in get_metadata
    return model.get(self[path][ListModel.COLUMN_UID])
  File "/usr/lib/python2.7/site-packages/gi/overrides/Gtk.py", line 782, in __getitem__
    aiter = self._getiter(key)
  File "/usr/lib/python2.7/site-packages/gi/overrides/Gtk.py", line 776, in _getiter
    aiter = self.get_iter(key)
  File "/usr/lib/python2.7/site-packages/gi/overrides/Gtk.py", line 798, in get_iter
    path = TreePath(path)
  File "/usr/lib/python2.7/site-packages/gi/overrides/Gtk.py", line 1209, in __new__
    if len(path) == 0:
TypeError: object of type 'NoneType' has no len()

I not sure why I'm getting this error or how to reproduce it. It happens mainly at startup and not even over the same Journal entry the second time.

• file: src/sugar3/graphics/palettewindow.py
  • CellRendererInvoker
    • Adding some log to the __long_pressed_event_cb I found it's called 7 times when you longpress over the icon and the third time it enters on the if (self.point_in_cell_renderer returns True)
  • CursorInvoker
    • Adding some log to the __long_pressed_event_cb I found it's called just once when you longpress over the icon

[humitos]

Replying to humitos:

Trying to reproduce this bug I found this traceback:

This traceback is because sometimes CellRendererInvoker.__long_pressed_event_cb is called without a previous call to CellRendererInvoker.__motion_notify_event_cb where the self.path is set to some value different from None.

This is possible just using touchscreen.

[humitos]

The patch attached fixes the issue with the traceback from comment:2.