This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#2653 - added one-click-to-insert functionality in file tree
Attached to Project:
Rockbox
Opened by rubberglove (rubberglove) - Sunday, 28 August 2005, 02:59 GMT+1
Last edited by Martin Scarratt (mmmm-) - Monday, 18 September 2006, 20:54 GMT+1
Opened by rubberglove (rubberglove) - Sunday, 28 August 2005, 02:59 GMT+1
Last edited by Martin Scarratt (mmmm-) - Monday, 18 September 2006, 20:54 GMT+1
|
Detailsthat's it. for iriver (i think) click the joystick
(select) to insert the selected track/directory/playlist to the current playlist (or create a new one if none) seems to work, use with caution, enjoy. 
oneclick.diff
(3.2 KiB)
onplay.c | 9 +++++++++ onplay.h | 1 + tree.c | 44 +++++++++++++++++++++++++++++++++++++------- 3 files changed, 47 insertions(+), 7 deletions(-)
oneclick_custom.difff
(7.1 KiB)
oneclick-custom-insert_rubbergloveMmmm.patch
(7.9 KiB)
lang/english.lang | 13 +++++++++++++ onplay.c | 22 ++++++++++++++++++++++ onplay.h | 1 + playlist_menu.c | 17 +++++++++++++++++ settings.c | 1 + settings.h | 3 +++ tree.c | 52 +++++++++++++++++++++++++++++++++++++++++++++------- tree.h | 2 ++ 8 files changed, 104 insertions(+), 7 deletions(-)
oneclick-custom_insert_rubbergloveMmmm.patch
(7.9 KiB)
lang/english.lang | 13 +++++++++++++ onplay.c | 22 ++++++++++++++++++++++ onplay.h | 1 + playlist_menu.c | 17 +++++++++++++++++ settings.c | 2 +- settings.h | 3 +++ tree.c | 52 +++++++++++++++++++++++++++++++++++++++++++++------- tree.h | 2 ++ 8 files changed, 104 insertions(+), 8 deletions(-) |
This task depends upon
an update that works with recent builds (2005-10-31) can be
found here: http://forums.rockbox.org/index.php?
action=dlattach;topic=1617.0;id=249
I hope the patch lasts until it finally find its way into the CVS ;)
First patch ever, so watch out. :)
Any chance to get an update for the new changes in the settings.c code?
Thanks a lot, Norbert
Still hoping this can be included in svn
Hope it works for you.
Also, there is no chance of commit as long as 1) this isnt ifdefed to only work for targets which have a spare button (i.e not ondio), 2) is changed to replace ACTION_STD_OK when a target has 2 buttons which can do that (iriver, sansa, ipod, you wold use right instead of SELECT), 3) add an enum for the various setting values, if(global_settings.click_insert==1) is nasty. 4) the stuff in playlist_menu.c (the option config) needs to be moved to settings_list.c (copy the CHOICE_SETTING() macro items).
-This patch causes SELECT not to work in the menus (have to use NEXT to choose menu items). It would be better if it did not affect the menus, only the file browser, in my opinion. I looked at the code a little, but it is beyond me. By the way, I am using a Gigabeat F.
-It would be nice if there were a "replace current playlist" option as well (similar to pressing ENTER while a track is selected from the file browser).