Skip to content
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

[addons] label2 cleanup #9156

Merged
merged 3 commits into from
Feb 19, 2016
Merged

[addons] label2 cleanup #9156

merged 3 commits into from
Feb 19, 2016

Conversation

tamland
Copy link
Member

@tamland tamland commented Feb 18, 2016

Fixes the label2 values for addon listings. It should now use the label mask from sorting as we do everywhere else.

@tamland tamland added Type: Fix non-breaking change which fixes an issue Type: Improvement non-breaking change which improves existing functionality labels Feb 18, 2016

return result;
}

void CGUIWindowAddonBrowser::SetItemLabel2(CFileItemPtr item)
void CGUIWindowAddonBrowser::UpdateStaus(const CFileItemPtr& item)

This comment was marked as spam.

@tamland
Copy link
Member Author

tamland commented Feb 18, 2016

thank @razzeee! jenkins build this please

tamland added a commit that referenced this pull request Feb 19, 2016
@tamland tamland merged commit fc9e97e into xbmc:master Feb 19, 2016
@Hitcher
Copy link
Contributor

Hitcher commented Feb 27, 2016

@tamland this breaks the 'Downloading nn%' label2.

@tamland
Copy link
Member Author

tamland commented Feb 28, 2016

@HitcherUK 'Breaks' how and where..?

@Hitcher
Copy link
Contributor

Hitcher commented Feb 28, 2016

Add on browser when installing an add on. Label2 should show 'Downloading nn%' but it doesn't after this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Add-ons Type: Fix non-breaking change which fixes an issue Type: Improvement non-breaking change which improves existing functionality v17 Krypton
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants