feat: skip credits

[cd16b]

Support for jumoog's Intro Skipper:

• FindroidSegment
• Skip Intros
• Skip Credits

Keep using the old API for intros, and still support ConfusedPolarBear's Intro Skipper.

fix #620

[Gauvino]

Does this could support https://github.com/endrl/jellyfin-plugin-media-analyzer ?

[cd16b]

Does this could support https://github.com/endrl/jellyfin-plugin-media-analyzer?

This PR only allows skipping credits from Jumoog's Intro Skipper v0.1.8.0, a new release of ConfusedPolarBear's Intro Skipper.
The plugin "endrl/jellyfin-plugin-media-analyzer" needs the segments API, but it is not currently supported by Jellyfin version 10.8.xx. Therefore, we will have to wait at least for Jellyfin 10.9.0.

For more information, please refer to:

• Replace ConfusedPolarBear's intro-skipper with jellyfin-plugin-media-analyzer #590

[jumoog]

I only forked the plugin for myself because ConfusedPolarBear quit. I also left everything as it is so that you only have to change the repo url and everything continues to work.

[cd16b]

Seems silly to submit an entire PR for this, but you want the line above to check the hideSkipPromptAt value - 1. Thought it might be better to suggest throwing it in with yours.

Ok, I've added the - 1, but sometimes the button is still visible for some time due to this:

findroid/player/video/src/main/java/dev/jdtech/jellyfin/viewmodels/PlayerActivityViewModel.kt

Line 260 in 0a1582f

handler.postDelayed(this, 1000L)

[AbandonedCart]

That's a good alternative.

[jarnedemeulemeester]

I started looking at this, but have yet to review all of the changes.
I've changed it to load the segments at the beginning of each episode instead of all at once.

Other than that I have a question about the FindroidSegment. Why does it have a property skip? It seems like it is always set to true?

[cd16b]

The skip property can be removed. I thought of using it in case some segments should not be skipped, but it can also be done using the segment type

[jarnedemeulemeester]

Made some more changes:

• Restyled the buttons to the original outlined style with semi transparent background
• Changed the Continue watching button to a X icon button
• Also display the X button for intros
• Keep the buttons visible when the player controls are visible (I see no reason to hide them)

[jarnedemeulemeester]

More changes:

• Introduce enum FindroidSegmentType to specify the type instead of using a string
• Use a map to retrieve the segments from the API and simplify the conversion to FindroidSegments
• Save each segment in a row and it's properties in columns instead of using a single json

[jarnedemeulemeester]

Made some final changes.
If everything looks good to you I think we can merge this 🙂

[cd16b]

Made some more changes:

• Keep the buttons visible when the player controls are visible (I see no reason to hide them)

For these, I copied the implementation of the main streaming apps which have different behaviour for intro and credits:

• Intro button is visible for n seconds, after that it is visible only if the player controls are visible
• Credits button is always visible unless the player controls are visible, so you don't have more buttons for the same function Skip credits and Next, but using this logic means overwriting IntroSkipper logic (maybe we can use the same logic as Intro or add a preference switch)

Everything else looks fine 🙂👍

[jarnedemeulemeester]

Let's follow your Intro logic. The button will be visible for n seconds, after that it's only visible when the player controls are shown. Let's do this for both the Intro and Credits.

I will be implementing that change now.

Btw I made an other pretty big change. I changed the Skip credits button to actually only skip the credits and not move on to the next episode. This is because sometimes there are extra scenes after the credits which a viewer would want to watch. I got this idea from Crunchyroll (I don't have access to other streaming services), and it makes sense in my opinion. Skip intro skips the intro and Skip credits skips the credits.
It also makes the code a little less complex 🙂