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

[skin.copacetic] 1.1.25 #433

Closed
wants to merge 5 commits into from

Conversation

realcopacetic
Copy link

Description

1.1.25

1.1.24

  • Disabled debugging flag

1.1.23

  • Fixed alignment bug and missing scrollbar on smartplaylisteditor window that could cause list of rules to exceed the sie of the panel broken playlist editor  realcopacetic/skin.copacetic#129
  • Added star rating widget to videoosd controls allowing user to view/set a userrating from videosd screen
  • Added language and subtitle quick toggles to videoosd screen

1.1.22

1.1.21

  • Added a fallback label for when clearlogo is not present in home screen background slideshow Add media title in home screen realcopacetic/skin.copacetic#120
  • Refined now playing expressions to make for better matching of currently displayed tv show/ season list item to videoplayer tvshowdibid. This will avoid two shows with the same name both showing as playing now when an episode from one is played Frasier Vs Frasier now playing  realcopacetic/skin.copacetic#100
  • Added studio flag for Mubi New studio flags realcopacetic/skin.copacetic#119
  • Added extra condition preventing windowopen animation on Home screen background artwork when returning to the homescreen immediately after ending playback of a video (Window.Previous(fullscreenvideo) + !Player.HasVideo). The reason for this is that, for an unknown reason, the window open animation was not triggering in this particular instance, causing the homescreen background not to load until navigating to another window and back to home.
  • Cleaned up some naming conventions between script.copacetic.helper and skin.copacetic

1.1.20

1.1.18-19

1.1.17

  • Fixed bug causing dialogaddoninformation to lose focus on opening for some addons
  • Added fade effect to infoscreen thumbnail when shutdownmenu visible on top
  • Added optional shutdown menu shortcut to hidden views menu The shutdown menu is difficult to access with a CEC TV remote realcopacetic/skin.copacetic#90
  • Fixed error in in skinshortcuts management window - non-existent control 309
  • Moved custom playlist background slideshow from homescreen to global via script.copacetic.helper 1.0.9
  • Rearranged skin settings to consolidate all background settings in one screen
  • When global background slideshow is showing or live tv is playing, removed fade in/out transitions between home, addonbrowser, settings screens, tvguide and tv/radio search for a smoother experience. This required removing Ken Burns effect from these background slideshows.
  • Removed some of the unnecessary transitions between different views when navigating through addonbrowser menus
  • Fixed multiart bug breaking artwork for billboard view in programs
  • Fixed typo breaking fix for previous issue Multiart not working on billboard home widgets realcopacetic/skin.copacetic#60
  • Fixed colour of volume indicator on home screen widgets when light theme selected
  • Added disabled indicator to addonbrowser for disabled addons in List and Grid views, plus installed indicator for installed addons when browsing repos
  • Redesigned script-upnext widget and added options to skinsettings to enable/disable addon and choose art type. NEXT UP  realcopacetic/skin.copacetic#102

Checklist:

  • My code follows the add-on rules and piracy stance of this project.
  • I have read the CONTRIBUTING document
  • Each add-on submission should be a single commit with using the following style: [plugin.video.foo] v1.0.0

Additional information :

  • Submitting your add-on to this specific branch makes it available to any Kodi version equal or higher than the branch name with the applicable Kodi dependencies limits.
  • add-on development wiki page.
  • Kodi pydocs provide information about the Python API
  • PEP8 codingstyle which is considered best practice but not mandatory.
  • This add-on repository has automated code guideline check which could help you improve your coding. You can find the results of these check at Codacy. You can create your own account as well to continuously monitor your python coding before submitting to repo.
  • Development questions can be asked in the add-on development section on the Kodi forum.

@realcopacetic
Copy link
Author

I notice it only seems to be my skin submissions that are failing this check @enen92 @ronie but I confess I have no idea as to why. Can you let me know when you have a moment please so I can fix it and resubmit, thanks

@Hitcher
Copy link

Hitcher commented Apr 4, 2024

It's not the naming convention is it?

[skin.copacetic 1.1.25] should be [skin.copacetic] 1.1.25 according to this.

@Hitcher Hitcher closed this Apr 4, 2024
@Hitcher
Copy link

Hitcher commented Apr 4, 2024

Sorry, didn't mean to close.

@Hitcher Hitcher reopened this Apr 4, 2024
@realcopacetic
Copy link
Author

Thanks @Hitcher - I didn't notice this before. Should I close this PR and open a new one?

@Hitcher
Copy link

Hitcher commented Apr 4, 2024

No idea as I was just checking the differences between your PR and the others. 😉

@realcopacetic realcopacetic changed the title [skin.copacetic 1.1.25] [skin.copacetic] 1.1.25 Apr 4, 2024
@Hitcher
Copy link

Hitcher commented Apr 4, 2024

Run it again with debugging enabled but I don't have a clue as to what they mean.

https://github.com/xbmc/repo-skins/actions/runs/8555723728?pr=433

@ronie
Copy link
Member

ronie commented Apr 4, 2024

there seems to be an issue with the encoding of these two filenames:
"skin.copacetic/media/flags/studios/AXN\302\240Espa\303\261a.png"
"skin.copacetic/media/flags/studios/AXN\302\240Polska.png"

there should be a space between AXN and España, but it uses some other character.
some applies to AXN Polska.png

@realcopacetic realcopacetic deleted the skin.copacetic-1.1.25 branch April 5, 2024 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants