mherger wrote: 
> > mherger wrote:
> >> Here's the item as it gets passed down to the "click()" function...
> > 
> > Can you try latest git version? I've changed the command that is sent
> > (so it specifies the range).
> 
> The issue still is that click() is being called with
> allowTextClick=false.

Try disabling the use of the grid. I'm 99.999% sure this is why you are
seeing this. In my quick hack I added the item onto a list. Only the
list section of the template is expecting to handle text items. If you
do this, does the code now work?

Now the question is. Should I allow plain text entries in a grid? That
seems a bit odd. So, I could disallow a grid when the set of items
contains a text item - this would work-around this particular issue.
But, why is this item here? This 'Add item to library' to me make more
sense as a context menu item for the album itself. Perhaps it should
even be in the 'More' menu of the album. Adding it after the list of
tracks just seems odd. If it really needs to be this way, could you
somehow flag the item as a 'parent action'? That way I could build a
list of all 'parent actions' and add them to the sub-toolbar (the one
that has the home and back icons) - as I think (at least for Material)
they'd make more sense there. I know this would be a change in the
protocol - but other clients would be free to ignore it.


------------------------------------------------------------------------
cpd73's Profile: http://forums.slimdevices.com/member.php?userid=66686
View this thread: http://forums.slimdevices.com/showthread.php?t=109624

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

Reply via email to