Rockbox

Tasklist

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

Attached to Project: Rockbox
Opened by Yann Muller (jasio) - Wednesday, 28 October 2009, 21:12 GMT
Last edited by Maurus Cuelenaere (mcuelenaere) - Wednesday, 28 October 2009, 21:36 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System Another
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

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.
This task depends upon

Closed by  Maurus Cuelenaere (mcuelenaere)
Wednesday, 28 October 2009, 21:36 GMT
Reason for closing:  Accepted
Additional comments about closing:  Thanks! In r23384.
Comment by Yann Muller (jasio) - Wednesday, 28 October 2009, 21:26 GMT
First patch added a space. Removed!

Loading...