- Status Closed
- Percent Complete
- Task Type Patches
- Category Music playback
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#9448 - Rewind on fade
This patch rewinds the current title when resuming from pause if fade in/out is enabled. It is based on the headphone detection code. The rewind amount is hard coded at 2 seconds. I plan to add settings to enable/disable and adjust rewind.
Closed by sideral
2011-05-13 23:03
Reason for closing: Out of Date
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
2011-05-13 23:03
Reason for closing: Out of Date
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
Subsumed by FS#11931 , which was committed in
r29876–r29878.
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
Updated with settings.
Settings → General Settings → Playback Settings → Fade on Stop
Rewind on Fade = enable/disable (default - disable)
Rewind on Fade Amount = select rewind amount in seconds (default - 2 s)
Wouldn’t it be better to derive the rewind amount from the fade in and fade out duration? E.g. as the sum? Then we’d need no additional settings and the goal would be achieved (which is: not to miss a thing when stopping and then starting playback).
That sounds like a good idea, but there is no setting for fade in / fade out duration, and my understanding is that the duration varies among the different targets.
The rewind issue isn’t related to fades, or pause on headphones, really. What I’d prefer to see is a setting for “Rewind on resume” - where any restart (or resume from pause), be it button or headphone controlled - includes an automatic rewind.
If I understand the original post correctly, the issue *is* related to fades. If you want a general ‘rewind on resume’ it’d be another feature. Or even better: we could have a setting ‘Rewind on resume’ with the possible values “Never”, “On fade only”, and “Always.” The question about the rewind amount still remains. For the first two values there is a logical answer, but not for the third. But even for that we can just hard code something, e.g. one second.
Sorry, I was incorrect to say it wasn’t related to fades. What I meant was it might be independent of fades (e.g. an audiobook user may want to auto-rewind by 30 seconds each time they resume, anyway).
I don’t think there’s a need to tie the config in with fades - a single option for length of “resume rewind” should be enough (0 by default). Then you can set that when you enable fades, and reset it when you disable fades (a .cfg file makes this sort of thing easy enough).
Ok, but I still think that the option ‘On fade only’ could make sense – if you want to not miss a second but also want to hear nothing twice. This is like “Show file extensions only if file filter is set to ‘supported’“. You could have the same effect by changing two options but the option is ‘smart’ so you can do it by changing just one.
I like the idea of “Rewind on resume”. This would be better than the code duplication I have added with my patch. I also like the settings fml2 suggested, but should values for headphone detection also be added?
As someone who uses rockbox primarily to listen to audiobooks I think a “Rewind on Resume” option would be very useful.
I love this feature.
Unfortunately, the patch does not apply any more. I’ve attached an adapted patch that I forward ported to a recent SVN revision.
I just removed the empty line added to the bottom of english.lang, because it prevents to patch it with further patches.