Search Dialog
#1
Dear Kodi Team,

I'd like to suggest one small but important improvement if I may. Estuary skin defines the search dialog, which is called from the main menu. This dialog has a hard coded section with different search options:

Code:

<content>
<item>
<label>$LOCALIZE[31113]</label>
<onclick>Dialog.Close(all)</onclick>
<onclick condition="System.AddonIsEnabled(script.globalsearch)">RunScript(script.globalsearch)</onclick>
<onclick condition="System.HasAddon(script.globalsearch) + !System.AddonIsEnabled(script.globalsearch)">EnableAddon(script.globalsearch)</onclick>
<onclick condition="!System.HasAddon(script.globalsearch)">InstallAddon(script.globalsearch)</onclick>
</item>
<item>
<label>$LOCALIZE[31145]</label>
<onclick>Dialog.Close(all)</onclick>
<onclick>ActivateWindow(addonbrowser,addons://search/,return)</onclick>
</item>
<item>
<label>$LOCALIZE[31114]</label>
<onclick>Dialog.Close(all)</onclick>
<onclick condition="System.AddonIsEnabled(plugin.video.youtube)">ActivateWindow(videos,"plugin://plugin.video.youtube/kodion/search/list/",return)</onclick>
<onclick condition="System.HasAddon(plugin.video.youtube) + !System.AddonIsEnabled(plugin.video.youtube)">EnableAddon(plugin.video.youtube)</onclick>
<onclick condition="!System.HasAddon(plugin.video.youtube)">InstallAddon(plugin.video.youtube)</onclick>
</item>
</content>

The issues with this piece:
1. It is non-extendable through add-ons other than skin addons. That is, the search options are limited only to the local library or YouTube. However, most of video addons have their own search functions. Examples are plenty. It would be nice to have this list of items configurable. Ideally, it should be an extension point in the settings.xml of an addon. This way each addon could define its ownsearch entry.
2. Having YouTube or Global Search addons should be removed from the skin default - instead the same configuration could be moved to respective section (extension point) of their settings.xml.
Reply
#2
Quote:settings.xml
Sorry, meant to say "addon.xml"
Reply
#3
Adding this to the thread tracking Skin API changes is completely inappropriate. If you have suggestions/questions please create your own thread in future.
Reply

Logout Mark Read Team Forum Stats Members Help
Search Dialog0