Kodi Community Forum
GlobalSearch - tiny changes - 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)
+--- Thread: GlobalSearch - tiny changes (/showthread.php?tid=126094)

Pages: 1 2


GlobalSearch - tiny changes - ronie - 2012-03-20

i've just pushed a new version to the addon repo.

if your skin includes script-globalsearch-contextmenu.xml, please note that a small update is in order.

i've added 1 extra button to the context menu (id 1003)
and changed the script code for position the dialog (so double-check if it still pops up where you want it to be)


RE: GlobalSearch - tiny changes - BigNoid - 2012-03-20

Although I don't agree with handling positions of the dialog in the script, thx for the heads up Smile


RE: GlobalSearch - tiny changes - ronie - 2012-03-20

(2012-03-20, 22:31)Big_Noid Wrote: Although I don't agree with handling positions of the dialog in the script, thx for the heads up Smile

it was handling them before, but much less now :-)

the script has to do some calculations as the height of the dialog varies
depending on the numbers of buttons available on the context menu.


RE: GlobalSearch - tiny changes - Hitcher - 2012-03-20

Revenge is sweet. Wink

(2012-03-19, 02:35)ronie Wrote: grumble.. i suppose i could... and for about 6 other scripts as well...grumble

;-)



RE: GlobalSearch - tiny changes - jmarshall - 2012-03-20

Heh


RE: GlobalSearch - tiny changes - BigNoid - 2012-03-20

Yeah for height I knew and understand, but posx calculation is new as far as i can tell.


RE: GlobalSearch - tiny changes - ronie - 2012-03-20

heh, if you want to get to the bottom of this ;-)
the initial dialog code was a straight copy of nuka's dialog for the amt script.
it had some hardcoded posx calculations, which i've removed now:
https://code.google.com/p/ronie/source/diff?spec=svn356&r=338&format=side&path=/trunk/script.globalsearch/resources/lib/contextmenu.py&old_path=/trunk/script.globalsearch/resources/lib/contextmenu.py&old=126


RE: GlobalSearch - tiny changes - Hitcher - 2012-03-20

Is it me or are the mirrors down for everyone?


RE: GlobalSearch - tiny changes - ronie - 2012-03-20

(2012-03-20, 23:21)Hitcher Wrote: Is it me or are the mirrors down for everyone?

most likely a side-effect of the Artwork Downloader update.
people are standing in line to get their hands on that one.


RE: GlobalSearch - tiny changes - Martijn - 2012-03-20

(2012-03-20, 23:24)ronie Wrote:
(2012-03-20, 23:21)Hitcher Wrote: Is it me or are the mirrors down for everyone?

most likely a side-effect of the Artwork Downloader update.
people are standing in line to get their hands on that one.

neah who wants that...

Side note:
ronie had like 7 script updated at the same time....
on the other hand the http://fanart.tv site is also having issues Tongue


RE: GlobalSearch - tiny changes - ronie - 2012-03-20

(2012-03-20, 23:28)Martijn Wrote: Side note: ronie had like 7 script update....

yeah, let's rub it in... grumble


RE: GlobalSearch - tiny changes - Hitcher - 2012-03-21

"With great power comes great responsibility"


RE: GlobalSearch - tiny changes - Eddage - 2012-03-23

I've been trying to skin Global Search at the moment but have run into a few issues (figured I'd post in here)...

  • the media flags don't seem to work for episodes, they appear fine for films though?
  • I can't get to the seasons. When I click find all seasons it gets stuck on the "searching tv shows" message and nothing shows up
  • Some results don't seem to be in order (e.g. 28 Days Later and Green Lantern appear after results like "Terminator")

I have looked at this in other skins/default addon skin and the same issue seems to be present in all of them. I also rolled back to the previous version and that seemed to solve the second two, but not the media flags.


Was also wondering if it was possible (and not too time consuming) to make a few additions...

Image

That's what the episode info screen currently looks like(you can see the missing flags), I was wondering if there was a way to get the season/tv show thumb to display and if writer, studio and mpaa rating could be added? Also if we could get a total episodes and unwatched episodes label for tv shows?

I understand if it would take too long to change things!


RE: GlobalSearch - tiny changes - ronie - 2012-03-23

(2012-03-23, 00:15)Eddage Wrote:
  • the media flags don't seem to work for episodes, they appear fine for films though?
  • I can't get to the seasons. When I click find all seasons it gets stuck on the "searching tv shows" message and nothing shows up
  • Some results don't seem to be in order (e.g. 28 Days Later and Green Lantern appear after results like "Terminator")
- mediaflags work ok for me. if your episodes are in .iso or .vob format, xbmc won't extract the media metadata iirc.
check if they show up when you browse your library.
- cheers, will fix that asap :-)
- grrrr...more grumbles, looks like sorting by title is broken in json-rpc. can't go back to sorting by label, at least not for episodes..
will have to implement a workaround in the script then...

(2012-03-23, 00:15)Eddage Wrote: That's what the episode info screen currently looks like(you can see the missing flags), I was wondering if there was a way to get the season/tv show thumb to display and if writer, studio and mpaa rating could be added? Also if we could get a total episodes and unwatched episodes label for tv shows?

'fraid not:
- writer/watched/unwatched: not implemented in json-rpc
- studio/mpaa/thumb: only available at tv show level (it would require a lot of additional db queries to fetch those)


RE: GlobalSearch - tiny changes - Eddage - 2012-03-23

(2012-03-23, 02:37)ronie Wrote:
(2012-03-23, 00:15)Eddage Wrote:
  • the media flags don't seem to work for episodes, they appear fine for films though?
  • I can't get to the seasons. When I click find all seasons it gets stuck on the "searching tv shows" message and nothing shows up
  • Some results don't seem to be in order (e.g. 28 Days Later and Green Lantern appear after results like "Terminator")
- mediaflags work ok for me. if your episodes are in .iso or .vob format, xbmc won't extract the media metadata iirc.
check if they show up when you browse your library.
- cheers, will fix that asap :-)
- grrrr...more grumbles, looks like sorting by title is broken in json-rpc. can't go back to sorting by label, at least not for episodes..
will have to implement a workaround in the script then...

(2012-03-23, 00:15)Eddage Wrote: That's what the episode info screen currently looks like(you can see the missing flags), I was wondering if there was a way to get the season/tv show thumb to display and if writer, studio and mpaa rating could be added? Also if we could get a total episodes and unwatched episodes label for tv shows?

'fraid not:
- writer/watched/unwatched: not implemented in json-rpc
- studio/mpaa/thumb: only available at tv show level (it would require a lot of additional db queries to fetch those)

Media Flags seem to be working now, think it was because I refreshed all the info for my TV Shows and it hadn't extracted the information yet!

With regards to the show thumb I currently use...

<texture diffuse="breeze/home/folder tile dif.png">$INFO[ListItem.path,,../season00.tbn]</texture>

for dialogvidinfo for episodes, is there a way to do something similar? I tried the same method but I think the problem is that path in the addon shows the filename as well? I'm still not entirely sure how to use the commas and periods in conjunction with the path, I've just cobbled it together by looking at other skins!