mherger wrote: 
> > Great. When you say it works - do you get a brief notification, and
> are
> > the contents of this as expected? i.e. does it show any response
> string
> > (if there is one)? I was hoping that if the command returned an item
> > with (e.g.) "Added album XX to your Spotify library", then Material
> > would show this in a notification. But, I have not tested the code -
> so
> > unsure if it works.
> 
> Hmm... where would I see this? I didn't see any notification. But this 
> seems to be kind of odd anyway: the response to this call would 
> basically return the menu item again, not the message. BUT on the player
> 
> I was controlling while doing this I did get a notification...

There would be a brief (3 seconds?) notification sliding down from the
main toolbar. I'm surprised you get no notification. The code attempts
to parse the response and if it finds one text item with a title, then
it should show a notification with that. If it can't find this, then it
should show a notification with the title of the clicked item (i.e. the
one that caused the action). Either way a notification should be shown.
line 589 of browse-page.js (handleTextClickResponse())

mherger wrote: 
> 
> > Great. When you say it works - do you get a brief notification, and
> are
> [color=blue]
> > I'm leaning towards if a listing has a "text" item, then only show as
> a
> > list, not a grid. In this case I think its correct anyway, as the
> > listing is a track listing - and I would not expect to see a grid for
> > tracks.
> 
> Please don't... I do prefer the grid view. And as I said nobody ever 
> complained about those items in any of the grid views I know. And it 
> shouldn't hurt you, as you don't use these services :-D.

Well it does - in that I need to add support for it in the grid template
(but that's easy enough) :-) But, OK, I'll add. Or rather, re-factor the
code so its not needed....


------------------------------------------------------------------------
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