Thanks for the documentation, just some comments and questions to
clarify some stuff:

REGISTERING PLUGINS
It might be better to describe this from a plugin developers point of
view.
You normally make a call to the addPageLinks function from your plugin
to register it like this:

Slim::Web::Pages->addPageLinks("plugin", { $name => $url });

To support the iPeng skin you will need to make two calls to this
method:

Slim::Web::Pages->addPageLinks("plugin", { $name => $url });
Slim::Web::Pages->addPageLinks("pluginiPeng", { $name => $url });

The first row will make your plugin show up as usual in the standard
web interfaces, the second row will make your plugin also show up in
the iPeng skin. As already described in the documentation, you can use
the same idea when registering plugins with addPageLinks under the
"browse" and "search" categories.


QUESTIONS/COMMENTS
1.
Is there a reason why contentanchor is a separate item ? Wouldn't it be
possible to let it be a part of the contentitem if the anchor parameter
has been specified, since this is how it works in the other skins ?

2.
I'm not sure I like the textlink as an in-parameter to contentitem, the
reason being that I don't think it could be solved like this for the
rest of the skins. The reason the rest of the skins can't work this way
is that they can contain several links inside the "content" of the
wrapper.
I think I would have prefered to have some sort of link component which
I could call with something like:
[% PROCESS rowtextlink textlink="/…/…html?…#…" content="xxx" %]
So the code would be like this:

Code:
--------------------
    
  [%- WRAPPER contentitem rightcontrols='controls' %]
  [%PROCESS rowtextlink textlink="/…/…html?…#…" content="xxx" %]
  [% END %]
  
--------------------

The rowtextlink component would then also make sense in the other skins
and be added in a future version of SC.


-- 
erland

Erland Isaksson
'My homepage' (http://erland.homeip.net) 'My download page'
(http://erland.homeip.net/download)
(Developer of 'TrackStat, SQLPlayList, DynamicPlayList, Custom Browse,
Custom Scan,  Custom Skip, Multi Library and Database Query plugins'
(http://wiki.erland.homeip.net/index.php/Category:SlimServer))
------------------------------------------------------------------------
erland's Profile: http://forums.slimdevices.com/member.php?userid=3124
View this thread: http://forums.slimdevices.com/showthread.php?t=41697

_______________________________________________
plugins mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/plugins

Reply via email to