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

Critical Commons video embed full screen & Firefox issue #43

Closed
swelland opened this issue May 9, 2018 · 5 comments
Closed

Critical Commons video embed full screen & Firefox issue #43

swelland opened this issue May 9, 2018 · 5 comments

Comments

@swelland
Copy link

swelland commented May 9, 2018

I remember that we explored this a while ago, but I just noticed that the full screen button on the Critical Commons video embed isn't working correctly (e.g. The Goddess clip). It pops out and then back to the item page. The video doesn't show at all in Safari.

Here's the message the Critical Commons developer sent back then, but I've made sure the embed code matches his workaround suggestion and it still doesn't work. Any thoughts of what else to try?


Hi, All-

Here is the response from my developer (in Athens), explaining a workaround. Thanks for your patience!

-Steve

When you visit a video and select the share option, you get an 'embed' code to paste on your site, eg

<iframe src='http://www.criticalcommons.org/Members/ccManager/clips/DarkKnightMontage.mov/embed_view' frameborder='0' width='630' height='460' allowfullscreen></iframe>

If you visit any video now you get the code that correctly uses fullscreen. Unfortunately this wasn't true until the last week, so all video embeds did not contain the "allowfullscreen" tag, thus they didn't resize to fullscreen on the site they have been embedded. We cannot fix this for all the sites that embed videos from criticalcommons.org, because this is actually code stored on their sites (eg on the HTML, database etc). We can point them to correct them, by adding the "allowfullscreen" tag on the embed code, but we can't do anything else. Of course if one gets the code to embed a video now, this will allow resize (the code is generated dynamically on criticalcommons.org).

So all we can suggest to them is if they can edit the embed videos and add the allowfullscreen on the end of the embed code, just as the above video. Hope this makes sense!

@zploskey
Copy link
Owner

zploskey commented May 9, 2018

You'll need to not be logged in to the admin site and on the http (not https) version of the site for the video to show up in any browser. There may be an additional workaround required for Safari/Webkit browsers, but I can't easily test it myself since I don't have a mac. Safari has some quirks when it comes to support for fullscreen with nested elements like this.

A couple of references on this issue:
https://caniuse.com/#search=fullscreen
https://developer.mozilla.org/en-US/docs/Web/API/Fullscreen_API

@swelland
Copy link
Author

swelland commented May 9, 2018

You are right about it working in Safari in the http version of the site. The fullscreen now works in Safari but not Firefox.

@zploskey
Copy link
Owner

zploskey commented May 9, 2018

I'm having a hard time understanding what is causing this, but I am pretty confident that the problem is not on our end. This simple html document that just embeds the iframe shows the same behavior of momentarily going full screen then popping back out in Firefox.

<!doctype html>
<html>
  <head>
    <meta charset="utf-8">
    <title>CC iframe test</title>
  </head>
  <body>
    <iframe
      src="http://www.criticalcommons.org/Members/swelland/clips/the-goddess-exploited-woman-city-of-exploitation-1/embed_view"
      width="630"
      height="460"
      frameborder="0"
      allowfullscreen>
    </iframe>
  </body>
</html>

@zploskey zploskey changed the title Critical Commons video embed full screen & Safari issue Critical Commons video embed full screen & Firefox issue May 9, 2018
@zploskey
Copy link
Owner

zploskey commented Jun 7, 2018

This seems to be the same issue: mediaelement/mediaelement#1484

Critical Commons seems to be using 2.14.2 which is an older version. I suspect this may be fixed in a newer version of mediaelement.

@zploskey
Copy link
Owner

As this is a CC problem, I advise taking it up with them.

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

No branches or pull requests

2 participants