pringuin / dataprivacybundle

Dataprivacy Bundle for Pimcore
Other
11 stars 4 forks source link

added youtube, vimeo and dailymotion #3

Closed breakone closed 3 years ago

breakone commented 3 years ago

added the options to activate youtube, vimeo and dailymotion services

Cruiser13 commented 3 years ago

Hi, thank you. These options will likely require changes to the way the pimcore_video element does work or is embedded. I think we should eighter include an editable for that or document it in the readme. What do you think?

breakone commented 3 years ago

Hi Lennart,

the pimcore video editable provides all the necessary information. I implemented the things for my video brick in the template like this:

…
{% set autoplay = autoplay.isChecked() ? true : false %}
{% set loop = loop.isChecked() ? true : false %}
{% set mute = muted.isChecked ? true: false %}

{% set video = pimcore_video('video') %}

{% if video.getVideoType == 'youtube' %}
  <div class="pimcore_tag_video pimcore_editable_video ">
    <div class="youtube_player" videoID="{{ video.id }}" width="{{ width }}" height="auto" theme="dark" rel="0" controls="1" showinfo="0" autoplay="{{ autoplay }}" mute="{{ mute }}" srcdoc="srcdoc" loop="{{ loop }}" loading="0"></div>
  </div>
{% elseif video.getVideoType == 'vimeo' %}
  <div class="pimcore_tag_video pimcore_editable_video ">
    <div class="vimeo_player" videoID="{{ video.id }}" width="{{ width }}" height="auto" autoplay="{{ autoplay }}" mute="{{ mute }}" loop="{{ loop }}"></div>
  </div>
{% elseif video.getVideoType == 'dailymotion' %}
  <div class="pimcore_tag_video pimcore_editable_video ">
    <div class="dailymotion_player" videoID="{{ video.id }}" width="{{ width }}" height="auto" showinfo="0" autoplay="{{ autoplay }}" embedType="video"></div>
  </div>
{% else %}
  {{ video|raw }}
{% endif %}
…

What do you think?

Cruiser13 commented 3 years ago

Thank you @breakone !