FS#7486 : Favourites plugin.

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

Quick links: Bugs · Patches · Rockbox frontpage

FS#7486 - Favourites plugin.

Attached to Project: Rockbox
Opened by Bryan Childs (GodEater) - Thursday, 26 July 2007, 17:03 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Sunday, 05 August 2007, 14:14 GMT+2

Task Type	Patches
Category	Plugins
Status	Closed
Assigned To	No-one
Player Type	All players
Severity	Low
Priority	Normal
Reported Version	Daily build (which?)
Due in Version	Undecided
Due Date	Undecided
Percent Complete
Private	No

Details

This is a first effort at a "favourites" plugin, which will allow the user to store up to ten shortcuts to directories somewhere in their file tree.

TODO:
Add functionality to write the .rockbox/rocks/favourites.link file since this currently needs to already exist with entries in it for the plugin to do much. The format of the file is assumed to be one directory path per line, e.g. :

/dir1/
/dir2/dir3/

The trailing slash is a must have.

BUGS:

Currently the call to set_current_file(), which is supposed to move the file browser to the selected directory is causing the sim to seg fault. Haven't even tried this on an actual DAP yet.

This task depends upon

Closed by Jonathan Gordon (jdgordon)
Sunday, 05 August 2007, 14:14 GMT+2
Reason for closing: Accepted

Comments (47)
Related Tasks (1/0)

Comment by Bryan Childs (GodEater) - Thursday, 26 July 2007, 17:07 GMT+2

And here's the patch...

favourites.patch (11.9 KiB)

 plugin.h               |    3 
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  342 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    2 
 tree.h                 |    1 
 6 files changed, 348 insertions(+), 2 deletions(-)

Comment by Bryan Childs (GodEater) - Thursday, 26 July 2007, 17:24 GMT+2

Fixed the segfault, but the plugin isn't displaying much after the set_current_file() call now.

favourites.patch (12.1 KiB)

 plugin.c               |    1 
 plugin.h               |    3 
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  338 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    2 
 tree.h                 |    1 
 7 files changed, 345 insertions(+), 2 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 27 July 2007, 13:08 GMT+2

With a bit more hackery, the plugin now jumps to the right place in the filetree even when launched from the plugin menu.

This means hacking the dirfilter setting on the plugin browser, but I don't think anyone will notice ;)

favourites.patch (12.8 KiB)

 plugin.c               |    2 
 plugin.h               |    4 
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  355 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    8 -
 tree.h                 |    2 
 7 files changed, 371 insertions(+), 2 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 27 July 2007, 14:18 GMT+2

Tidied the code up a lot, and split a lot of functions up to make them more re-use friendly.

TODO:

1) Still need the ability to write the favourites file itself.
2) Possibly add directly to the context menu for directories to make adding entries quick

BUGS:

None that I know of ;)

favourites.patch (13.8 KiB)

 plugin.c               |    2 
 plugin.h               |    4 
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  387 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 7 files changed, 404 insertions(+), 2 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 27 July 2007, 18:07 GMT+2

Bit more reorganisation.

favourites.patch (15 KiB)

 plugin.c               |    2 
 plugin.h               |    4 
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  346 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   41 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 8 files changed, 404 insertions(+), 2 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 13:14 GMT+2

This now features the writing of the favourites file, and a context menu entry for it when the user wants to add a directory.

Still TODO:
Allow user to remove entries from the favourites file.

favourites.patch (18.9 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   14 +
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  368 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   42 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 455 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 13:24 GMT+2

Removed a call to rb->gui_synclist_draw() at JdGordon's behest.

favourites.patch (18.9 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   14 +
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  368 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   42 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 455 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 13:33 GMT+2

Whoops - managed to leave a couple of calls to glibc's snprintf() instead of rb->snprintf() in the code.

favourites.patch (18.9 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   14 +
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  368 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   42 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 455 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 16:27 GMT+2

Another whoops - I managed to have "Add to favourites" in the context menu even if the selected item wasn't a directory. This fixes that.

favourites.patch (18.7 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   13 +
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  368 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   42 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 455 insertions(+), 5 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 17:03 GMT+2

Make the plugin display an error if the directory selected in the favourites file no longer exists on disk.

favourites.patch (19.1 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   13 +
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  378 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   42 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 465 insertions(+), 5 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 17:22 GMT+2

Tidied up some function prototypes to get rid of warnings when building for an actual DAP rather than just as a simulator.

favourites.patch (19.1 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   13 +
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  378 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   42 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 465 insertions(+), 5 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 21:28 GMT+2

Moved the location of the favourites.link file to the root dir (don't panic amiconn, it's only created if you use the plugin).

Fixed a bug with saving a new file.

favourites.patch (5.1 KiB)

 lang/english.lang      |   14 ++++++++++++++
 onplay.c               |   13 ++++++++++++-
 plugin.c               |    2 ++
 plugin.h               |    8 +++++---
 plugins/SOURCES        |    1 +
 plugins/viewers.config |    1 +
 tree.c                 |    9 ++++++++-
 tree.h                 |    2 ++
 8 files changed, 45 insertions(+), 5 deletions(-)

Comment by Bryan Childs (GodEater) - Monday, 30 July 2007, 21:53 GMT+2

Oops. Just noticed I missed out a load of things from the patch. Like most of it's functionality.

favourites.patch (18.9 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   13 +
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/favourites.c   |  378 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/favourites.h   |   42 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 465 insertions(+), 5 deletions(-)

Comment by Gerritt Gonzales (GRaTT) - Monday, 30 July 2007, 22:47 GMT+2

I am using a sansa and it will not work for me.
Error is "directory no longer exist on disk"
for all links made by the plugin.
If I edit the link file and remove the end /
the plugin exits to the file browser with the
favourite link selected.
In other words one directory up from the link
with the link directory selected.

I would like to see this selectable from the main menu
and be able to start plugins as well or songs.
GRaTT

Comment by Jeton Aliji (jeton) - Tuesday, 31 July 2007, 00:45 GMT+2

I would suggest you include a template file/config in the patch so that it doesn't cause problems with users trying to make it work. And from that template they could easily change the path of the folders.

Comment by Bryan Childs (GodEater) - Tuesday, 31 July 2007, 08:50 GMT+2

@GRaTT:

I'm willing to bet you created the .link file on your computer and copied it to the sansa correct ?

@jeton

No need to include a template - the plugin creates the file and adds the entries to it itself. Having a template file will only lead people to editing it in a windows text editor and breaking the format.

Comment by Bryan Childs (GodEater) - Tuesday, 31 July 2007, 15:57 GMT+2

Firstly, sorry for the size of the patch - my diff seems to think I've changed masses of stuff in onplay.c - which I haven't.

Secondly - this has now been completely reworked to use the rb->read_line() function instead of my own file parsing code - so windows edited files shouldn't break the functionality anymore.

Thirdly - you can now mark individual files as well as directories.

Fourthly - I've renamed the patch to "shortcuts" to avoid confusion with an earlier "favourites" plugin which was ditched months and months back.

favourites.patch (30.3 KiB)

 lang/english.lang      |   14 ++
 onplay.c               |  110 ++++++++-------
 plugin.c               |    2 
 plugin.h               |    8 -
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  338 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   46 ++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 477 insertions(+), 54 deletions(-)

Comment by Bryan Childs (GodEater) - Tuesday, 31 July 2007, 16:17 GMT+2

Be an idea to actuall rename the patch too I suppose ;)

shortcuts.patch (17.8 KiB)

 lang/english.lang      |   14 ++
 onplay.c               |   16 +-
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  338 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   46 ++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 429 insertions(+), 6 deletions(-)

Comment by Gerritt Gonzales (GRaTT) - Tuesday, 31 July 2007, 16:58 GMT+2

@Godeater
I only used the link file as created on the sansa.
Also edited it on the sansa to test it by removing the end slash(/).
I will try the new patch and recreate the link file.
GRaTT

Comment by Jeton Aliji (jeton) - Tuesday, 31 July 2007, 18:40 GMT+2

I'm using cpchan's build on Sansa. When i start the plugin i get "Couldn't find the file".

Comment by Bryan Childs (GodEater) - Tuesday, 31 July 2007, 21:38 GMT+2

@jeton : If you've compiled this plugin into your build, then you're not using cpchan's build anymore...

Comment by Bryan Childs (GodEater) - Wednesday, 01 August 2007, 14:46 GMT+2

First attempt at adding a "delete" function. Which sadly doesn't work at the moment. No idea why though.

shortcuts.patch (19.2 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   16 +-
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  379 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   54 ++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 +
 tree.h                 |    2 
 10 files changed, 478 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Wednesday, 01 August 2007, 17:21 GMT+2

Now with new working delete option!

In order to delete an entry from the links file, click on it with the "Menu" key.

**WARNING* There is no prompt currently to check if you're sure of not - it just gets wiped.

shortcuts.patch (20 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  405 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   54 ++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 -
 tree.h                 |    2 
 10 files changed, 504 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Wednesday, 01 August 2007, 21:50 GMT+2

After many requests, I've relented.

This patch now handles user created .link files too (you have to create them in some sort of text editor).

1) If you wish your shortcut to enter a directory, make sure the entry has a trailing / on it. If it doesn't, the plugin will "select" the directory in it's parent, rather than entering it.

2) If your user created .link file has only one entry, the plugin will take you straight there rather than displaying the menu.

3) I've disallowed deleting entries in user created .link files. At least for the time being.

shortcuts.patch (23.5 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  500 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   55 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 10 files changed, 600 insertions(+), 6 deletions(-)

Comment by Gerritt Gonzales (GRaTT) - Wednesday, 01 August 2007, 22:45 GMT+2

It still does not work for directories for me.
I have to remove the trailing slash (/) and then it will
go to the directory and highlight it not enter it.
As is it will display ""Directory no longer exists on disk"

The single link does not work for me either.
Thanks for relenting.
GRaTT

Comment by Gerritt Gonzales (GRaTT) - Wednesday, 01 August 2007, 23:09 GMT+2

Some additional info.
If I run the plugin from the plugins menu
and select my fixed link tailing slash removed,
or a file link, the plugin just drops me in the
directory and does not highlight the link file/dir.
If run from the link file it highlights the file/dir.
GRaTT

Comment by Bryan Childs (GodEater) - Thursday, 02 August 2007, 12:23 GMT+2

Fixed a few issues I found with handling user defined link files.

@GRaTT:

Could you attach some of your .link files for me to look at please ?

shortcuts.patch (23.6 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  500 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   55 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 10 files changed, 600 insertions(+), 6 deletions(-)

Comment by Gerritt Gonzales (GRaTT) - Thursday, 02 August 2007, 16:36 GMT+2

Here is a fresh made link file. Made by the plugin.
I have not yet updated to the last patch, and
I do have some other patches in my build.
If there is a trailing slash in the link file
an error as above "no longer exists"
GRaTT

shortcuts.link (0.1 KiB)

Comment by Bryan Childs (GodEater) - Thursday, 02 August 2007, 16:46 GMT+2

Please update to the latest patch version and let me know if that sorts the problem out first.
Thanks for the file.

Comment by Bryan Childs (GodEater) - Thursday, 02 August 2007, 16:49 GMT+2

GRaTT, do none of these entries work?

I have no idea if the <microSD1> entries will work or not as they're "fake" file system entries, but the first two look like they should work to me.
---
/MUSIC/
/RECORD/VOICE/
/<microSD1>/GRaTT/slideshow/
/<microSD1>/movies/movies/
/<microSD1>/music/Shakira/Laundry Service/Eyes Like Yours (Ojos Asi).mp3
---

Comment by Gerritt Gonzales (GRaTT) - Thursday, 02 August 2007, 17:09 GMT+2

GodEater
Only the one that points to a file works.
The others will work if I remove the trailing slash,
but only enter the dir above and highlight the link dir.
The <microSD1> "fake" file system entries act the same way so they do "work".
It did work originally for me so it may be one of the other patches.
I just svn uped and will try with just this patch.
GRaTT

Comment by Gerritt Gonzales (GRaTT) - Thursday, 02 August 2007, 17:40 GMT+2

Just tried and all the same symtoms as earlier posts.
Including different behavior between running the link file
and the plugin. Also single line links go to the list.
info: sansa E260 with 2GB SD card
OF Version ...18A
RB ver. r14147M-070802
shorcuts ver. Aug 02 10:23
The version I think worked. The comment was "Bit more reorganisation.
I will verify this in the next few days.
GRaTT

Comment by Bryan Childs (GodEater) - Thursday, 02 August 2007, 20:12 GMT+2

That's peculiar because I haven't touched the actual "change place in the file tree" code since the second revision.

Comment by Bryan Childs (GodEater) - Friday, 03 August 2007, 10:48 GMT+2

Re-organised the code a bit to get rid of a lot of the if/else nesting. Hopefully this makes it a bit easier to follow.

I've also removed the shortcuts.h file and put its content into shortcuts.c - linuxstb suggested this since the shortcuts code is currently unreferenced by any other part of the RB code - and so the .h file isn't really required.

Have so far not done anything about his suggestion to move away from a linked list to a static buffer. That's just too much re-writing for now!

shortcuts.patch (22.4 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  540 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 9 files changed, 585 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 03 August 2007, 12:08 GMT+2

Helps if you remove the #include too...

shortcuts.patch (22.4 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  539 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 9 files changed, 584 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 03 August 2007, 12:57 GMT+2

This should fix the issue with non-dircache targets (Sansa) thinking files/directories are missing when they're not!

shortcuts.patch (22.6 KiB)

 lang/english.lang      |   14 +
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  550 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 9 files changed, 595 insertions(+), 6 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 03 August 2007, 13:08 GMT+2

sync'd with svn.

shortcuts.patch (20.1 KiB)

 lang/english.lang      |    1 
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  405 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   54 ++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 -
 tree.h                 |    2 
 10 files changed, 490 insertions(+), 7 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 03 August 2007, 17:31 GMT+2

Sync'd again.

shortcuts.patch (24.4 KiB)

 lang/english.lang      |    1 
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  550 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   54 ++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 10 files changed, 635 insertions(+), 7 deletions(-)

Comment by Bryan Childs (GodEater) - Friday, 03 August 2007, 17:36 GMT+2

Oops - shortcuts.h resurrected itself!

shortcuts.patch (22.6 KiB)

 lang/english.lang      |    1 
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  550 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 9 files changed, 581 insertions(+), 7 deletions(-)

Comment by Gerritt Gonzales (GRaTT) - Friday, 03 August 2007, 18:58 GMT+2

The latest version works for me as expected with directories.
The above discrepancies between running the link file
or running the plugin and the file/dir getting selected/highlighted still apply.
The single link still does not work.
GRaTT

Comment by Gerritt Gonzales (GRaTT) - Saturday, 04 August 2007, 22:29 GMT+2

My error, single links works as expected,
only on user defined link files.
GRaTT

Comment by Bryan Childs (GodEater) - Saturday, 04 August 2007, 22:44 GMT+2

Correct - that's as designed. Only user created links allow the "single entry" feature - since they have to be manually edited.

The "shortcuts.link" file however can't work like that, or the user is unable to delete the entry from it.

Comment by Gerritt Gonzales (GRaTT) - Sunday, 05 August 2007, 10:42 GMT+2

last bit of info:
I only get the non highlight behavior when run from the plugins menu.
I tried running the shorcuts.rock file from the directory view and
both the directory without a / and the file were highlighted as expected.
GRaTT

Comment by Bryan Childs (GodEater) - Sunday, 05 August 2007, 11:14 GMT+2

Launching the plugin from "directory" view (i.e. the file browser) and the plugin's menu is the same thing. The plugins menu is simply the file browser with a filter applied to it so you can only see .rock files.

Comment by Bryan Childs (GodEater) - Sunday, 05 August 2007, 11:28 GMT+2

Latest version sync'd against svn.

shortcuts.patch (23.5 KiB)

 lang/english.lang      |    4 
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    4 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  500 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/shortcuts.h    |   55 +++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 10 files changed, 587 insertions(+), 7 deletions(-)

Comment by Bryan Childs (GodEater) - Sunday, 05 August 2007, 11:37 GMT+2

Sync'd again

shortcuts.patch (0 KiB)

 0 files changed

Comment by Bryan Childs (GodEater) - Sunday, 05 August 2007, 11:39 GMT+2

Note to self - make sure you've commited to git before doing the diff, and also check the size of your patch file!

shortcuts.patch (22.6 KiB)

 lang/english.lang      |    1 
 onplay.c               |   16 +
 plugin.c               |    2 
 plugin.h               |    6 
 plugins/SOURCES        |    1 
 plugins/shortcuts.c    |  550 +++++++++++++++++++++++++++++++++++++++++++++++++
 plugins/viewers.config |    1 
 tree.c                 |    9 
 tree.h                 |    2 
 9 files changed, 581 insertions(+), 7 deletions(-)

	Tasks related to this task (1)
	~~FS#2513 - Support for Shortcut Files~~

Duplicate tasks of this task (0)

Rockbox

FS#7486 - Favourites plugin.

Details

Loading...