Rockbox

Tasklist

FS#2648 - text viewer button mods

Attached to Project: Rockbox
Opened by rubberglove (rubberglove) - Wednesday, 24 August 2005, 00:27 GMT
Last edited by Torne Wuff (torne) - Saturday, 20 March 2010, 21:16 GMT
Task Type Patches
Category
Status Closed
Assigned To No-one
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

modified button assignment in text viewer slightly so that you have
to HOLD left/right to jump to the beginning/end of the file, instead of
simply pressing... this should avoid accidental jumping around.
This task depends upon

Closed by  Torne Wuff (torne)
Saturday, 20 March 2010, 21:16 GMT
Reason for closing:  Fixed
Additional comments about closing:  Implemented in r25262
Comment by rubberglove (rubberglove) - Thursday, 25 August 2005, 15:52 GMT

changed button mapping slightly so that (in wide mode) you must press
and hold left/right to jump to the beginning/end of the file, to prevent
accidental jumping around.
also added basic track skipping (previous and next track) for iriver
next=play+select
previous=play+stop
Comment by Victor Karpunin (kva) - Friday, 29 December 2006, 07:17 GMT
It's good patch for me. Because I sometimes click left/right buttons in narrow mode and it isn't comfortably to jump to the beginning/end of the file, when I read it. So I try to rebuild my own rockbox version with the patch. And I found some troubles with it. Really I'm not a programmer, C programmer but I corrected errors, I think so, and for me patch works. So if someone wants to use it, I upload it.
Comment by Victor Karpunin (kva) - Monday, 30 July 2007, 13:24 GMT
My last sync.
Comment by Philip Hansen (Philip Hansen) - Monday, 31 March 2008, 20:01 GMT
Has anyone made a patch that allows you to change pages by pressing the NEXT and PREV buttons, instead of scrolling? Sometimes I accidently touch the scroll pad and it's rather anoying.
Comment by Jonathan Gordon (jdgordon) - Thursday, 17 April 2008, 13:25 GMT
So, I was about to commit this but we need your full real name.. please add it in a comment so this can go in.
Comment by Victor Karpunin (kva) - Wednesday, 23 April 2008, 18:34 GMT
@Jonathan Gordon:
If you mean my real name, it's Victor Karpunin.
If you mean rubberglove, I don't know who is he :)
Comment by Torne Wuff (torne) - Saturday, 20 March 2010, 21:16 GMT
I have implemented this differently (it just remaps the buttons before entering the switch statement, which is less code), and it's committed in r25262.

The track skipping buttons are not implemented as the current version of the text viewer uses more button combinations for bookmarks and other new features, and there isn't room any more :)

Loading...