View Issue Details

IDProjectCategoryView StatusLast Update
0006712OXID eShop (all versions)2.3. Extensions (modules, themes)public2017-10-18 16:06
Reporterstroebjo 
PrioritylowSeverityfeatureReproducibilityalways
Status acknowledgedResolutionopen 
Product Version4.10.5 / 5.3.5 
Target VersionFixed in Version 
Summary0006712: VisualCMS Responsive Video embed
DescriptionInside the VisualCMS Text element you can add a YouTube etc. Video into your element.

The Video embed code is generated by the VisualCMS JavaScript. It puts the iframe directly inside the DOM with fixed width/height. The embedded video will not scale to adapt smaller screens.

I would suggest to add an additional div wrapper around it, so the classic "responsive video embed" fix can be used [0].

I would also suggest to put descriptive classes on this div node, like class"video-embed video-embed--{youtube|vimeo|...}".

[0]: http://www.holgerkoenemann.de/ein-vimeo-oder-youtube-video-responsive-einbinden/
TagsNo tags attached.
ThemeAll
BrowserAll
PHP VersionNot defined
MySQL VersionNot defined

Activities

QA

2017-10-18 15:23

administrator  

Widget-Text.png (21,210 bytes)
Widget-Text.png (21,210 bytes)

QA

2017-10-18 15:25

administrator  

QA

2017-10-18 15:26

administrator   ~0012245

The code for embedding a video is a template which you could change to your needs by using the source code view in the text widget and also in the CMS overview. Screenshots added.

stroebjo

2017-10-18 15:43

reporter   ~0012246

I'm well aware that I can paste any custom video embed code into the code view, nevertheless none of my clients want's to be bothered with editing HTML. If they would be willing and capable of doing that I would not need the VisualCMS.

Since the VisualCMS claims to be responsive AND provides a button to include video embeds by URL (three buttons to the left from the code view in the WYSISWYG) I think it should be added automatically by the VisualCMS editor.


Nevertheless, in my case I added a new shortcode for the VisualCMS for videos. This could be further extended to include Privacy mode etc. I attached the file.

stroebjo

2017-10-18 15:43

reporter  

rwdvideo.php (1,323 bytes)

QA

2017-10-18 16:04

administrator   ~0012247

Thank you for your hint.