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

Moving files from MePo theme Titan skin with FanartHandler support #311

Conversation

KJ-Cox
Copy link
Contributor

@KJ-Cox KJ-Cox commented May 14, 2024

Moving files from MePo theme Titan skin with FanartHandler support

@KJ-Cox KJ-Cox requested a review from andrewjswan as a code owner May 14, 2024 13:10
@github-actions github-actions bot added this to the 1.34 Pre Release milestone May 14, 2024
@andrewjswan andrewjswan changed the base branch from MP1.34_Pre.Release_Test3 to MP1.34_Pre.Release_Test5 May 14, 2024 13:25
@andrewjswan andrewjswan self-assigned this May 14, 2024
<posY>266</posY>
<width>48</width>
<height>48</height>
<texture>lock.png</texture>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did we remove this picture?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was copied from MovingPictures and looks better.

@@ -6,7 +6,6 @@
<define>#header.label:604</define>
<controls>

<import>common.default.background.xml</import>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we need background here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably, I just copied this over from MePo but did not check it. I personally have not made any changes to this as I don't use the plugin.

@andrewjswan andrewjswan merged commit 7283f43 into MediaPortal:MP1.34_Pre.Release_Test5 May 15, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants