Posted by levavie on February 20, 2009 at 2:20pm
| Project: | Video Filter |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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.
| Attachment | Size |
|---|---|
| video_filter.codecs.inc_.patch | 670 bytes |
Comments
#1
@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
#2
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.
#3
@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'?
#4
@scottrigby - Yeah, something like that would be great.
#5
@blackdog how's this?
#6
Committed. Thanks!
#7
Sweet!
#8
Automatically closed -- issue fixed for 2 weeks with no activity.