Review Request 129145: UI access to libavfilter's lut effect

alcinos alcinos french.ebook.lover at gmail.com
Mon Oct 10 21:55:37 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129145/
-----------------------------------------------------------

Review request for Kdenlive.


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 left as a text field for example)
 * It is proably time to define a standard way to standardize 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/effects/avfilter/CMakeLists.txt 2260572 
  src/effectslist/initeffects.cpp e6dd1c7 
  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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdenlive/attachments/20161010/b7fd258d/attachment.html>


More information about the kdenlive mailing list