> 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. > > Vincent Pinon wrote: > 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...
Thanks for the info guys. For now I work in a local branch on my computer, so that I can write a decent history (merging directly this diff would not be so great because we would incorporate too much changes, making reverting or bissecting painfull). For the name, it is not a big deal, I can rewrite history locally. As for the feature freeze, I'm aware of it, thanks JB. I hope to be done with it by then, but anyway I'm being carefull not break any features (except deprecated one, eg displayMessage in my last commit). After that I plan to fire a bunch of VMs with popular distros, to fix the visual discrepancies that we have accross them. For the KDE dev account that'd be great, let me know if I need to do anything. - alcinos ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129270/#review100447 ----------------------------------------------------------- On Nov. 4, 2016, 11:53 a.m., alcinos alcinos wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129270/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2016, 11:53 a.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/choosecolorwidget.h 736bdf7 > src/effectstack/widgets/choosecolorwidget.cpp 546d0be > src/effectstack/widgets/colorpickerwidget.h edee5e2 > src/effectstack/widgets/colorpickerwidget.cpp f567c20 > 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/mltcontroller/clippropertiescontroller.cpp 1e0a340 > 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 > >