-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skin.SelectBool() - allow skins to set header label #9317
Conversation
@@ -112,14 +112,18 @@ static int SelectBool(const std::vector<std::string>& params) | |||
|
|||
CGUIDialogSelect* pDlgSelect = (CGUIDialogSelect*)g_windowManager.GetWindow(WINDOW_DIALOG_SELECT); | |||
pDlgSelect->Reset(); | |||
pDlgSelect->SetHeading(CVariant{g_localizeStrings.Get(424)}); | |||
int iHeader = atoi(params[0].c_str()); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
b27d9bb
to
bc13e1b
Compare
updated as suggested by @xhaggi jenkins build this please |
did you missed to adjust the required param count at https://github.com/xbmc/xbmc/blob/master/xbmc/interfaces/builtins/SkinBuiltins.cpp#L584 |
nope, it's there: |
sorry for the noise I'm on my phone :) |
Skin.SelectBool() - allow skins to set header label
Forgot to say I tested it yesterday and it was all good. Thanks. |
allow skins to define the header label instead of hardcoding it in core:
Skin.SelectBool(headerlabel, setting1label|setting1name, setting2label|setting2name...)
also added a sanity check on the setting label/name pairs (prevent kodi from crashing)
as requested by @HitcherUK
@xhaggi for review