Skip Applying RemainingSecondsOfIntro for Segments at the End of the Video

IntroSkipper/Configuration/inject.js

@@ -228,10 +228,10 @@ const introSkipper = {
 			const segment = this.skipSegments[key];
 			if (
 				(position > segment.ShowSkipPromptAt &&
-					position < segment.HideSkipPromptAt - 1) ||
+					position < segment.HideSkipPromptAt) ||
 				(this.osdVisible() &&
 					position > segment.IntroStart &&
-					position < segment.IntroEnd - 1)
+					position < segment.IntroEnd - 3)
 			) {
 				return { ...segment, SegmentType: key };
 			}
@@ -250,31 +250,13 @@ const introSkipper = {
 	},
 	/** Playback position changed, check if the skip button needs to be displayed. */
 	videoPositionChanged() {
-		if (!this.skipButton) return;
+		if (!this.skipButton || !this.allowEnter) return;
-		const embyButton = this.skipButton.querySelector(".emby-button");
+		const { SegmentType: segmentType } = this.getCurrentSegment(this.videoPlayer.currentTime);
-		const segmentType = this.getCurrentSegment(
-			this.videoPlayer.currentTime,
-		).SegmentType;
 		if (
 			segmentType === "None" ||
-			this.currentOption === "Off" ||
+			this.currentOption === "Off"
-			!this.allowEnter
 		) {
-			if (this.skipButton.classList.contains("show")) {
+			this.hideSkipButton();
-				this.skipButton.classList.remove("show");
-				embyButton.addEventListener(
-					"transitionend",
-					() => {
-						this.skipButton.classList.add("hide");
-						if (this.osdVisible()) {
-							this.osdElement.querySelector("button.btnPause").focus();
-						} else {
-							embyButton.originalBlur();
-						}
-					},
-					{ once: true },
-				);
-			}
 			return;
 		}
 		if (
@@ -285,22 +267,42 @@ const introSkipper = {
 			this.doSkip();
 			return;
 		}
+        const button = this.skipButton.querySelector(".emby-button");
 		this.skipButton.querySelector("#btnSkipSegmentText").textContent =
 			this.skipButton.dataset[segmentType];
 		if (!this.skipButton.classList.contains("hide")) {
-			if (!this.osdVisible() && !embyButton.contains(document.activeElement))
+			if (!this.osdVisible() && !button.contains(document.activeElement)) {
-				embyButton.focus();
+                button.focus();
-			return;
+            }
+            return;
 		}
 		requestAnimationFrame(() => {
 			this.skipButton.classList.remove("hide");
 			requestAnimationFrame(() => {
 				this.skipButton.classList.add("show");
-				this.overrideBlur(embyButton);
+				this.overrideBlur(button);
-				embyButton.focus();
+				button.focus();
 			});
 		});
 	},
+    hideSkipButton() {
+        if (this.skipButton.classList.contains("show")) {
+            this.skipButton.classList.remove("show");
+            const button = this.skipButton.querySelector(".emby-button");
+            button.addEventListener(
+                "transitionend",
+                () => {
+                    this.skipButton.classList.add("hide");
+                    if (this.osdVisible()) {
+                        this.osdElement.querySelector("button.btnPause").focus();
+                    } else {
+                        button.originalBlur();
+                    }
+                },
+                { once: true },
+            );
+        }
+    },
 	/** Seeks to the end of the intro. */
 	doSkip() {
 		if (!this.allowEnter) return;
@@ -310,19 +312,15 @@ const introSkipper = {
 			return;
 		}
 		this.d(`Skipping ${segment.SegmentType}`);
-		this.allowEnter = false;
 		const seekedHandler = () => {
 			this.videoPlayer.removeEventListener("seeked", seekedHandler);
 			setTimeout(() => {
 				this.allowEnter = true;
-			}, 500);
+			}, 700);
 		};
 		this.videoPlayer.addEventListener("seeked", seekedHandler);
-		this.videoPlayer.currentTime =
+		this.videoPlayer.currentTime = segment.IntroEnd;
-			segment.SegmentType === "Credits" &&
+        this.hideSkipButton();
-			this.videoPlayer.duration - segment.IntroEnd < 3
-				? this.videoPlayer.duration + 10
-				: segment.IntroEnd;
 	},
 	createButton(ref, id, innerHTML, clickHandler) {
 		const button = ref.cloneNode(true);

IntroSkipper/Controllers/SkipIntroController.cs

@@ -157,18 +157,21 @@ public class SkipIntroController : ControllerBase
             var segment = new Intro(timestamp);
 
             var config = Plugin.Instance!.Configuration;
-            segment.IntroEnd -= config.RemainingSecondsOfIntro;
+            segment.IntroEnd = mode == AnalysisMode.Credits
+                ? GetAdjustedIntroEnd(id, segment.IntroEnd, config)
+                : segment.IntroEnd - config.RemainingSecondsOfIntro;
+
             if (config.PersistSkipButton)
             {
                 segment.ShowSkipPromptAt = segment.IntroStart;
-                segment.HideSkipPromptAt = segment.IntroEnd;
+                segment.HideSkipPromptAt = segment.IntroEnd - 3;
             }
             else
             {
                 segment.ShowSkipPromptAt = Math.Max(0, segment.IntroStart - config.ShowPromptAdjustment);
                 segment.HideSkipPromptAt = Math.Min(
                     segment.IntroStart + config.HidePromptAdjustment,
-                    segment.IntroEnd);
+                    segment.IntroEnd - 3);
             }
 
             return segment;
@@ -179,6 +182,14 @@ public class SkipIntroController : ControllerBase
         }
     }
 
+    private static double GetAdjustedIntroEnd(Guid id, double segmentEnd, PluginConfiguration config)
+    {
+        var runTime = TimeSpan.FromTicks(Plugin.Instance!.GetItem(id)?.RunTimeTicks ?? 0).TotalSeconds;
+        return runTime > 0 && runTime < segmentEnd + 1
+            ? runTime
+            : segmentEnd - config.RemainingSecondsOfIntro;
+    }
+
     /// <summary>
     /// Erases all previously discovered introduction timestamps.
     /// </summary>