Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add playsInline attribute on the video element #7364

Closed
wants to merge 1 commit into from
Closed

Add playsInline attribute on the video element #7364

wants to merge 1 commit into from

Conversation

itsdonnix
Copy link

Fixes #7357

@IvanSanchez
Copy link
Member

Just checking - has this been tested on an actual iOS device?

@itsdonnix
Copy link
Author

Not yet... but maybe @Malvoz can help to test it...

@Malvoz
Copy link
Member

Malvoz commented Dec 4, 2020

@donnisnoni I don't own an iOS device (I just made the observation and reported on the potential requirement), sorry.

@IvanSanchez
Copy link
Member

Well, even though the code looks good (thanks @donnisnoni for sending the PR), I'd rather not merge until it has been tested (and I don't have a iOS device either)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Video overlay: playsinline attribute required for phones on iOS
4 participants