2017-02-24, 21:18
(2017-02-19, 20:26)braz Wrote: Thanks Bob, no rush. I also tried label="24" but that results in Kodi's localized string with that ID. For now I'll just define the numbers I need in the skin's language files.
Hopefully will be addressed this weekend. If you get chance, have a check that the issue exists on the Leia branch - there's already a small improvement to the way strings are handled there (though my memory is it won't actually affect this issue, rather it deals with shortcut labels).
(2017-02-20, 06:41)braz Wrote:(2015-10-09, 04:09)jurialmunkey Wrote: Talking about overrides. I'm trying to get a groupoverride working for all submenus, however I can't figure out a way to do it.
Basically I have this for the main menu (id 9000) which works perfectly
Code:<groupoverride group="mainmenu" condition="true">SetProperty(FO9001,2,home)</groupoverride>
However, I can't get the same behaviour for the submenu
Code:<groupoverride group="submenu" condition="true">SetProperty(FO9001,2,home)</groupoverride>
Removing the group="submenu" or leaving it empty doesn't seem to work.
Basically, I need a way to trigger an additional action before the onclick using
condition="!IsEmpty(Container(9002).ListItem.Property(isSubmenu))"(2015-10-09, 09:24)BobCratchett Wrote: @jurialmunkey - I've just updated my open PR to make the 'group' optional for global overrides, so you can apply one to all shortcuts. Can you please test and let me know if it works. Cheers.Hi Bob, I'm also trying to add a groupoverride for my submenu items and am running into the same problem described by jurialmunkey. It looks like there was a PR back in 2015 to make the group tag optional, but I can't get any groupoverride options to work unless I specify the group. Any ideas?
Yes, it's not possible to specify 'all submenu's', rather you can either specify (if memory serves) all menu's, or menu's with a specific labelID. I've added this to my to-look at list.
(2017-02-19, 14:22)badaas Wrote: Hi Bob, I'll add a request if do-able. Add a line in settings to be able to add our own visible condition for each menu item. Would save adding all the checks you already have for built in visible conditions for some buttons already.
You should be able to follow Skin Shortcuts Docs: Overriding an Action to add an additional visibility condition. Otherwise, please explain the use case that currently isn't covered
(2017-02-20, 16:42)bsoriano Wrote: Hello Bob,
Following is a link for an updated Spanish strings.po, based on current master branch.
https://1drv.ms/u/s!AlII29kkG6TFh-QHspLlybsRVlcx8Q
Regards,
Bart
Thanks Are you able to do a PR? That would be a quicker way of getting it onto git, otherwise I'll do my best to add it over the weekend.