Review Request 129270: Moving DragValue class and create base type for parameter widgets

[alcinos alcinos]

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

(Updated oct. 31, 2016, 2:41 après-midi)


Review request for Kdenlive.


Changes
-------

Remove deprecated geometry widget (dead code)


Repository: kdenlive


Description
-------

This is a first step towards a refactoring of the effects stack. Currently the different widgets do not share a base class and their creation based on the xml is a bit cluttered, with a lot of redundancies. The plan is to unify this by creating a proper factory method.

I'll split the work into small and hopefully easy to review bits so that we make sure that no feature is lost on the way.


Diffs (updated)
-----

  src/CMakeLists.txt 2814fa3 
  src/effectstack/CMakeLists.txt 80ae0d6 
  src/effectstack/geometryval.h 82d7e71 
  src/effectstack/geometryval.cpp 3e0436c 
  src/effectstack/parametercontainer.h 3b1b1b3 
  src/effectstack/parametercontainer.cpp ccec798 
  src/effectstack/widgets/abstractparamwidget.h PRE-CREATION 
  src/effectstack/widgets/boolparamwidget.h PRE-CREATION 
  src/effectstack/widgets/boolparamwidget.cpp PRE-CREATION 
  src/effectstack/widgets/doubleparameterwidget.h 7c81a2e 
  src/effectstack/widgets/doubleparameterwidget.cpp bf7f245 
  src/effectstack/widgets/draggablelabel.h PRE-CREATION 
  src/effectstack/widgets/draggablelabel.cpp PRE-CREATION 
  src/effectstack/widgets/listparamwidget.h PRE-CREATION 
  src/effectstack/widgets/listparamwidget.cpp PRE-CREATION 
  src/ui/boolval_ui.ui 5f22525 
  src/ui/geometryval_ui.ui 1705840 
  src/ui/listval_ui.ui 7bc1925 

Diff: https://git.reviewboard.kde.org/r/129270/diff/


Testing
-------

Tested with a mock project


Thanks,

alcinos alcinos

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