diff --git a/src/apps/stable/features/playback/utils/mediaSegmentSettings.ts b/src/apps/stable/features/playback/utils/mediaSegmentSettings.ts index e190a60f95..45a3b83581 100644 --- a/src/apps/stable/features/playback/utils/mediaSegmentSettings.ts +++ b/src/apps/stable/features/playback/utils/mediaSegmentSettings.ts @@ -5,10 +5,16 @@ import { UserSettings } from 'scripts/settings/userSettings'; import { MediaSegmentAction } from '../constants/mediaSegmentAction'; const PREFIX = 'segmentTypeAction'; +const DEFAULT_ACTIONS: Partial> = { + [MediaSegmentType.Intro]: MediaSegmentAction.AskToSkip, + [MediaSegmentType.Outro]: MediaSegmentAction.AskToSkip +}; export const getId = (type: MediaSegmentType) => `${PREFIX}__${type}`; -export function getMediaSegmentAction(userSettings: UserSettings, type: MediaSegmentType): MediaSegmentAction | undefined { +export function getMediaSegmentAction(userSettings: UserSettings, type: MediaSegmentType): MediaSegmentAction { const action = userSettings.get(getId(type), false); - return action ? action as MediaSegmentAction : undefined; + const defaultAction = DEFAULT_ACTIONS[type] || MediaSegmentAction.None; + + return action ? action as MediaSegmentAction : defaultAction; } diff --git a/src/components/playbackSettings/playbackSettings.js b/src/components/playbackSettings/playbackSettings.js index 05374a78d5..81e82d0415 100644 --- a/src/components/playbackSettings/playbackSettings.js +++ b/src/components/playbackSettings/playbackSettings.js @@ -65,7 +65,7 @@ function populateMediaSegments(container, userSettings) { ].map(segmentType => { const segmentTypeLabel = globalize.translate('LabelMediaSegmentsType', globalize.translate(`MediaSegmentType.${segmentType}`)); const id = getId(segmentType); - selectedValues[id] = getMediaSegmentAction(userSettings, segmentType) || MediaSegmentAction.None; + selectedValues[id] = getMediaSegmentAction(userSettings, segmentType); return `