Issue #747074: Allow other modules to unset/remove default codecs implements a hook to unset/remove and an UI for it is left for another issue (never created, until now).
This issue is to request a UI for enable/disable a codec.

Files: 
CommentFileSizeAuthor
#5 video_filter-n2071113-5.patch5.18 KBDamienMcKenna

Comments

moskito’s picture

Status:Active» Needs review
StatusFileSize
new2.95 KB

I create a filter settings that list all available codecs (default codecs are set as enable by default) and a hook 'video_filter_get_codec_enabled($filter)' to list all enabled codecs for a filter.

Chris Burge’s picture

This patch tests successfully for me.

It would be useful if the instructions provided by the WYSIWYG plugin were modified to reflect enabled codecs. Instructions for all codecs are still presented to end users regardless of which ones are enabled.

moskito’s picture

In this new patch instructions provided by the WYSIWYG plugin are modified to reflect enabled codecs. As codecs are enabled per text format basis, but WYSIWYG plugin don't have text format context, the instruction shows all codecs available to the user (may not available in current text format, but for sure in one that he can use).

Chris Burge’s picture

Patch #3 works as described. Thanks for circling back around on this issue. It's too bad that WYSIWYG plugins aren't aware of the active text format. The functionality provided by #3 will probably meet the majority of use cases.

DamienMcKenna’s picture

StatusFileSize
new5.18 KB

A minor update that slightly improves the "not all codecs available" disclaimer in the popup, and tightens up the code a little.

DamienMcKenna’s picture

I was going to set this as RTBC, but then I tweaked the patch. This is good to go, adding it to the proposed issues list for 3.2.