Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/rockbox.org/flyspray/adodb/adodb.inc.php on line 845 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.user.php on line 111 FS#4874 : menu for plugins so u can fiddle with play status without exiting plugin

Rockbox

Tasklist

FS#4874 - menu for plugins so u can fiddle with play status without exiting plugin

Attached to Project: Rockbox
Opened by Jonathan Gordon (jdgordon) - Tuesday, 21 March 2006, 13:53 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

adds a menu that plugins can call which will let u change tracks and other audio settings without exiting the plugin
(application/octet-stream)    plugin_audioaccess.patch (8.5 KiB)
This task depends upon

Closed by  Linus Nielsen Feltzing (linusnielsen)
Sunday, 26 March 2006, 22:33 GMT
Reason for closing:  Accepted
Comment by Jonathan Gordon (jdgordon) - Sunday, 26 March 2006, 10:35 GMT
ok, updated... its now in the plugin lib..
i guess ill have to update this when localisation works in plugins...

all u need to do is call playback_control(rb) from inside the plugin and it will do the rest :)


(application/octet-stream)    playback_control.diff (6.7 KiB)
Comment by Linus Nielsen Feltzing (linusnielsen) - Sunday, 26 March 2006, 10:44 GMT
I like it!

Nitpick: "Pause / Play PLayback", isn't the "PLayback" kind of redundant?

This is CVS material if you ask me.
Comment by Jonathan Gordon (jdgordon) - Sunday, 26 March 2006, 11:45 GMT
ha, yes, it is.. its also bad shift-key management... just remove the word and commit :D

i know its easy to add more functionality once its in.. but can u tihnk of any other functions that should be added?
Comment by Linus Nielsen Feltzing (linusnielsen) - Sunday, 26 March 2006, 11:50 GMT
I can imagine a WPS-style control of next/prev using left/right, but not much more than that. I think it's enough for now.
Comment by Jonathan Gordon (jdgordon) - Sunday, 26 March 2006, 11:54 GMT
in the origional patch i had the view playlist screen which i still recon should go in.. but i dont know if u want to add that to the plugin api or not...

Loading...