mirror of
https://github.com/ajayyy/SponsorBlock.git
synced 2025-05-11 18:36:15 +02:00
feat: Option to auto-skip non-music on youtube music only
This commit is contained in:
parent
7904913fdb
commit
e656dd8d8b
5 changed files with 32 additions and 10 deletions
|
@ -1 +1 @@
|
|||
Subproject commit b22477a8640b407910b24d5bec13d0d6d4d3cf01
|
||||
Subproject commit 81b414988814f5735ac6fb621e95537680bdf42d
|
|
@ -1 +1 @@
|
|||
Subproject commit e9efadcf82316b358e3f81cba71b20d7ed1ead86
|
||||
Subproject commit e399706837d1415c9875c4fba8044b204923324d
|
|
@ -243,6 +243,9 @@ class CategorySkipOptionsComponent extends React.Component<CategorySkipOptionsPr
|
|||
return [{
|
||||
configKey: "autoSkipOnMusicVideos",
|
||||
label: chrome.i18n.getMessage("autoSkipOnMusicVideos"),
|
||||
}, {
|
||||
configKey: "skipNonMusicOnlyOnYoutubeMusic",
|
||||
label: chrome.i18n.getMessage("skipNonMusicOnlyOnYoutubeMusic"),
|
||||
}];
|
||||
default:
|
||||
return [];
|
||||
|
|
|
@ -57,6 +57,7 @@ interface SBConfig {
|
|||
donateClicked: number;
|
||||
autoHideInfoButton: boolean;
|
||||
autoSkipOnMusicVideos: boolean;
|
||||
skipNonMusicOnlyOnYoutubeMusic: boolean;
|
||||
colorPalette: {
|
||||
red: string;
|
||||
white: string;
|
||||
|
@ -322,6 +323,7 @@ const syncDefaults = {
|
|||
donateClicked: 0,
|
||||
autoHideInfoButton: true,
|
||||
autoSkipOnMusicVideos: false,
|
||||
skipNonMusicOnlyOnYoutubeMusic: false,
|
||||
scrollToEditTimeUpdate: false, // false means the tooltip will be shown
|
||||
categoryPillUpdate: false,
|
||||
showChapterInfoMessage: true,
|
||||
|
|
|
@ -35,7 +35,7 @@ import { ChapterVote } from "./render/ChapterVote";
|
|||
import { openWarningDialog } from "./utils/warnings";
|
||||
import { isFirefoxOrSafari, waitFor } from "../maze-utils/src";
|
||||
import { getErrorMessage, getFormattedTime } from "../maze-utils/src/formating";
|
||||
import { getChannelIDInfo, getVideo, getIsAdPlaying, getIsLivePremiere, setIsAdPlaying, checkVideoIDChange, getVideoID, getYouTubeVideoID, setupVideoModule, checkIfNewVideoID, isOnInvidious, isOnMobileYouTube, isOnYTTV, getLastNonInlineVideoID, triggerVideoIDChange, triggerVideoElementChange, getIsInline, getCurrentTime, setCurrentTime, getVideoDuration, verifyCurrentTime, waitForVideo } from "../maze-utils/src/video";
|
||||
import { getChannelIDInfo, getVideo, getIsAdPlaying, getIsLivePremiere, setIsAdPlaying, checkVideoIDChange, getVideoID, getYouTubeVideoID, setupVideoModule, checkIfNewVideoID, isOnInvidious, isOnMobileYouTube, isOnYouTubeMusic, isOnYTTV, getLastNonInlineVideoID, triggerVideoIDChange, triggerVideoElementChange, getIsInline, getCurrentTime, setCurrentTime, getVideoDuration, verifyCurrentTime, waitForVideo } from "../maze-utils/src/video";
|
||||
import { Keybind, StorageChangesObject, isSafari, keybindEquals, keybindToString } from "../maze-utils/src/config";
|
||||
import { findValidElement } from "../maze-utils/src/dom"
|
||||
import { getHash, HashedValue } from "../maze-utils/src/hash";
|
||||
|
@ -1891,16 +1891,33 @@ function createButton(baseID: string, title: string, callback: () => void, image
|
|||
}
|
||||
|
||||
function shouldAutoSkip(segment: SponsorTime): boolean {
|
||||
return (!Config.config.manualSkipOnFullVideo || !sponsorTimes?.some((s) => s.category === segment.category && s.actionType === ActionType.Full))
|
||||
&& (utils.getCategorySelection(segment.category)?.option === CategorySkipOption.AutoSkip ||
|
||||
(Config.config.autoSkipOnMusicVideos && sponsorTimes?.some((s) => s.category === "music_offtopic")
|
||||
&& segment.actionType === ActionType.Skip)
|
||||
|| sponsorTimesSubmitting.some((s) => s.segment === segment.segment));
|
||||
if (Config.config.manualSkipOnFullVideo && !sponsorTimes?.some((s) => s.category === segment.category && s.actionType === ActionType.Full)) {
|
||||
return false;
|
||||
}
|
||||
|
||||
return ( // Normal skip
|
||||
utils.getCategorySelection(segment.category)?.option === CategorySkipOption.AutoSkip
|
||||
// Forbid skipping of non-music if we are not on Youtube Music
|
||||
&& !(segment.category === "music_offtopic" && Config.config.skipNonMusicOnlyOnYoutubeMusic && !isOnYouTubeMusic())
|
||||
)
|
||||
||
|
||||
( // Skip every segment, if it's a music video
|
||||
|
||||
// Forbid autoSkipOnMusicVideos if if we are not on Youtube Music
|
||||
!(Config.config.skipNonMusicOnlyOnYoutubeMusic && !isOnYouTubeMusic())
|
||||
&& Config.config.autoSkipOnMusicVideos
|
||||
&& sponsorTimes?.some((s) => s.category === "music_offtopic")
|
||||
&& segment.actionType === ActionType.Skip
|
||||
)
|
||||
||
|
||||
sponsorTimesSubmitting.some((s) => s.segment === segment.segment);
|
||||
}
|
||||
|
||||
function shouldSkip(segment: SponsorTime): boolean {
|
||||
return (segment.actionType !== ActionType.Full
|
||||
&& segment.source !== SponsorSourceType.YouTube
|
||||
if (segment.actionType === ActionType.Full) {
|
||||
return false;
|
||||
}
|
||||
return (segment.source !== SponsorSourceType.YouTube
|
||||
&& utils.getCategorySelection(segment.category)?.option !== CategorySkipOption.ShowOverlay)
|
||||
|| (Config.config.autoSkipOnMusicVideos && sponsorTimes?.some((s) => s.category === "music_offtopic")
|
||||
&& segment.actionType === ActionType.Skip);
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue