Rockbox

  • Status Closed
  • Percent Complete
    100%
  • 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
Attached to Project: Rockbox
Opened by Paul van der Heu - 2006-04-01
Last edited by Jonathan Gordon - 2007-07-31

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.. :)

   plugin-backlight.patch (3.9 KiB)
Closed by  Jonathan Gordon
2007-07-31 12:54
Reason for closing:  Rejected
Additional comments about closing:  

This idea was commited a while ago, but immediatly reverted because it was decided it wasnt worth the bin size cost

Paul van der Heu commented on 2006-04-01 21:04

small change

moved setting save to end of block (as suggested..)

   plugin-backlight.patch (3.8 KiB)
Paul van der Heu commented on 2006-04-01 21:18

Ahum.. one should check a patch before uploading.. Revious one works, but needed -p1 ..

This one’s tested and requires no ‘special tricks..’ :)

   plugin-backlight.patch (3.6 KiB)
Matthias Mohr (aka Massa) commented on 2006-04-06 07:55

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)...

(application/octet-stream)    plugin-backlight_20040505.pat... (3.8 KiB)
Paul van der Heu commented on 2006-04-06 13:54

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.. :^)

Matthias Mohr (aka Massa) commented on 2006-04-07 11:59

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 ;)

Paul van der Heu commented on 2006-08-22 15:57

Updated patch to prevent double patches in language file.

I would like to see this in CVS.. but if not Oh well.. ;^)

   plugin-backlight_060822.patch (3.8 KiB)
Chris commented on 2007-01-27 23:01

Requires updating for SVN

Paul van der Heu commented on 2007-01-28 10:53

It’s really not that hard to do..

   plugin-backlight_070128.patch (3.3 KiB)
Chris commented on 2007-01-28 10:56

Sorry I don’t know how to, I wish I knew... I’ll try learning myself

Chris commented on 2007-02-03 06:30

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

Chris commented on 2007-02-05 01:43

Tested on:

5.5G
iPod
iPod
iPod Nano

Chris commented on 2007-02-05 07:20

Also tested on 4G Grayscale. SVN ready me thinks

Max Weninger commented on 2007-03-03 15:14

synced

   plugin-backlight_070303.patch (3.5 KiB)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing