- Status Closed
- Percent Complete
- 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
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.
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
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.
:)
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
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)
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..
.. 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)
well, for me even clean cvs viewer crashes sim … but works fine on player (ipodmini2g)
hm … but with this patch i can’t crash it on my target’s sim as well as on player :)
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 :)
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
well, those bug mentioned before looks like ipod-specific … http://www.rockbox.org/tracker/task/4729 and doesn’t go here