Skip to content

Conversation

spaceninja
Copy link
Member

Overview

This PR addresses a bug where iframe video players like YouTube or
Vimeo used in an Embed object would offset to the left when playing
in Safari, resulting in the left side of the player being cut off.

Testing

  1. On the deploy environment, visit the Embed > Video story in Safari and play the video. The player should not shift to the right.

This PR addresses a bug where iframe video players like YouTube or
Vimeo used in an Embed object would offset to the left when playing
in Safari, resulting in the left side of the player being cut off.

Fixes #828
@spaceninja spaceninja requested a review from tylersticka July 15, 2022 22:26
@changeset-bot
Copy link

changeset-bot bot commented Jul 15, 2022

🦋 Changeset detected

Latest commit: 516eb02

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cloudfour/patterns Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link

netlify bot commented Jul 15, 2022

Deploy Preview for cloudfour-patterns ready!

Name Link
🔨 Latest commit 516eb02
🔍 Latest deploy log https://app.netlify.com/sites/cloudfour-patterns/deploys/62d1e9fe0f3af600089ddfab
😎 Deploy Preview https://deploy-preview-1957--cloudfour-patterns.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@spaceninja spaceninja merged commit 1c7f142 into v-next Jul 15, 2022
@spaceninja spaceninja deleted the bugfix/safari-embed-clipping branch July 15, 2022 22:31
@github-actions github-actions bot mentioned this pull request Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants