----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129145/ -----------------------------------------------------------
(Updated Oct. 14, 2016, 10:52 a.m.) Status ------ This change has been marked as submitted. Review request for Kdenlive. Changes ------- Submitted with commit 55f3848ceba2d80a43694bb2ca7522e315d2fb01 by Jean-Baptiste Mardelle to branch master. Repository: kdenlive Description ------- This simply exposes access to the LUT effect of libavfilter. This is for initial review only and probably shouldn't be merged right away for several reasons: * There is a bug in melt regarding this effect that needs to be fixed first (affects users with european locale). See https://github.com/mltframework/mlt/issues/130 * The UI is not optimally pretty because I cannot get the URL Picker to behave nicely (it takes all space possible instead of justifying to the right as a text field for example) * It is proably time to define a standard way to define the extension filters of the urlpicker. Currently the xml object corresponding to an url has no way to define this (and I had to temporarily disable filtering altogether in the code) * I've set a "full" description of the filter but I don't see where it is showing up in the current UI. Diffs ----- data/blacklisted_effects.txt eaa8303 data/effects/README 5fff9ef data/effects/avfilter/CMakeLists.txt 2260572 data/effects/avfilter/avfilter_lut3d.xml PRE-CREATION data/kdenliveeffectscategory.rc c645229 src/effectstack/parametercontainer.cpp 9136892 src/ui/urlval_ui.ui 68901f4 Diff: https://git.reviewboard.kde.org/r/129145/diff/ Testing ------- Works on latest master with a US locale. Thanks, alcinos alcinos
