- Status Unconfirmed
- Percent Complete
- 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
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.
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
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
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.
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?
Resynced to 30835.
Resynced.
Resynced.
you'll probably have more luck if you push this to gerrit.