Support » Plugin: Complianz - GDPR/CCPA Cookie Consent » Youtube player changes in size
Youtube player changes in size
-
Hi guys,
In Complianz, I finally found a cookie plugin that works! You guys are the best.
the Wizard/turtorial was very helpful as well.There is one thing that confuses me, on activating the plugin it changes the size of my embedded youtube players. Is there any way to fix this? Because while it funtions correctly (blocking the video until consent is given) it doesn’t look as pretty.
Thanks in advance for your reply.
-
@bonaldi good point! 🙂
fixed, will push soon.
This should be
cmplz_dailymotion_image_
I also noticed the use of
thumbnail_large_url
which is deprecated, it should be replaced bythumbnail_1080_url
or at leastthumbnail_720_url
case 'dailymotion': if (preg_match('/dailymotion\.com\/(embed\/video)\/([^_]+)[^#]*\?|dailymotion\.com\/(embed\/video|video|hub)\/([^_]+)[^#]*(#video=([^_&]+))?|(dai\.ly\/([^_]+))!/i', $src, $matches)) { if (isset($matches[6])) { $daily_motion_id = $matches[6]; }elseif (isset($matches[4])) { $daily_motion_id = $matches[4]; }else{ $daily_motion_id = $matches[2]; } $new_src = get_transient("cmplz_dailymotion_image_$daily_motion_id"); if (!$new_src) { $thumbnail_1080_url='https://api.dailymotion.com/video/'.$daily_motion_id.'?fields=thumbnail_1080_url'; //pass thumbnail_60_url (60px height), thumbnail_120_url (120px height), thumbnail_180_url (180px height), thumbnail_240_url (240px height), thumbnail_360_url (360px height), thumbnail_480_url (480px height), thumbnail_720_url (720px height), thumbnail_1080_url (1080px height), for different sizes $json_thumbnail = file_get_contents($thumbnail_1080_url); $arr_dailymotion = json_decode($json_thumbnail, TRUE); $new_src = $arr_dailymotion['thumbnail_1080_url']; $new_src = cmplz_download_to_site($new_src, $type.$daily_motion_id); set_transient("cmplz_dailymotion_image_$daily_motion_id", $new_src, WEEK_IN_SECONDS); } } break;
For DailyMotion height problem : in class-cookie-blocker.php line 162,
replace :
$video_class = (strpos($iframe_src, 'youtube')!==false || strpos($iframe_src, 'vimeo')!==false) ? 'cmplz-video' : '';
with :
$video_class = (strpos($iframe_src, 'youtube')!==false || strpos($iframe_src, 'vimeo')!==false || strpos($iframe_src, 'dailymotion')!==false) ? 'cmplz-video' : '';
That was already done, I removed it earlier on purpose because the aspect ratio of the placeholder didn’t match the video. But it’s on git already together with the other edits.
Hi Rogier,
In function.php, this comment could be updated because its terms are deprecated:
//pass thumbnail_large_url, thumbnail_medium_url, thumbnail_small_url for different sizes
Similarly, the name of the variable
$thumbnail_large_url
could be replaced to avoid confusion.These are just a few suggestions to help, but of course, I would not want you to feel challenged in your choices.
Regards.
Hi Rogier,
I have identified a new problem: when cookies are accepted, the height value of the div is not correct.
It should no longer be calculated based on the thumbnail, since it is no longer in src.
The height should be inherited in css.Regards.
There is a risk in this kind of adjustments. If the placeholder has a very different size, the div will get a completely different aspect ratio than the movie, which seems to happen with the dailymotion placeholder. I’m not sure this is the best way to solve this yet, but I’ll leave it in if there are no obvious issues on our testing environments.
I have adjusted the script to set the container div to ‘inherit’ after enabling cookies.
Let me know if this works for you.
Hi Rogier,
It doesn’t work.
After enabling cookies,
cmplz-blocked-content-container cmplz-video
still has the height injected by thesetBlockedContentContainerAspectRatio
function.This injected height should be removed if cookies are accepted, and the standard
.cmplz-blocked-content-container
css (inherit) height should be used.You can test it in my home page video header.
Maybe a problem targeting the right parent folder?Regards.
When I check your site, and accept cookies, I see the following in the source:
<div class="cmplz-blocked-content-container cmplz-video" style="height: inherit;">
And the container has taken the height of the video. Possibly the file is still cached in your browser.
Strange.
Of course, I emptied my cache and I have this:
<div class="cmplz-blocked-content-container cmplz-video" style="height: 713.25px;">
I will investigate more.
I’m coming back towards you.So when we click to accept cookies, we have:
<div class="cmplz-blocked-content-container cmplz-video" style="height: inherit;">
.If we reload the page, we come back to:
<div class="cmplz-blocked-content-container cmplz-video" style="height: 713.25px;">
Ah, I now see the problem. Because of your FitVid implementation, an extra div is inserted. The height is reset on the parent of the video. I have now changed this to be adjusted directly on the cmplz-video class.
Yes, Fitvids always inserts an extra div.
I tried with the latest version of Github, but nothing seems to change.
Is it up to date?I think I didn’t merge it yet, it was still in the pull request.
The new source should contain these lines:
https://github.com/rlankhorst/complianz-gdpr/blob/400d9a0c9d16cf9b6a465c25c66c553e0b2488a8/core/assets/js/cookieconfig.js#L105-L107
- The topic ‘Youtube player changes in size’ is closed to new replies.