Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#6064 - Report current recording directory in recording directory menu,

Attached to Project: Rockbox
Opened by Davide (Davide-NYC) - Monday, 25 September 2006, 18:37 GMT+2
Task Type Feature Requests
Category Recording
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

I humbly ask that an extra static line be added to the - menu -> recording -> recording settings -> directory which reports back to the user what the current recording directory is along with explanatory tags.
This would greatly clarify what is going on to the new user.
Example:

Directory
/UserSelectedRecordingDirectory
Default (/recording)
Current Directory (/LastDirectoryInFileBrowser)

The line with the "UserSelectedRecordingDirectory" would not be selecatable, but would change to "LastDirectoryInFileBrowser" if the user selects 'Current Directory' and then presses Navi. That way the user gets some feedback that something has aactually happened when they do modify the Recording Directory. In the default config it would simply say "/recording".

This task depends upon

Closed by  Björn Stenberg (zagor)
Wednesday, 04 April 2007, 21:38 GMT+2
Reason for closing:  Out of Date
Additional comments about closing:  Closing all feature requests.
Comment by Davide (Davide-NYC) - Monday, 25 September 2006, 18:48 GMT+2
In case this is unclear, "UserSelectedRecordingDirectory" will be the actual currently set recording directory, "LastDirectoryInFileBrowser" is the actual last directory selected in the file browser. The only static one is "recording" which is the default.
Comment by Linus Nielsen Feltzing (linusnielsen) - Monday, 25 September 2006, 19:35 GMT+2
Current Directory means that when you start recording, the files will end up in the directory you browsed last. Always. From your request, it seems as if you think that the directory name is remembered in the settings when you select Current Directory, which it isn't. You can select Current Directory and then browse to another one afterwards. Then the files will end up there.
Comment by Davide (Davide-NYC) - Monday, 25 September 2006, 20:51 GMT+2
Oh my. That is (in my opinion) terrible.
Who want's to scatter recordings in that manner?
I will proselytize on IRC immediately.
Comment by Peter D'Hoye (petur) - Monday, 25 September 2006, 21:17 GMT+2
sorry to have caused confusion, I was thinking it stored the current directory location and always used that one... skuze me
Comment by Davide (Davide-NYC) - Monday, 25 September 2006, 21:36 GMT+2
This feature request is still valid. Having the recording directory 'jump around' with your playback habits is crazy in my opinion. At the very least the "set current directory" option should be 'sticky' in that it gets stored and doesn't follow your file browsing.
Comment by Jonas Häggqvist (rasher) - Monday, 25 September 2006, 21:41 GMT+2
The problem with that is that it would take a huge amount of space in the config block, which is not feasible without storing config in a file (which afaik is being worked on).

I think you should open a new feature request which is more specific about wanting a user-selectable (and static) location for recordings.
Comment by Martin Scarratt (mmmm-) - Wednesday, 27 September 2006, 11:32 GMT+2
Here's a patch that saves the setting you set when you last clicked on Current directory which I agree with Davide on, should be the way it works... but Jonas, you're right, A 260 byte setting stored in the config block...NAH...
   recpath.patch.txt (4.8 KiB)
Comment by Davide (Davide-NYC) - Tuesday, 31 October 2006, 18:38 GMT+2
This feature request is *still* valid. You should see what the recording directory is set to without having to change it.
This is how it is:

Directory
/recordings
Current Directory

This is what I'm proposing at the very least:

Directory (/MySetRecDir)
/recordings
Current Directory (/LastBrowsedRec)

Just so you know what the values are currently and what the would become without having to make a test recording and leave recording altogether to confirm.
Comment by Davide (Davide-NYC) - Tuesday, 31 October 2006, 18:39 GMT+2
For clarity: (/LastBrowsedRec) should be (/LastBrowsedDir) in the above example.
Comment by Jonas Häggqvist (rasher) - Tuesday, 31 October 2006, 18:42 GMT+2
But "Current Directory" doesn't mean any one directory, so writing a directory name after that setting would cause confusion.
Comment by Davide (Davide-NYC) - Tuesday, 31 October 2006, 18:51 GMT+2
OK, so newline it

Directory
Current /MySetRecDir
Default (/recordings)
Last Browsed (/LastBrowsedRec)

Where the Current line woould not be 'selectable' thus the user would know that that is the currently set recording directory.
Comment by Jonas Häggqvist (rasher) - Tuesday, 31 October 2006, 19:32 GMT+2
Perhaps that wasn't entirely clear.

You can't put anything after "Last Browsed" that would make any sense, since it's not stored as any specific location, but changes, so the user would be confused if he picked that option.
Comment by Jonas Häggqvist (rasher) - Tuesday, 31 October 2006, 19:34 GMT+2
Also, as for your previous comment:
> Who want's to scatter recordings in that manner?

If you know how that setting is implemented, you'd simply browse to the dir where you want to save, and then enter the recording screen. It's a very useful setting if you ask me.
Comment by Jonas Häggqvist (rasher) - Tuesday, 31 October 2006, 20:24 GMT+2
To be honest, I think this request should be closed entirely and replaced by  FS#6268 . There's no way to properly tell the user which folder is going to be used, since it'll change all the time.
Comment by Davide (Davide-NYC) - Wednesday, 04 April 2007, 21:29 GMT+2
This feature request should be closed and either
A: be resubmitted (by me) to ask for a While Recording Screen line that reports to the user where the recording will be (or is being) written
or
B: have  FS#6268  implemented (which would be way better in my opinion).

Loading...