Skip to content

Commit 746d852

Browse files
committed
Improve naming
1 parent 2771bff commit 746d852

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

dotcom-rendering/src/components/SelfHostedVideo.importable.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -200,9 +200,9 @@ const doesVideoHaveAudio = (video: HTMLVideoElement): boolean =>
200200

201201
/**
202202
* The Fullscreen api is not supported by Safari mobile,
203-
* so we need to check if we have ios html video element properties we can use instead.
203+
* so we need to check if we have access to the webkit api we can use instead.
204204
* */
205-
const shouldFullscreenOnIOS = (video: HTMLVideoElement): boolean => {
205+
const shouldUseWebkitFullscreen = (video: HTMLVideoElement): boolean => {
206206
return (
207207
'webkitDisplayingFullscreen' in video &&
208208
'webkitEnterFullscreen' in video &&
@@ -635,7 +635,7 @@ export const SelfHostedVideo = ({
635635

636636
if (!video) return;
637637

638-
if (shouldFullscreenOnIOS(video)) {
638+
if (shouldUseWebkitFullscreen(video)) {
639639
/***
640640
* webkit fullscreen methods are not part of the standard HTMLVideoElement
641641
* type definition as they are iOS only.

0 commit comments

Comments
 (0)