- Status Closed
- Percent Complete
- Task Type Patches
- Category User Interface
- 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#4988 - Optionally keep backlight on while plugins are running
This patch will add an option to General settings/Display/LCD settings to keep backlight on when using plugins
My first patch, so be gentle.. :)
Closed by jdgordon
2007-07-31 12:54
Reason for closing: Rejected
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
2007-07-31 12:54
Reason for closing: Rejected
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
This idea was commited a while ago, but
immediatly reverted because it was
decided it wasnt worth the bin size
cost
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
small change
moved setting save to end of block (as suggested..)
Ahum.. one should check a patch before uploading.. Revious one works, but needed -p1 ..
This one’s tested and requires no ‘special tricks..’ :)
Good
I think each plugin should decide itself if it should allow to switch off the backlight or
does absolutely make no sense for game plugins to switch off the
e.g. for encoding or battery testing it does absolutely make sense!
So I suggest, to add a API function with which the plugins could decide theirselve if the
be switched off or not.
Nevertheless I resynched your patch with the current CVS (langv2 changes)...
Actually this exists in the current API (backlight_on, backlight_off and backlight_set_timeout) but is never used it seems..
I just like the option to keep backlight on for plugins.. If it never makes it into CVS I’ll stil use it, unless plugin writers start using the available options.. :^)
As far as I know backlight_on switches the backlight on (for the
switches it
backlight_set_timeout changes the timeout for switching the backlight automatically
me if I’m wrong)
That’s not exactly what I
thought about an easy function, with which the plugin can either
ever switch the backlight off as long as I
“use the normal backlight settings”
“use my special timeout”
That’s all - and this should then automatically be used as long as the plugin
resets to normal behaviour when the plugin
further function call of the plugin itself).
BTW, grep at the plugin directory you’ll find some
backlight_off calls ;)
Updated patch to prevent double patches in language file.
I would like to see this in CVS.. but if not Oh well.. ;^)
Requires updating for SVN
It’s really not that hard to do..
Sorry I don’t know how to, I wish I knew... I’ll try learning myself
Great patch works on 5.5G iPod 30GB. One suggestion: changes the options to On and Off to keep it consistent with the other backlight settings. Not sure if this is causing Doom to turn off the backlight in game though
Tested on:
5.5G
iPod
iPod
iPod Nano
Also tested on 4G Grayscale. SVN ready me thinks
synced