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
[adsp] remove usage of list item entries on skins #9708
Conversation
@phil65 @roni @AchimTuran can check them. |
nice, skin changes are ok. |
725c21a
to
4d5a718
Compare
Nice, thx for goin after my request that fast. :) |
CFileItemPtr item(new CFileItem()); | ||
item->SetLabel(g_localizeStrings.Get(dsp_mode_types[i].iName)); | ||
item->SetLabel2(g_localizeStrings.Get(dsp_mode_types[i].iDescription)); | ||
item->SetProperty("currentMode", dsp_mode_types[i].sModeType); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
I guess something similiar like |
Already a pull request on confluence: xbmc/skin.confluence#9 :D If I'm not wrong, works it also if the set is still is still in xml and the values from Kodi becomes overwritten. |
jenkins build this please |
This remove the set of list entries on skin itself and do this on code side.
Further becomes with this way only the Mode type shown who is present.
Here screenshots about the changes