• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category User Interface
  • Assigned To No-one
  • Operating System Another
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.4
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by jasio - 2009-10-28
Last edited by mcuelenaere - 2009-10-28

FS#10733 - Touchscreen targets: software hold also locks touchscreen

When the software lock is on, the touchscreen should be locked too. This is what this tiny patch does.

In action.c function get_action_worker() handles the lock by discarding the button events before they are processed. The same doesn't happen for touchscreen events because the code is placed before the lock handling.
I have moved the HAVE_TOUCHSCREEN section just after the HAS_BUTTON_HOLD section and the touchscreen actions are now blocked when the lock is active.

Closed by  mcuelenaere
2009-10-28 21:36
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Thanks! In r23384.

jasio commented on 2009-10-28 21:26

First patch added a space. Removed!


Available keyboard shortcuts


Task Details

Task Editing