Release skin helper service - Printable Version +- Kodi Community Forum (https://forum.kodi.tv) +-- Forum: Development (https://forum.kodi.tv/forumdisplay.php?fid=32) +--- Forum: Skinning (https://forum.kodi.tv/forumdisplay.php?fid=12) +---- Forum: Skin helper addons (https://forum.kodi.tv/forumdisplay.php?fid=300) +---- Thread: Release skin helper service (/showthread.php?tid=235676) Pages:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
|
RE: skin helper service - guido1138 - 2015-10-05 (2015-10-05, 16:20)marcelveldt Wrote: I don't think it is related to that as that code is perfectly fine ;-). Must be a typo in the skin code.Well, I admit I had to stare at it for a while too, but then you notice a missing ")" at the end: Code: <visible>Container.Content(episodes) + RE: skin helper service - User 224999 - 2015-10-05 (2015-10-05, 17:32)guido1138 Wrote: Well, I admit I had to stare at it for a while too, but then you notice a missing ")" at the end: oops! haha :-) Now fixed, thanks! RE: skin helper service - tomer953 - 2015-10-05 Did you changed something for the nextepisodes widget? It's showing the same tvshow.. Multiple episodes for the same one.. check it out in your system plz RE: skin helper service - Mr. V - 2015-10-06 Hi Marcel, I having been compiling a list of the type and target property assigned to each widget. This is so I can better understand how I should set my visibility condition to limit artwork choices for each widget. A few things I noticed; Favourite Episodes - target="unknown" Favourite Media - type="unknown" My TV Shows Airing Today - type="movies" Upcoming Movies (Rotten Tomatoes) - type="unknown" Upcoming Movies (TMDB) - type="unknown" Opening Movies (Rotten Tomatoes) - type="unknown" User-Created Lists (TMDB) - type="unknown" In Cinema Movies (Rotten Tomatoes) - type="unknown" Also I do have a query related to two of the widget property's I am looking at. We have widget="addon, source, library, playlists ect" Then type="movies, episodes, albums ect" As I understand these property's are used to help set visibility conditions in the skin. For the type its quite self explanatory and suits the purpose. For the widget, I do find it difficult to make use of it in visibility conditions. For example the library data provider is an addon. The widget property is set to 'addon'. So within the widget list there are around 15 selectable widgets using library data providor. The addon library data provider is getting its information from the library. So to me it would make more sense if the widget property was set to 'library' as that is where the data comes from for the widget. I could then use a visibility condition such as; Code: <property property="widgetArt" condition="StringCompare(Container(211).ListItem.Property(widget),library) + StringCompare(Container(211).ListItem.Property(widgetType),movies)">ClearArt</property> In its current state I need to do something like this; Code: <property property="widgetArt" condition="StringCompare(Container(211).ListItem.Property(widgetType),movies) + !SubString(Container(211).ListItem.Property(widgetPath),extendedinfo)">ClearArt</property> What I am suggesting is it possible to set the widget property to where the information is taken from. I have compiled a list of every property that each widget sets. If its any help I can post a link to my excel worksheet including my suggested changes. RE: skin helper service - im85288 - 2015-10-06 (2015-10-05, 16:21)marcelveldt Wrote:(2015-10-04, 15:49)im85288 Wrote: [quote='marcelveldt' pid='2121932' dateline='1443833249'] Just normally by having the cast in a list and pressing select on them...I will try again tonight and see if I can debug a bit further. Thanks RE: skin helper service - User 224999 - 2015-10-06 (2015-10-06, 06:36)Mr. V Wrote: For the widget, I do find it difficult to make use of it in visibility conditions. For example the library data provider is an addon and that is what is set for the widget property. So within the widget list there are around 15 selectable widgets using this addon. The addon library data provider is getting its information from the library. So to me it would make more sense if the widget property was set to library as that is where the data comes from for the widget. Thanks for your list, I will update the code to improve the content detection. Setting the widget type property to library is not very usefull because you will have to know what content you're skinning against, for exampe movies, episodes etc. Just setting it to library won't tell you a thing about the content because it can be anything... The whole idea for the type property is to help skinners a bit with determining the content type in their widget but you can't fully rely on it ever, there will always be addons that don't follow the normal guidelines and set the contenttype wrong or just provide something non standard. RE: skin helper service - User 224999 - 2015-10-06 (2015-10-06, 06:36)Mr. V Wrote: What I am suggesting is it possible to set the widget property to where the information is taken from. That would be possible... Like the name of the addon for example ? And library if the data was parsed from the library ? RE: skin helper service - Mr. V - 2015-10-06 (2015-10-06, 12:56)marcelveldt Wrote:(2015-10-06, 06:36)Mr. V Wrote: What I am suggesting is it possible to set the widget property to where the information is taken from. Cheers Marcel. That exactly it. In version 2 of the widgets for Nox Silvo, I premade a list with certain codes so I could get around this. I found it worked quite well. When a user selected a widget the number was written to the guisettings.xml associated with the menu item. This what I used Code: <!-- WidgetType: --> I'm sure with a few name changes for the widget property it will make thing much nicer for visibility conditions. RE: skin helper service - Jayz2K - 2015-10-06 Hi Marcel, I'm facing many infolables missing in the TV widget (Like ListItem.NextTiltle, NextStartTime, etc.) Just my 2 cent but why not just passing the target and content to skinshortcuts like : Code: <content target="">pvr://channels/tv/1/;reload=$INFO[Window(Home).Property(widgetreload)]</content> I was using this before and can of course still use it but it duplicates the widgets for TV. The vfs path above returns all the infolabels like the in MyPVRChannels.xml. This shoul also avoid you to search for fixes. Cheers RE: skin helper service - User 224999 - 2015-10-06 (2015-10-06, 13:47)Jayz2K Wrote: Hi Marcel, That's still possible, in fact it's the faster/better solution... You will not have live thumbs with that approach... RE: skin helper service - Jayz2K - 2015-10-06 (2015-10-06, 13:49)marcelveldt Wrote: That's still possible, in fact it's the faster/better solution... You will not have live thumbs with that approach... Hmm, right, didn't think about that. But can't we use Window(Home).Property(SkinHelper.PVR.xxartxx) the same way ? RE: skin helper service - Mr. V - 2015-10-06 Hi Marcel, Hopefully this might be some help. Its what I had in mind. http://1drv.ms/1FRtmsB Edit: Another thing I forgot to mention for version 2 of the widgets was when a user selected a widget an additional property was set. Code: <onclick>Skin.SetString($INFO[Container(9000).ListItem.Property(Item),,HomeItem.WidgetAllowedArt],_PP_FA_LS_CL_CA_BA_DA_)</onclick> Each Code PP,FA ect was for a particular artwork type. I then used a Substring in the visibility condition for the Artwork Type. RE: skin helper service - User 224999 - 2015-10-06 (2015-10-06, 13:58)Jayz2K Wrote:Yes, you can but only for the selected listitem...(2015-10-06, 13:49)marcelveldt Wrote: That's still possible, in fact it's the faster/better solution... You will not have live thumbs with that approach... See the readme, you must set the ID of your widgetcontainer in a window prop BTW: Are you only missing the next info labels? I can take a look if I can add them to the "enhanced" pvr widgets... RE: skin helper service - Jayz2K - 2015-10-06 (2015-10-06, 14:31)marcelveldt Wrote: BTW: Are you only missing the next info labels? I can take a look if I can add them to the "enhanced" pvr widgets... Well, yes, at this time, I only see the "Next" infolabels. Hope this won't be too much with NextStartTime / NextEndTime since time issue is now solved RE: skin helper service - Rjsachse - 2015-10-07 Are you able to usr fanart api to get pvr logos and poster and clearart and stuff. So pvr can look like movies and tv shows |