I finally got video filter module to work on Drupal 7 with CKeditor only combination. By adding the code mentioned in the readme text file and copying the ckeditor folder from video filter module into ckeditor module. So the blue icon for video filter was showing and got added to the 'full html' toolbar, all of a sudden now clicking the icon/button does not open the dialog box with video URL and width and height settings anymore.
The video filter functionality is there and working if I type in [video:...], but not sure how to fix the window to pop up again.
Thanks.

Comments

sin@ created an issue. See original summary.

sin@’s picture

Issue summary: View changes
laso25’s picture

The problem is that the module is calling a function that is deprecated...

https://developer.mozilla.org/en-US/docs/Web/API/Window/showModalDialog

video_filter/editors/ckeditor/plugin.js

window.showModalDialog(path, { 'opener' : window, 'editorname' : editor.name }, "dialogWidth:580px; dialogHeight:480px; center:yes; resizable:yes; help:no;");

I'm trying to fix it ASAP, having to rewrite everything (the plugin.js and the video_filter_dialog.js), and then I'll send a patch.

sin@’s picture

Thanks a lot that would be very helpful.
I managed to make it finally work by removing everything and turning off the module and then start from scratch and it started working again.
Also the readme file with the module asks for adding a few lines of code to the js file but I have seen newer comments from other members that using the latest version on drupal 7 does not need those lines anymore, if that is true, it would be great if that text file can also be updated so next person who is installing it would know for sure.
Is there anyway through this module that we can embed our video without showing a link to the original host (making sure user does not go away)?

minnur’s picture

Status: Active » Postponed (maintainer needs more info)

Is this still an issue ? I fee like we can close this issue. Please advise.

minnur’s picture

Status: Postponed (maintainer needs more info) » Closed (fixed)

This seems to me no longer an issue. Closing. Re-open if you think otherwise.