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.

Sign up for GitHub

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

Jump to bottom

add ListItem.IsParentFolder infobool #8775

Merged
merged 1 commit into from
Jan 9, 2016
Merged

add ListItem.IsParentFolder infobool #8775

merged 1 commit into from
Jan 9, 2016

Conversation

ronie
Copy link
Member

@ronie ronie commented Jan 6, 2016

provide skins with an easy way to check if the focused item is the parent folder item.

currently skins have to use workarounds like:
StringCompare(ListItem.Label,..)

@razzeee
Copy link
Member

razzeee commented Jan 7, 2016

We might want to add
if (item->IsFileItem())
before that?

@ronie
Copy link
Member Author

ronie commented Jan 7, 2016

thx! updated.

@razzeee
Copy link
Member

razzeee commented Jan 9, 2016

Can you rebase?

@ronie
Copy link
Member Author

ronie commented Jan 9, 2016

rebased.

@razzeee
Copy link
Member

razzeee commented Jan 9, 2016

jenkins build and merge

@razzeee razzeee added this to the Krypton 17.0-alpha1 milestone Jan 9, 2016
@razzeee razzeee added the Type: Feature non-breaking change which adds functionality label Jan 9, 2016
jenkins4kodi added a commit that referenced this pull request Jan 9, 2016
@jenkins4kodi
Merge pull request #8775 from ronie/parentfolder
7cbd950
@jenkins4kodi jenkins4kodi merged commit 7cbd950 into xbmc:master Jan 9, 2016
@ronie ronie deleted the parentfolder branch February 26, 2016 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Component: GUI engine Type: Feature non-breaking change which adds functionality v17 Krypton
Projects
None yet
Milestone
Krypton 17.0-alpha1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ronie @razzeee @jenkins4kodi