Rockbox

Tasklist

FS#12024 - List scroll acceleration on non-scrollwheel targets

Attached to Project: Rockbox
Opened by Andree Buschmann (Buschel) - Tuesday, 22 March 2011, 07:16 GMT
Last edited by Andree Buschmann (Buschel) - Thursday, 24 March 2011, 18:53 GMT
Task Type Bugs
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.8
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

A forum post (http://forums.rockbox.org/index.php/topic,27409.0.html) made me check the code and the manual for the settings "List Acceleration Start Delay" and "List Acceleration Speed".

1) There is no manual entry for those settings.
2) The unit for "List Acceleration Start Delay" was UNIT_MS, correct is UNIT_SEC.
3) The implementation used HZ/2 instead of HZ for the delays.
4) The implementation did not double the scroll speed each step.
5) The recognition of a repeated key in action.c uses a too short window (HZ/10). The recognition works more reliable in my tests with HZ/4.
This task depends upon

Closed by  Andree Buschmann (Buschel)
Thursday, 24 March 2011, 18:53 GMT
Reason for closing:  Fixed
Additional comments about closing:  Kept the linear acceleration but fixed manual and the acceleration in general.
Comment by Thomas Martitz (kugel.) - Tuesday, 22 March 2011, 08:33 GMT
Why double the scroll speed? That's nonlinear acceleration, that doesn't sound nice.
Comment by Andree Buschmann (Buschel) - Tuesday, 22 March 2011, 12:33 GMT
Depends on how it feels if used. I cannot judge this as I do only have scrollwheel targets and could only test on simulator. Regarding nonlinearity: On scrollwheel targets like the iPod the acceleration is list speed = v^3 (v=rotation speed of wheel) -- for this usecase it feels just right.

Do you have the possibility to test this patch on a target?
Comment by Andree Buschmann (Buschel) - Tuesday, 22 March 2011, 20:41 GMT
New version of the patch for the manual.
Comment by Andree Buschmann (Buschel) - Tuesday, 22 March 2011, 21:16 GMT
Submitted apps/settings_list.c and manual/configure_rockbox/display_options.tex as a first step (r29634). Using the correct time base (HZ instead of HZ/2) was 2nd step (r29635).

Loading...