The following patch hides the related videos displayed on youtube films. This is the default option for emfield. But this module is better since it allows to embed videos in the middle of the post.

CommentFileSizeAuthor
#5 video_filter_378148_5.patch2.15 KBscottrigby
#1 video_filter_378148_1.patch809 bytesscottrigby
video_filter.codecs.inc_.patch670 bytesdruvision

Comments

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented
Version: 6.x-2.4 » 6.x-2.x-dev
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new video_filter_378148_1.patch809 bytes

@levavie: Nice work :) I want to mark RTBC - the patch above is against 6.x-2.4, so needs to be updated.

Rerolling against 6.x-2.x-dev

blackdog’s picture

blackdog commented
Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

I wonder if we should make this a setting? I'm not sure, but if we change this now, when people are 'used' to having related content show up, some might be surprised.

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented

@blackdog I fully agree. Do you have any preferences for where / how this should be configured? I assume this is only for youtube, just a new variable like variable_set('video_filter_related', 1) defaulting to 'Show related videos'?

blackdog’s picture

blackdog commented

@scottrigby - Yeah, something like that would be great.

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new video_filter_378148_5.patch2.15 KB

@blackdog how's this?

blackdog’s picture

blackdog commented
Status: Needs review » Fixed

Committed. Thanks!

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented

Sweet!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.