2015-10-03, 04:35
(2015-10-03, 00:46)BobCratchett Wrote: As one of the contributors of the Skin Shortcuts script, I'm torn on this one myself. Balancing between being able to recreate any menu item that you delete, versus being able to hide an unused menu item (though if you don't have a PVR backend, most skins will hide the PVR menu's anyway, so they can just be ignored) is not easy.
If it's something anyone feels strongly about, feel free to post it as a feature request in the Skin Shortcuts thread in the Skinning forum. In terms of coding it, though, it's not an easy one as it would either require every skin using Skin Shortcuts to incorporate an additional GUI button to enable/disable any given shortcut, or would mean that you could disable a shortcut on a skin that supports it, but not be able to re-enable it if you switched to a skin that didn't support it....
I did have a thought, hopefully not a terrible one , since adding an additional required button seems like the wrong way to go...
What about adding an additional option to RunScript(script.skinshortcuts,type=resetall) to show a list of defaults?
Something like Restore All, Restore Single, Exit instead of just Yes/No. Then one could add back a deleted default menu item or restore it to it's defaults if it was modified. Perhaps make it an optional thing to enable in overrides.xml or even a new RunScript call