Skip to content

Commit

Permalink
Merge pull request #749 from samvera-labs/bug-fix
Browse files Browse the repository at this point in the history
Enable error close button for multiple canvas manifests
  • Loading branch information
Dananji authored Dec 6, 2024
2 parents 530797e + da7e71b commit 651a142
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/components/MediaPlayer/MediaPlayer.js
Original file line number Diff line number Diff line change
Expand Up @@ -186,9 +186,9 @@ const MediaPlayer = ({
? [sources[srcIndex]]
: sources,
errorDisplay: {
// If only one source is available and it is not playable remove
// close button for the error modal dialog
uncloseable: sources?.length > 1 ? false : true,
// Show the close button for the error modal, if more than one source OR multiple
// canvases are available
uncloseable: (sources?.length > 1 || isMultiCanvased) ? false : true,
},
} : { ...defaultOptions, sources: [] };
}, [isVideo, playerConfig, srcIndex]);
Expand Down

0 comments on commit 651a142

Please sign in to comment.