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

Problem with Essential theme #1

Open
oertel opened this issue Nov 19, 2014 · 3 comments
Open

Problem with Essential theme #1

oertel opened this issue Nov 19, 2014 · 3 comments

Comments

@oertel
Copy link

oertel commented Nov 19, 2014

Hi Justin,

the widely used Essential theme for Moodle uses in the current version an option to enable/disable FitVids - default it is enabled.

The Essential description:
Enable FitVids (fitvidsjs.com) to make your embedded videos responsive. If FitVids is on and you want a video to be excluded then add 'class="fitvidsignore"' to the 'iframe' tag in the HTML mode of the editor. For example: '<iframe class="fitvidsignore" width="420" height="315" src="http://www.youtube.com/embed/enmEmym85xc" frameborder="0" allowfullscreen=""></iframe>'.

If FitVids is enabled, there are some problems with the PoodLL recorder - see old bug report. The recorder has an unusual size and overlaps sometimes important Moodle elements (e.g. if you grading PoodLL submissions). The old bug report describes a solution for that:

If embedded flash objects still have the issue (say as a part of a label) then add the 'fitvidsignore ' class to the container (use HTML mode in the editor) -> https://github.com/davatron5000/FitVids.js.

I could disable FitVids in the theme settings, but it provides responive design functionality for other video players. So it would be easier if you add the class="fitvidsignore" to the surrounding flash container in the filter code.

Thanks, Stefan

@justinhunt
Copy link
Owner

I added fitvidsignore a few days ago. I tried to test it myself, but couldn't really tell if it was working or not. Could you pull the latest changes and tell me if it has fixed anything?

I added the fitvidsignore class to the table cell that all the flash widgets are loaded in, and to the iframe that they are loaded in, in some places. There probably is no real need to load the widgets in a single cell table, it was something I did a long time ago and never changed. So if it is necessary to use a div we can do that.

@oertel
Copy link
Author

oertel commented Nov 24, 2014

Hi Justin, thanks a lot.

I have checked with Moodle 2.7.3+ (Build: 20141120) and filter_poodll 2.7.3(Build 2014111601).

The situation is the same as before:

bild1

If I disable the option "Use FitVids" in the general settings of Essential everything is fine, but the responsive style of embedded videos is gone:

bild3

@justinhunt
Copy link
Owner

OK. Let me try the same test and see what I can come up with.

Thanks

On Mon, Nov 24, 2014 at 5:41 PM, oertel [email protected] wrote:

Hi Justin, thanks a lot.

I have checked with Moodle 2.7.3+ (Build: 20141120) and filter_poodll
2.7.3(Build 2014111601 <2014111601>)
.

The situation is the same as before:

[image: bild1]
https://cloud.githubusercontent.com/assets/6482225/5162191/dcdc6ac6-73bb-11e4-9964-8449d0730d2d.JPG

If I disable the option "Use FitVids" in the general settings of
Essential everything is fine, but the responsive style of embedded
videos is gone:

[image: bild3]
https://cloud.githubusercontent.com/assets/6482225/5162317/04c4949e-73be-11e4-8759-03f8b9ccac4a.JPG


Reply to this email directly or view it on GitHub
#1 (comment)
.

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