remapping buttons for list scrolling (ipod video)?

[peterw]

I'd like to remap Hold Prev and Hold Next so they can be used to scroll through long lists, as when browsing the database. I hoped this might be as simple as the following patch, but apparently it's not. Could someone help me figure this out?

Thanks,

-Peter

--- apps/keymaps/keymap-ipod.c  (revision 14204)
+++ apps/keymaps/keymap-ipod.c  (working copy)
@@ -41,10 +41,12 @@
 static const struct button_mapping button_context_standard[]  = {
     { ACTION_STD_PREV,          BUTTON_SCROLL_BACK,                 BUTTON_NONE },
     { ACTION_STD_PREVREPEAT,    BUTTON_SCROLL_BACK|BUTTON_REPEAT,   BUTTON_NONE },
+    { ACTION_STD_PREV,          BUTTON_LEFT|BUTTON_REPEAT,          BUTTON_NONE },
     { ACTION_STD_NEXT,          BUTTON_SCROLL_FWD,                  BUTTON_NONE },
     { ACTION_STD_NEXTREPEAT,    BUTTON_SCROLL_FWD|BUTTON_REPEAT,    BUTTON_NONE },
-    { ACTION_STD_CANCEL,        BUTTON_LEFT,                        BUTTON_NONE },
-    { ACTION_STD_OK,            BUTTON_RIGHT,                       BUTTON_NONE },
+    { ACTION_STD_NEXT,          BUTTON_RIGHT|BUTTON_REPEAT,         BUTTON_NONE },
+    { ACTION_STD_CANCEL,        BUTTON_LEFT|BUTTON_REL,             BUTTON_NONE },
+    { ACTION_STD_OK,            BUTTON_RIGHT|BUTTON_REL,            BUTTON_NONE },

     { ACTION_STD_OK,            BUTTON_SELECT|BUTTON_REL,           BUTTON_SELECT },
     { ACTION_STD_MENU,          BUTTON_MENU|BUTTON_REL,             BUTTON_MENU },

[peterw]

OK, bad form, replying to my own post. I got it figured out, and posted a patch on Flyspray:
http://www.rockbox.org/tracker/task/7545

-Peter

[Llorean]

Wouldn't a better solution be improving the scroll acceleration code, so that holding left and right can be used to scroll long filenames manually as can be done on other targets?

[tdtooke]

There is another patch with addresses this that you may be interested in.  You have an added scroll-acceleration setting for fast, faster, fastest which you can set to whatever suits you.

[peterw]

There is another patch with addresses this that you may be interested in.  You have an added scroll-acceleration setting for fast, faster, fastest which you can set to whatever suits you.

Thank you. I think that will become abother "core patch" in my build -- that makes the wheel much more usable!

-Peter