diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap index c4fe624870..09319a07d5 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap @@ -78,6 +78,7 @@ exports[`VideoSourceWidget snapshots snapshots: renders as expected with default ({ export const fallbackHooks = ({ fallbackVideos, dispatch }) => ({ addFallbackVideo: () => dispatch(actions.video.updateField({ fallbackVideos: [...fallbackVideos, ''] })), + /** * Deletes the first occurrence of the given videoUrl from the fallbackVideos list * @param {string} videoUrl - the video URL to delete */ - deleteFallbackVideo: (videoUrl) => { - const index = fallbackVideos.findIndex(video => video === videoUrl); + deleteFallbackVideo: (videoIndex) => { const updatedFallbackVideos = [ - ...fallbackVideos.slice(0, index), - ...fallbackVideos.slice(index + 1), + ...fallbackVideos.slice(0, videoIndex), + ...fallbackVideos.slice(videoIndex + 1), ]; + dispatch(actions.video.updateField({ fallbackVideos: updatedFallbackVideos })); }, }); diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/index.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/index.jsx index bac6c347cb..dd4a3899ab 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/index.jsx +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/index.jsx @@ -101,7 +101,7 @@ const VideoSourceWidget = ({ {fallbackVideos.formValue.length > 0 ? fallbackVideos.formValue.map((videoUrl, index) => ( - + deleteFallbackVideo(videoUrl)} + onClick={() => deleteFallbackVideo(index)} />