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

[gui] Add icon for Blu-rays. #14418

Merged
merged 6 commits into from
Sep 12, 2018
Merged

[gui] Add icon for Blu-rays. #14418

merged 6 commits into from
Sep 12, 2018

Conversation

ace20022
Copy link
Member

Rebase of #6946. It's not optimal, especially when there are multiple drives available, but I have not time to work me in that part and refactor it.

Could someone please make the icon DefaultBluray.png? Maybe @ronie?



In memory of @un1versal.



@ace20022 ace20022 added Type: Improvement non-breaking change which improves existing functionality WIP PR that is still being worked on Component: Skin v19 Matrix labels Sep 10, 2018
@MartijnKaijser
Copy link
Member

Any specific reason to not put it in v18? Does it have issues with multiple drives?

@ace20022
Copy link
Member Author

Any specific reason to not put it in v18? Does it have issues with multiple drives?

I thought it is a problem that the icon is missing. And it needs more testing, especially on non win desktop platforms.

@ronie
Copy link
Member

ronie commented Sep 10, 2018

here you go, click below:
defaultbluray

@ronie
Copy link
Member

ronie commented Sep 11, 2018

@ace20022 here's a better version:
defaultbluray

@ace20022
Copy link
Member Author

The last commit only has an effect if folder stacking is disabled. For bds I want to add a thumb from the bds meta data if possible.

@ace20022 ace20022 removed the WIP PR that is still being worked on label Sep 12, 2018
@ace20022 ace20022 added this to the Leia 18.0-beta3 milestone Sep 12, 2018
@ace20022 ace20022 merged commit a87e3ad into xbmc:master Sep 12, 2018
@ace20022 ace20022 deleted the drive_icon branch September 12, 2018 06:03
{
SetIconImage("DefaultBluray.png");
}
else if (IsDVDFolder())

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Skin Type: Improvement non-breaking change which improves existing functionality v18 Leia
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants