Hi, thanks for feedback. Few comments from my side
$ESC moves parsing string logic to the last instance that has nothing to do with coding - UI design. I don't know what are the reasons why XBMC is parsing strings inside $INFO and not taking it verbatim, I don't know why comma is being used as parameter separator inside $INFO. The way I see it is that this all should be handled before reaching skin. I have no idea what's going in background and I don't really want to know. I would like to _design_ if possible leaving all non related things to programmers as I have no control and knowledge about them. I'm following commits through RSS ATM and yes it's far from ideal but bigger problem is that even if I see them I may interpreting them in a wrong way. For instance: https://github.com/xbmc/xbmc/pull/124 - I have read as not concerning me as I don't use skin.properties in that way. Let's be honest - there is almost a zero chance that any skinner would think as: hey I'm trying to solve recently added layout on home screen, but wait.. I've seen commit about $ESC some months ago. Hmm.. This I better use because some character could decide to use comma in his path and since that's probably being hidden in this <onclick>PlayMedia("$INFO[Window.Property(LatestMovie.5.Path)]")</onclick> I'll better use them. Last but not least I would like to be sure when skin related part ends and XBMC code begins for trouble shooting purposes. It is already very messy and confusing for user as they are often sent from skin thread to script and/or xbmc code threads and back again in order to find where their problems are coming from. Peca On 28. 2. 2012, at 20:52, ronie wrote: On 28-02-12 11:27, Predrag Mitić wrote: > Use $ESCINFO[] rather than $INFO[] > > When was this changed? Consider me dumb, but except for recently added > example I have no idea where else should it be used?? I just don't get it. > > Do skins really need to think about this kind of issues? > > Why don't we post such things to forum? It is very hard to follow this list > as info gets mixed with code. hi, $ESCINFO[] was added about 10 months ago: https://github.com/xbmc/xbmc/commit/7d77070d7e14aa7d1a225e705c9c28f8755a48f7 some info on why it was added can be found on the PR: https://github.com/xbmc/xbmc/pull/124 the usecase is not only passing an infolabel to playmedia, basically it should be used when you pass an infolabel that may contain either quotes or commas, to a built-in function. i stumbled upon this issue last year when i was trying to copy an infolabel to a windowproperty. for example: SetProperty(MoviePlot,$INFO[ListItem.Plot]) since the movie description is likely to contain a comma, the built-in function will fail in this case as it will interpret it as multiple arguments being passed to it. it's likely this info never hit the forum, as i either only used trac to report the issue or discussed it with some of the devs on irc... can't remember :-) currently the best way to stay up-to-date with skin engine changes is to keep an eye on the commits on git or check the timeline on trac. i understand many skinners consider this to be far from ideal, so i'm planning to start a thread in the skin-dev forum after eden, where we can collect all skin related changes for frodo. that way, everyone should have an easy overview of what's new and which things have changed. cheers, ronie ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ _______________________________________________ Xbmc-addons mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/xbmc-addons
