Rockbox

Tasklist

FS#10151 - e200v2: scrollwheel acceleration

Attached to Project: Rockbox
Opened by Dustin Skoracki (sko) - Sunday, 19 April 2009, 20:09 GMT
Last edited by Thomas Martitz (kugel.) - Wednesday, 29 April 2009, 20:50 GMT
Task Type Patches
Category Drivers
Status Closed
Assigned To No-one
Operating System Sansa e200
Severity Low
Priority Normal
Reported Version Version 3.2
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch by kugel and me enables scrollwheel acceleration for the e200v2: fast wheel turning will cause lists to jump over items for fast scrolling.
   e200v2_scrollwheel_accelerati... (3.5 KiB)
This task depends upon

Closed by  Thomas Martitz (kugel.)
Wednesday, 29 April 2009, 20:50 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed in r20822. Good work!
Comment by Dustin Skoracki (sko) - Tuesday, 21 April 2009, 19:08 GMT
Here is a new patch, it's compiling again (old patch didn't work since r20757), has a little faster acceleration and is more simple.
   e200v2_scrollwheel_accelerati... (2 KiB)
Comment by Dustin Skoracki (sko) - Thursday, 23 April 2009, 14:35 GMT
New one: a little more time between to wheelclicks to be recognized as repeat and a little comment change to make it more clear.
   e200v2_scrollwheel_accelerati... (2 KiB)
Comment by Michael Chicoine (mc2739) - Thursday, 23 April 2009, 21:22 GMT
Seems to work well on my e260v2
Comment by Thomas Martitz (kugel.) - Sunday, 26 April 2009, 11:09 GMT
During a missed read, you're not executing scrollwheel(), is that intended? I'd think you get more smooth acceleration if you executed it anyway (it'll give the same button, but increase repeat).
Comment by Dustin Skoracki (sko) - Sunday, 26 April 2009, 12:46 GMT
Yes, at the last try it was intended, but it worked not as good as the first try, so, here is a new patch which executes scrollwheel during a missed read (very similar to first patch).
   e200v2_scrollwheel_accelerati... (3.8 KiB)
Comment by Dustin Skoracki (sko) - Sunday, 26 April 2009, 18:14 GMT
This is basically the same patch than the one before, but with imo better acceleration behavior.
   e200v2_scrollwheel_accelerati... (3.8 KiB)

Loading...