Rockbox

  • Status Unconfirmed
  • Percent Complete
    0%
  • Task Type Patches
  • Category Settings
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.8.1
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by tenfoot - 2011-04-19

FS#12075 - Add timeout to quickscreen

Following my patch to control line-out on the Fuze via a setting on the quickscreen (FS#12074), I found I kept forgetting to leave the quickscreen. This meant that when I intended to change track, I was actually changing the shuffle / repeat settings etc and had to fiddle around to get back to where I wanted to - not a good idea when driving at >70mph!

The attached patch adds a configurable timeout to the quickscreen so that it returns to the WPS screen if no button is touched within N seconds. 0 (the default) will disable the timeout preserving the existing behaviour.

Not sure if it needs to be configurable. 1s or 2 seems good for everyone IMO.

use TIME_BEFORE() or TIME_AFTER() to do the time checking or your patch wont do what you want it to do.

kugel, are you perhaps forgetting that we absolutly dont allow screens to change without the user knowing? this requires a setting if it has any chance of being accepted (I like the idea)

Do you mean we need a setting (just to make the user aware) or that we just just reject this?

EDIT: Forget the above sentence, I didn't read carefully enough.

Anyway, I didn't know about this. It's not on http://www.rockbox.org/wiki/NoDo

Admin
fg commented on 2011-04-20 11:23

I agree that it needs to be configurable. 1s or 2s may be good for some people, but I'm pretty sure that some other people will find that too fast. As soon as the actual time is configurable, disabling the timeout is trivial.

I haven't looked at the actual patch yet, but does this take voice into account? Breaking off voice halfway reading the first setting wouldn't be very nice.

I like the idea in general, and I suspect that I'll enable it on my players.

I'd love to see this in. The timeout should apply to the pitch screen as well.

I suggest to let the timeout be configurable or to set it to the backlight timeout.

Admin
fg commented on 2011-04-20 11:33

You can't really use the backlight timeout: not all players have a backlight in the first place, and some (especially mono and greyscale) screens are perfectly readable without a backlight which means that a lot of people will have set their backlight to off altogether.

If this sort of thing is added, it must be optional. If you're not looking at the screen (or are blind), then any kind of timed mode change is not nice. I prefer to know the UI is where I left it.

Every screen could theoretically benefit from the timeout. There've been several requests to have it return to the WPS after a timeout.

Rather than having specific screens have timeouts, why not just have a "Return to WPS after X seconds" option, if music is playing, that will cover pitch, quickscreen, main menu, filetree, etc all with a single setting?

dfkt commented on 2011-10-25 14:52

Resynced to 30835.

you'll probably have more luck if you push this to gerrit.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing