Rockbox Technical Forums

Support and General Use => User Interface and Voice => Topic started by: Mad Big Sausage on December 04, 2009, 06:26:44 AM

Title: First button press enables backlight only / Only on NAVI
Post by: Mad Big Sausage on December 04, 2009, 06:26:44 AM
Hello again Rb fans and devs. Long time no see.
I've just updated to the latest nightly after using a custom build from 2006 for the past few years. It has served me well but I got the urge to update and see what amazing new features and functionality you guys have come up with. I must say that I'm impressed.

But anyway, the purpose of this post is to find out what happened to one of my favourite H300 specific features: Backlight only on NAVI [select] (Flyspray (http://www.rockbox.org/tracker/task/2920)).
The patch is marked as commited but the current functionality is [all buttons] either perform action and enable backlight or just enable backlight without performing the action. Apart from playlist modifications I control my jukebox almost exclusively from inside my pocket. The use of the backlight at all in there is just a waste of battery life for me.

Anyone know of another patch created to replace the one that was nerfed? Or am I just overlooking a setting that's moved to another place in the settings' menu?

Cheers,
MbS
Title: Re: First button press enables backlight only / Only on NAVI
Post by: Llorean on December 04, 2009, 06:33:50 AM
When it was committed it was decided to be "all buttons" which you'll see if you read the comments on the task fully and note the later versions of the patch.

There is no H300-specific alternate version where only the Navi button does this. Also it seems you're slightly confused as to what the patch did - with the patch in its old state a press on any button *but* navi would do that button's function and turn the light on. If the backlight was off a press on navi would only turn the light on. If the light was on, then the press on navi would also do whatever navi did in that screen.

It's not away from avoiding turning the light on. It's a way to avoid performing an action when you just want to light up the screen so you can see it.
Title: Re: First button press enables backlight only / Only on NAVI
Post by: Mad Big Sausage on December 04, 2009, 06:37:04 AM
You're quite right. It was this patch (http://www.rockbox.org/tracker/task/5047) that I believe I was remembering.
It has been marked as rejected without any explanation or input from those on high.
Title: Re: First button press enables backlight only / Only on NAVI
Post by: Llorean on December 04, 2009, 06:47:34 AM
Your best bet is to just set a relatively short backlight timeout and not worry about it too much. I'm not certain but I seem to recall the remote not even lighting up the main screen LCD when it's used so you may wish to try one of those (though I'm basing my memory on rather ancient use on an H100 so it may have changed or be entirely incorrect).

Basically, there's considerable weighing between "keeping things streamlined" and "adding more functionality" and the battery life gained from this probably just wasn't terribly significant. Unless you're making constant adjustments you're just not going to have it on long enough.
Title: Re: First button press enables backlight only / Only on NAVI
Post by: Mad Big Sausage on December 04, 2009, 07:21:26 AM
Thanks Llorean. Great to see you're still a wealth of knowledge and common sense.

You're correct about the remote and main LCDs being seperate. I've got an LCD remote but usually wouldn't use it on a day-to-day basis. Excess cabling and limited pocket space with what I fondly refer to as my "brick" already in there. It normally only comes out when I travel.

I get what you are saying about the backlight length but it's a toss up between battery life and usability. With lengths of 5s and shorter the light frequently cuts out before actions are performed (at least in the older versions I've been using. Maybe the response times have improved enough so that this isn't an issue in the recent builds) and anything over that adds up to a lot of backlight usage. My normal listening practice is with playlists of 2,000+ tracks that I'm constantly skipping through to get to the tracks that currently fit my mood/situation so there are a lot of button presses.

Maybe I'll see if I can sync the old patch to today's builds. I'm expecting that it will be a sizable undertaking with the time that has passed since I was last involved here. Has the Rb architecture been completely revamped since I was last fiddling with it in '07 or will I still be able to navigate the code fairly easily? (I have moderate experience making my own builds with minor tweaks to existing patches. Also some basic knowledge of C).
Title: Re: First button press enables backlight only / Only on NAVI
Post by: pondlife on December 08, 2009, 02:45:43 AM
FWIW, I've found that it's easiest to use the hold button to enable the backlight.   

Because i have fat fingers and operate the device without looking at it, i used to often click on left or right as well as NAVI. 

Probably just me though!

pondlife