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

[Vincent Pinon]

> On Oct. 31, 2016, 6:39 p.m., Jean-Baptiste Mardelle wrote:
> > Thanks, had a quick look and seems ok, definitely a step in the right direction. 
> > Just to make sure you are aware of it, Kdenlive 16.12 feature freeze is on the 17th of november, after that only bug fixes should get in (A 16.12 branch will be created and master will be open for new developments).
> > I can merge your changes now if they are safe, but if you plan more imnportant changes that could introduce regressions, might be better to wait until the 17th when we can commit to master without consequences for the 16.12 release.
> > Also I think I will ask the sysadmins to give you commit access if you want, it will make it easier for you to work.

It is always possible (and recommended) to create a new development branch to store the WIP aside from master until the task gets completed ; on KDE infra it can also be done in a personal clone...

And yes I would support the request for a KDE dev account for you.
Note: KDE doesn't allow pseudo's as committers, you will have to declare your real name...


- Vincent


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


On Oct. 31, 2016, 5:30 p.m., alcinos alcinos wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129270/
> -----------------------------------------------------------
> 
> (Updated Oct. 31, 2016, 5:30 p.m.)
> 
> 
> Review request for Kdenlive.
> 
> 
> 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
> -----
> 
>   src/CMakeLists.txt 2814fa3 
>   src/bin/generators/generators.cpp 03f1479 
>   src/effectstack/CMakeLists.txt 80ae0d6 
>   src/effectstack/collapsiblegroup.h d0c0f00 
>   src/effectstack/effectstackedit.h 4d51f9b 
>   src/effectstack/geometryval.h 82d7e71 
>   src/effectstack/geometryval.cpp 3e0436c 
>   src/effectstack/keyframeedit.h f63f697 
>   src/effectstack/keyframeedit.cpp 5739ea0 
>   src/effectstack/parametercontainer.h 3b1b1b3 
>   src/effectstack/parametercontainer.cpp ccec798 
>   src/effectstack/widgets/abstractparamwidget.h PRE-CREATION 
>   src/effectstack/widgets/animationwidget.h 4e13503 
>   src/effectstack/widgets/animationwidget.cpp cc86e26 
>   src/effectstack/widgets/boolparamwidget.h PRE-CREATION 
>   src/effectstack/widgets/boolparamwidget.cpp PRE-CREATION 
>   src/effectstack/widgets/cornerswidget.h 221bb3d 
>   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/geometrywidget.h 681e5cb 
>   src/effectstack/widgets/geometrywidget.cpp 9201e3b 
>   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/20161101/59a9b369/attachment.html>