Try disabling the use of the grid. I'm 99.999% sure this is why you are seeing this.
Aha! Confirmed: with the grid disabled, this menu item is working as expected.
Now the question is. Should I allow plain text entries in a grid? That seems a bit odd.
Should you worry about this? IMHO you shouldn't. You should display whatever menu item you get. If the service or plugin is implementing the feature the way it is implemented, then you shouldn't take decision to not show it or whatever. It's the menu provider's job to come up with a good menu.
I do agree that those items look odd in grid view. But there are certain technical limitations in what eg. mysb.com services can deliver. When I do implement a menu, then I try to make it work on all interfaces (the old line oriented UI you probably don't even know, SqueezePlay/Touch/Radio, web). Sometimes this is not sexy, but does the job. Form follows function.
So, I could disallow a grid when the set of items contains a text item - this would work-around this particular issue.
It wouldn't really solve any issue but the visual irritation caused by text menus within a grid. But it's a visual issue, not a functional, right? I've been using this with the Default skin or JiveLite's grid view for years. Never had complaints about it, really.
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.
As I said sometimes there are technical limitations. Eg. I wouldn't get the necessary information back from an API before drilling down in to the track list. And in a list of hundreds of albums you simply can't reasonably pre-fetch all tracks for them. Really, you shouldn't care about the why. It's not your fault, it's the menu provider's fault that the menu isn't pleasing or irritating. Please just make it work :-).
-- Michael _______________________________________________ plugins mailing list [email protected] http://lists.slimdevices.com/mailman/listinfo/plugins
