Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Plugins
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by jdgordon - 2006-04-07

FS#5056 - reworked viewer plugin

i have reworked a fair bit of the viewer plugin to make it a bit nicer… the off button now quits again, and each target now has a dedicated menu button. all the plugin settings are edited from the menu so no need to remeber which key does which anymore..

also, ive added a new setting to choose if u want to scroll by page or line (up/down only atm) and you can save settings as the default settings, not only per-file settings now.

everything works, except the scrollbar setting, and im not sure why.. so someone else can see if they an figure that out.

(application/octet-stream)    viewer.diff (18 KiB)
Closed by  midkay
2006-05-01 06:13
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Added in CVS. Thanks. :)

theli commented on 2006-04-17 13:56

bug: enter setting ‘Scrollbar Mode’ → don’t change → press menu → it displays ‘canceled’ but enter again → setting reversed
pushing ‘select’ has the same effect … otherwise scrollbar setting really works

except that when scrollbar is disabled there is still space for it in the left

otherwise this patch greatly enhances viewer (cause using different hotkeys is REALLY horrible (for example - on ipod pressing select+prev+next is way hard)

merging this with ‘half-screen width’ bugfix from
http://www.rockbox.org/tracker/task/4873 would make viewer way better and comfortable to use

(well, codepage selection would be nice too… but not critical)

jdgordon commented on 2006-04-17 14:20

ok, ive removed some code which im not 100% sure what it does, but it fixed the scroll bar problem :p it apears to be working… merging that other patch will be fun… ill give it a go..

(application/octet-stream)    viewer.diff (18.2 KiB)
jdgordon commented on 2006-04-17 15:10

.. well that was simple… the other patch merged perfectly (well almost) with my changes… so here we have both patches together (with encoding selection, dunno if it works tho)..
only problem is that the line mode and scrollbar options crashes the sim but works fine on the target (h300).. so if some1 wants to figure that out? (i dont have gdb setup)

(application/octet-stream)    viewer_encoding.diff (32.2 KiB)
theli commented on 2006-04-17 15:33

well, for me even clean cvs viewer crashes sim … but works fine on player (ipodmini2g)

theli commented on 2006-04-17 15:38

hm … but with this patch i can’t crash it on my target’s sim as well as on player :)

theli commented on 2006-04-17 15:42

well… configuring options would be cool feature too :P
cause for example, on ipod, scrolling with wheel is not to convinient … as for me i would disable wheel scroll … have select+menu for menu, (or just select) , and play - scroll down, menu - scroll up

well, maybe that is just the matter of taste so .. ability to modify control keys would be cool to :)

theli commented on 2006-04-17 17:53

hm … another bug?
1. viewer doesn’t saves file position on player but does on simulator :|
2. i have always had for unknown reason a symbol ‘J’ in places where there are end of lines in file … this happened both with sim and player… now with the sim it is fixed but i still have this on player …

it looks like this
http://theli.ho.com.ua/book.png with vanilla viewer.c

theli commented on 2006-04-20 13:57

well, those bug mentioned before looks like ipod-specific … http://www.rockbox.org/tracker/task/4729 and doesn’t go here

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing