]> git.sesse.net Git - kdenlive/blobdiff - src/effectstack/collapsibleeffect.cpp
Some progress on effect group
[kdenlive] / src / effectstack / collapsibleeffect.cpp
index 4165e3be0bbd9b1659c64e4d4d31b73eba304db1..e533b2f911c3748821415ee886d51dcb1de75c92 100644 (file)
@@ -24,6 +24,8 @@
 #include "ui_boolval_ui.h"
 #include "ui_wipeval_ui.h"
 #include "ui_urlval_ui.h"
+#include "ui_keywordval_ui.h"
+#include "ui_fontval_ui.h"
 #include "complexparameter.h"
 #include "geometryval.h"
 #include "positionedit.h"
 #include "colortools.h"
 #include "doubleparameterwidget.h"
 #include "cornerswidget.h"
+#include "dragvalue.h"
 #include "beziercurve/beziersplinewidget.h"
 #ifdef USE_QJSON
 #include "rotoscoping/rotowidget.h"
 #endif
 
-
+#include <QInputDialog>
 #include <QDialog>
+#include <QMenu>
 #include <QVBoxLayout>
 #include <KDebug>
 #include <KGlobalSettings>
 #include <KLocale>
+#include <KMessageBox>
+#include <KStandardDirs>
 #include <KFileDialog>
 #include <KUrlRequester>
 
@@ -68,28 +74,64 @@ class Urlval: public QWidget, public Ui::Urlval_UI
 {
 };
 
+class Keywordval: public QWidget, public Ui::Keywordval_UI
+{
+};
+
+class Fontval: public QWidget, public Ui::Fontval_UI
+{
+};
+
 QMap<QString, QImage> CollapsibleEffect::iconCache;
 
+void clearLayout(QLayout *layout)
+{
+    QLayoutItem *item;
+    while((item = layout->takeAt(0))) {
+        if (item->layout()) {
+            clearLayout(item->layout());
+            delete item->layout();
+        }
+        if (item->widget()) {
+            delete item->widget();
+        }
+        delete item;
+    }
+}
+
+MySpinBox::MySpinBox(QWidget * parent):
+    QSpinBox(parent)
+{
+    setFocusPolicy(Qt::StrongFocus);
+}
+
+void MySpinBox::focusInEvent(QFocusEvent*)
+{
+     setFocusPolicy(Qt::WheelFocus);
+}
+
+void MySpinBox::focusOutEvent(QFocusEvent*)
+{
+     setFocusPolicy(Qt::StrongFocus);
+}
 
 
-CollapsibleEffect::CollapsibleEffect(QDomElement effect, ItemInfo info, int ix, EffectMetaInfo *metaInfo, bool lastEffect, QWidget * parent) :
+CollapsibleEffect::CollapsibleEffect(QDomElement effect, QDomElement original_effect, ItemInfo info, int ix, EffectMetaInfo *metaInfo, bool lastEffect, bool isGroup, QWidget * parent) :
         QWidget(parent),
         m_paramWidget(NULL),
         m_effect(effect),
+        m_original_effect(original_effect),
         m_lastEffect(lastEffect),
-        m_active(false)
+        m_isGroup(isGroup),
+        m_active(false),
+        m_index(ix)
 {
-    setFont(KGlobalSettings::toolBarFont());
-    setMouseTracking(true);
+    //setMouseTracking(true);
     setupUi(this);
-    QDomElement namenode = m_effect.firstChildElement("name");
-    if (namenode.isNull()) return;
-    QString type = m_effect.attribute("type", QString());
-    KIcon icon;
-    if (type == "audio") icon = KIcon("kdenlive-show-audio");
-    else if (m_effect.attribute("tag") == "region") icon = KIcon("kdenlive-mask-effect");
-    else if (type == "custom") icon = KIcon("kdenlive-custom-effect");
-    else icon = KIcon("kdenlive-show-video");
+    m_info.fromString(effect.attribute("kdenlive_info"));
+    frame->setBackgroundRole(QPalette::Midlight);
+    frame->setAutoFillBackground(true);
+    setFont(KGlobalSettings::smallestReadableFont());
    
     buttonUp->setIcon(KIcon("go-up"));
     buttonUp->setToolTip(i18n("Move effect up"));
@@ -99,12 +141,9 @@ CollapsibleEffect::CollapsibleEffect(QDomElement effect, ItemInfo info, int ix,
     }
     buttonDel->setIcon(KIcon("edit-delete"));
     buttonDel->setToolTip(i18n("Delete effect"));
-    buttonSave->setIcon(KIcon("document-save"));
-    buttonSave->setToolTip(i18n("Save effect"));
 
     buttonUp->setVisible(false);
     buttonDown->setVisible(false);
-    buttonSave->setVisible(false);
     buttonDel->setVisible(false);
     
     /*buttonReset->setIcon(KIcon("view-refresh"));
@@ -112,12 +151,32 @@ CollapsibleEffect::CollapsibleEffect(QDomElement effect, ItemInfo info, int ix,
     //checkAll->setToolTip(i18n("Enable/Disable all effects"));
     //buttonShowComments->setIcon(KIcon("help-about"));
     //buttonShowComments->setToolTip(i18n("Show additional information for the parameters"));
-            
-    title->setText(i18n(namenode.text().toUtf8().data()));
-    effectIcon->setPixmap(icon.pixmap(QSize(16,16)));
+    m_menu = new QMenu;
+    m_menu->addAction(KIcon("view-refresh"), i18n("Reset effect"), this, SLOT(slotResetEffect()));
+    m_menu->addAction(KIcon("document-save"), i18n("Save effect"), this, SLOT(slotSaveEffect()));
+    
+    if (!m_isGroup) {
+       QDomElement namenode = m_effect.firstChildElement("name");
+       if (namenode.isNull()) return;
+       title->setText(i18n(namenode.text().toUtf8().data()));
+       QString type = m_effect.attribute("type", QString());
+       KIcon icon;
+       if (type == "audio") icon = KIcon("kdenlive-show-audio");
+       else if (m_effect.attribute("tag") == "region") icon = KIcon("kdenlive-mask-effect");
+       else if (type == "custom") icon = KIcon("kdenlive-custom-effect");
+       else icon = KIcon("kdenlive-show-video");
+       title->setIcon(icon);
+       m_menu->addAction(KIcon("folder-new"), i18n("Create Group"), this, SLOT(slotCreateGroup()));
+       setupWidget(info, ix, metaInfo);
+    }
+    else {
+       setAcceptDrops(true);
+       title->setText(i18n("Effect Group"));
+       title->setIcon(KIcon("folder"));
+    }
+    
+    title->setMenu(m_menu);
     
-    //QLabel *lab = new QLabel("HEllo", widgetFrame);
-    //vbox->addWidget(lab);
     if (m_effect.attribute("disable") == "1") {
         enabledBox->setCheckState(Qt::Unchecked);
         title->setEnabled(false);
@@ -131,37 +190,111 @@ CollapsibleEffect::CollapsibleEffect(QDomElement effect, ItemInfo info, int ix,
     connect(buttonUp, SIGNAL(clicked()), this, SLOT(slotEffectUp()));
     connect(buttonDown, SIGNAL(clicked()), this, SLOT(slotEffectDown()));
     connect(buttonDel, SIGNAL(clicked()), this, SLOT(slotDeleteEffect()));
-    setupWidget(info, ix, metaInfo);
+
+    Q_FOREACH( QSpinBox * sp, findChildren<QSpinBox*>() ) {
+        sp->installEventFilter( this );
+        sp->setFocusPolicy( Qt::StrongFocus );
+    }
+    Q_FOREACH( KComboBox * cb, findChildren<KComboBox*>() ) {
+       cb->installEventFilter( this );
+        cb->setFocusPolicy( Qt::StrongFocus );
+    }
+    Q_FOREACH( QProgressBar * cb, findChildren<QProgressBar*>() ) {
+       cb->installEventFilter( this );
+        cb->setFocusPolicy( Qt::StrongFocus );
+    }
 }
 
 CollapsibleEffect::~CollapsibleEffect()
 {
+    if (m_paramWidget) delete m_paramWidget;
+    delete m_menu;
+}
+
+void CollapsibleEffect::slotCreateGroup()
+{
+    emit createGroup(m_paramWidget->index());
+}
+
+bool CollapsibleEffect::eventFilter( QObject * o, QEvent * e ) 
+{
+    if (e->type() == QEvent::Wheel) {
+       QWheelEvent *we = static_cast<QWheelEvent *>(e);
+       if (we->modifiers() != Qt::NoModifier) {
+           e->accept();
+           return false;
+       }
+       if (qobject_cast<QAbstractSpinBox*>(o)) {
+           if(qobject_cast<QAbstractSpinBox*>(o)->focusPolicy() == Qt::WheelFocus)
+           {
+               e->accept();
+               return false;
+           }
+           else
+           {
+               e->ignore();
+               return true;
+           }
+       }
+       if (qobject_cast<KComboBox*>(o)) {
+           if(qobject_cast<KComboBox*>(o)->focusPolicy() == Qt::WheelFocus)
+           {
+               e->accept();
+               return false;
+           }
+           else
+           {
+               e->ignore();
+               return true;
+           }
+       }
+       if (qobject_cast<QProgressBar*>(o)) {
+           if(qobject_cast<QProgressBar*>(o)->focusPolicy() == Qt::WheelFocus)
+           {
+               e->accept();
+               return false;
+           }
+           else
+           {
+               e->ignore();
+               return true;
+           }
+       }
+    }
+    return QWidget::eventFilter(o, e);
+}
+
+QDomElement CollapsibleEffect::effect() const
+{
+    return m_effect;
 }
 
 void CollapsibleEffect::setActive(bool activate)
 {
     m_active = activate;
-    title->setBackgroundRole(m_active ? QPalette::AlternateBase : QPalette::Window);
-    title->setAutoFillBackground(m_active);    
+    frame->setBackgroundRole(m_active ? QPalette::Mid : QPalette::Midlight);
+    frame->setAutoFillBackground(activate);
 }
 
 void CollapsibleEffect::mouseDoubleClickEvent ( QMouseEvent * event )
 {
-    if (title->underMouse() && collapseButton->isEnabled()) slotSwitch();
+    if (frame->underMouse() && collapseButton->isEnabled()) slotSwitch();
     QWidget::mouseDoubleClickEvent(event);
 }
 
-void CollapsibleEffect::mousePressEvent ( QMouseEvent * event )
+void CollapsibleEffect::mousePressEvent ( QMouseEvent *event )
 {
-    if (!m_active) emit activateEffect(m_paramWidget->index());
+    if (!m_active && m_paramWidget) emit activateEffect(m_paramWidget->index());
+    QWidget::mousePressEvent(event);
 }
 
 void CollapsibleEffect::enterEvent ( QEvent * event )
 {
-    if (m_paramWidget->index() > 0) buttonUp->setVisible(true);
+    if (m_paramWidget == NULL || m_paramWidget->index() > 0) buttonUp->setVisible(true);
     if (!m_lastEffect) buttonDown->setVisible(true);
-    buttonSave->setVisible(true);
     buttonDel->setVisible(true);
+    if (!m_active) frame->setBackgroundRole(QPalette::Midlight);
+    frame->setAutoFillBackground(true);
     QWidget::enterEvent(event);
 }
 
@@ -169,34 +302,82 @@ void CollapsibleEffect::leaveEvent ( QEvent * event )
 {
     buttonUp->setVisible(false);
     buttonDown->setVisible(false);
-    buttonSave->setVisible(false);
     buttonDel->setVisible(false);
+    if (!m_active) frame->setAutoFillBackground(false);
     QWidget::leaveEvent(event);
 }
 
 void CollapsibleEffect::slotEnable(bool enable)
 {
     title->setEnabled(enable);
-    m_effect.setAttribute("disable", enable ? 0 : 1);
-    if (enable || KdenliveSettings::disable_effect_parameters()) {
-        widgetFrame->setEnabled(enable);
+    if (m_isGroup) {
+       QVBoxLayout *vbox = static_cast<QVBoxLayout *>(widgetFrame->layout());
+       if (vbox == NULL) return;
+       for (int i = 0; i < vbox->count(); i++) {
+           CollapsibleEffect *e = static_cast<CollapsibleEffect *>(vbox->itemAt(i)->widget());
+           if (e) e->enabledBox->setChecked(enable);// slotEnable(enable);
+       }
+    } else {
+       m_effect.setAttribute("disable", enable ? 0 : 1);
+       if (enable || KdenliveSettings::disable_effect_parameters()) {
+           widgetFrame->setEnabled(enable);
+       }
+       emit effectStateChanged(!enable, m_paramWidget->index());
     }
-    emit effectStateChanged(!enable, m_paramWidget->index());
 }
 
 void CollapsibleEffect::slotDeleteEffect()
 {
-    emit deleteEffect(m_effect, m_paramWidget->index());
+    if (!m_isGroup) emit deleteEffect(m_effect, m_paramWidget->index());
 }
 
 void CollapsibleEffect::slotEffectUp()
 {
-    emit changeEffectPosition(m_paramWidget->index(), true);
+    if (!m_isGroup) emit changeEffectPosition(m_paramWidget->index(), true);
 }
 
 void CollapsibleEffect::slotEffectDown()
 {
-    emit changeEffectPosition(m_paramWidget->index(), false);
+    if (!m_isGroup) emit changeEffectPosition(m_paramWidget->index(), false);
+}
+
+void CollapsibleEffect::slotSaveEffect()
+{
+    QString name = QInputDialog::getText(this, i18n("Save Effect"), i18n("Name for saved effect: "));
+    if (name.isEmpty()) return;
+    QString path = KStandardDirs::locateLocal("appdata", "effects/", true);
+    path = path + name + ".xml";
+    if (QFile::exists(path)) if (KMessageBox::questionYesNo(this, i18n("File %1 already exists.\nDo you want to overwrite it?", path)) == KMessageBox::No) return;
+
+    QDomDocument doc;
+    QDomElement effect = m_effect.cloneNode().toElement();
+    doc.appendChild(doc.importNode(effect, true));
+    effect = doc.firstChild().toElement();
+    effect.removeAttribute("kdenlive_ix");
+    effect.setAttribute("id", name);
+    effect.setAttribute("type", "custom");
+    QDomElement effectname = effect.firstChildElement("name");
+    effect.removeChild(effectname);
+    effectname = doc.createElement("name");
+    QDomText nametext = doc.createTextNode(name);
+    effectname.appendChild(nametext);
+    effect.insertBefore(effectname, QDomNode());
+    QDomElement effectprops = effect.firstChildElement("properties");
+    effectprops.setAttribute("id", name);
+    effectprops.setAttribute("type", "custom");
+
+    QFile file(path);
+    if (file.open(QFile::WriteOnly | QFile::Truncate)) {
+        QTextStream out(&file);
+        out << doc.toString();
+    }
+    file.close();
+    emit reloadEffects();
+}
+
+void CollapsibleEffect::slotResetEffect()
+{
+    emit resetEffect(m_paramWidget->index());
 }
 
 void CollapsibleEffect::slotSwitch()
@@ -210,13 +391,62 @@ void CollapsibleEffect::slotShow(bool show)
     widgetFrame->setVisible(show);
     if (show) {
         collapseButton->setArrowType(Qt::DownArrow);
+       m_info.isCollapsed = false;
     }
     else {
         collapseButton->setArrowType(Qt::RightArrow);
+       m_info.isCollapsed = true;
     }
+    m_effect.setAttribute("kdenlive_info", m_info.toString());
+    emit parameterChanged(m_original_effect, m_effect, m_index);
+    
+}
+
+void CollapsibleEffect::setGroupIndex(int ix)
+{
+    m_info.groupIndex = ix;
+}
+
+void CollapsibleEffect::addGroupEffect(CollapsibleEffect *effect)
+{
+    QVBoxLayout *vbox = static_cast<QVBoxLayout *>(widgetFrame->layout());
+    if (vbox == NULL) {
+       vbox = new QVBoxLayout();
+       vbox->setContentsMargins(10, 0, 0, 0);
+       vbox->setSpacing(2);
+       widgetFrame->setLayout(vbox);
+    }
+    effect->setGroupIndex(groupIndex());
+    vbox->addWidget(effect);
+}
+
+int CollapsibleEffect::index() const
+{
+    return m_index;
+}
+
+int CollapsibleEffect::groupIndex() const
+{
+    if (m_isGroup) return m_index;
+    return -1;
+}
 
+int CollapsibleEffect::effectIndex() const
+{
+    if (m_effect.isNull()) return -1;
+    return m_effect.attribute("kdenlive_ix").toInt();
 }
 
+void CollapsibleEffect::updateWidget(ItemInfo info, int index, QDomElement effect, EffectMetaInfo *metaInfo)
+{
+    if (m_paramWidget) {
+       // cleanup
+       delete m_paramWidget;
+       m_paramWidget = NULL;
+    }
+    m_effect = effect;
+    setupWidget(info, index, metaInfo);
+}
 
 void CollapsibleEffect::setupWidget(ItemInfo info, int index, EffectMetaInfo *metaInfo)
 {
@@ -224,15 +454,18 @@ void CollapsibleEffect::setupWidget(ItemInfo info, int index, EffectMetaInfo *me
 //         kDebug() << "// EMPTY EFFECT STACK";
         return;
     }
+
     if (m_effect.attribute("tag") == "region") {
         QVBoxLayout *vbox = new QVBoxLayout(widgetFrame);
         vbox->setContentsMargins(0, 0, 0, 0);
+       vbox->setSpacing(2);
         QDomNodeList effects =  m_effect.elementsByTagName("effect");
+       QDomNodeList origin_effects =  m_original_effect.elementsByTagName("effect");
         QWidget *container = new QWidget(widgetFrame);
         vbox->addWidget(container);
         m_paramWidget = new ParameterContainer(m_effect.toElement(), info, metaInfo, index, container);
         for (int i = 0; i < effects.count(); i++) {
-            CollapsibleEffect *coll = new CollapsibleEffect(effects.at(i).toElement(), info, i, metaInfo, container);
+            CollapsibleEffect *coll = new CollapsibleEffect(effects.at(i).toElement(), origin_effects.at(i).toElement(), info, i, metaInfo, container);
             m_subParamWidgets.append(coll);
             //container = new QWidget(widgetFrame);
             vbox->addWidget(coll);
@@ -248,8 +481,15 @@ void CollapsibleEffect::setupWidget(ItemInfo info, int index, EffectMetaInfo *me
             widgetFrame->setVisible(false);            
         }
     }
-    if (collapseButton->isEnabled()) slotShow(true);
+    if (collapseButton->isEnabled() && m_info.isCollapsed) {
+       widgetFrame->setVisible(false);
+       collapseButton->setArrowType(Qt::RightArrow);
+       
+    }
     connect (m_paramWidget, SIGNAL(parameterChanged(const QDomElement, const QDomElement, int)), this, SIGNAL(parameterChanged(const QDomElement, const QDomElement, int)));
+    
+    connect(m_paramWidget, SIGNAL(startFilterJob(QString,QString,QString,QString,QString,QString)), this, SIGNAL(startFilterJob(QString,QString,QString,QString,QString,QString)));
+    
     connect (this, SIGNAL(syncEffectsPos(int)), m_paramWidget, SIGNAL(syncEffectsPos(int)));
     connect (this, SIGNAL(effectStateChanged(bool)), m_paramWidget, SIGNAL(effectStateChanged(bool)));
     connect (m_paramWidget, SIGNAL(checkMonitorPosition(int)), this, SIGNAL(checkMonitorPosition(int)));
@@ -273,14 +513,38 @@ void CollapsibleEffect::slotSyncEffectsPos(int pos)
     emit syncEffectsPos(pos);
 }
 
+void CollapsibleEffect::dragEnterEvent(QDragEnterEvent *event)
+{
+    if (event->mimeData()->hasFormat("kdenlive/effectslist"))
+       event->acceptProposedAction();
+}
 
+void CollapsibleEffect::dropEvent(QDropEvent *event)
+{
+    const QString effects = QString::fromUtf8(event->mimeData()->data("kdenlive/effectslist"));
+    //event->acceptProposedAction();
+    QDomDocument doc;
+    doc.setContent(effects, true);
+    const QDomElement e = doc.documentElement();
+    int ix = e.attribute("kdenlive_ix").toInt();
+    int last_index = -1;
+    if (m_isGroup) {
+       QVBoxLayout *vbox = static_cast<QVBoxLayout *>(widgetFrame->layout());
+       if (vbox == NULL) return;
+       CollapsibleEffect *e = static_cast<CollapsibleEffect *>(vbox->itemAt(vbox->count() -1)->widget());
+       last_index = e->effectIndex();
+    }
+    emit moveEffect(ix, this, last_index);
+    event->setDropAction(Qt::MoveAction);
+    event->accept();
+}
 
 ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, EffectMetaInfo *metaInfo, int index, QWidget * parent) :
+       m_index(index),
         m_keyframeEditor(NULL),
         m_geometryWidget(NULL),
-        m_effect(effect),
         m_metaInfo(metaInfo),
-        m_index(index)
+        m_effect(effect)
 {
     m_in = info.cropStart.frames(KdenliveSettings::project_fps());
     m_out = (info.cropStart + info.cropDuration).frames(KdenliveSettings::project_fps()) - 1;
@@ -297,8 +561,9 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
     parent->setEnabled(!disable);
 
     bool stretch = true;
-    QVBoxLayout *vbox = new QVBoxLayout(parent);
-    vbox->setContentsMargins(0, 0, 0, 0);
+    m_vbox = new QVBoxLayout(parent);
+    m_vbox->setContentsMargins(0, 0, 0, 0);
+    m_vbox->setSpacing(2);
 
     for (int i = 0; i < namenode.count() ; i++) {
         QDomElement pa = namenode.item(i).toElement();
@@ -331,13 +596,15 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
 
             DoubleParameterWidget *doubleparam = new DoubleParameterWidget(paramName, value.toDouble(), min, max,
                     pa.attribute("default").toDouble(), comment, -1, pa.attribute("suffix"), pa.attribute("decimals").toInt(), parent);
-            vbox->addWidget(doubleparam);
+           doubleparam->setFocusPolicy(Qt::StrongFocus);
+            m_vbox->addWidget(doubleparam);
             m_valueItems[paramName] = doubleparam;
             connect(doubleparam, SIGNAL(valueChanged(double)), this, SLOT(slotCollectAllParameters()));
             connect(this, SIGNAL(showComments(bool)), doubleparam, SLOT(slotShowComment(bool)));
         } else if (type == "list") {
             Listval *lsval = new Listval;
             lsval->setupUi(toFillin);
+           lsval->list->setFocusPolicy(Qt::StrongFocus);
             QStringList listitems = pa.attribute("paramlist").split(';');
             if (listitems.count() == 1) {
                 // probably custom effect created before change to ';' as separator
@@ -388,7 +655,7 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
         } else if (type == "complex") {
             ComplexParameter *pl = new ComplexParameter;
             pl->setupParam(effect, pa.attribute("name"), 0, 100);
-            vbox->addWidget(pl);
+            m_vbox->addWidget(pl);
             m_valueItems[paramName+"complex"] = pl;
             connect(pl, SIGNAL(parameterChanged()), this, SLOT(slotCollectAllParameters()));
         } else if (type == "geometry") {
@@ -403,7 +670,7 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
                     m_geometryWidget->setupParam(pa, m_in, m_out);
                 else
                     m_geometryWidget->setupParam(pa, minFrame, maxFrame);
-                vbox->addWidget(m_geometryWidget);
+                m_vbox->addWidget(m_geometryWidget);
                 m_valueItems[paramName+"geometry"] = m_geometryWidget;
                 connect(m_geometryWidget, SIGNAL(seekToPos(int)), this, SIGNAL(seekTimeline(int)));
                 connect(this, SIGNAL(syncEffectsPos(int)), m_geometryWidget, SLOT(slotSyncPosition(int)));
@@ -414,7 +681,7 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
                     geo->setupParam(pa, m_in, m_out);
                 else
                     geo->setupParam(pa, minFrame, maxFrame);
-                vbox->addWidget(geo);
+                m_vbox->addWidget(geo);
                 m_valueItems[paramName+"geometry"] = geo;
                 connect(geo, SIGNAL(parameterChanged()), this, SLOT(slotCollectAllParameters()));
                 connect(geo, SIGNAL(seekToPos(int)), this, SIGNAL(seekTimeline(int)));
@@ -438,7 +705,7 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
                 } else {
                     geo = new KeyframeEdit(pa, m_in, m_out, m_metaInfo->timecode, e.attribute("active_keyframe", "-1").toInt());
                 }
-                vbox->addWidget(geo);
+                m_vbox->addWidget(geo);
                 m_valueItems[paramName+"keyframe"] = geo;
                 m_keyframeEditor = geo;
                 connect(geo, SIGNAL(parameterChanged()), this, SLOT(slotCollectAllParameters()));
@@ -451,9 +718,8 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
         } else if (type == "color") {
             if (value.startsWith('#'))
                 value = value.replace('#', "0x");
-            bool ok;
-            ChooseColorWidget *choosecolor = new ChooseColorWidget(paramName, QColor(value.toUInt(&ok, 16)), parent);
-            vbox->addWidget(choosecolor);
+            ChooseColorWidget *choosecolor = new ChooseColorWidget(paramName, value, parent);
+            m_vbox->addWidget(choosecolor);
             m_valueItems[paramName] = choosecolor;
             connect(choosecolor, SIGNAL(displayMessage(const QString&, int)), this, SIGNAL(displayMessage(const QString&, int)));
             connect(choosecolor, SIGNAL(modified()) , this, SLOT(slotCollectAllParameters()));
@@ -466,7 +732,7 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
                 pos = m_out - pos;
             }
             PositionEdit *posedit = new PositionEdit(paramName, pos, 0, m_out - m_in, m_metaInfo->timecode);
-            vbox->addWidget(posedit);
+            m_vbox->addWidget(posedit);
             m_valueItems[paramName+"position"] = posedit;
             connect(posedit, SIGNAL(parameterChanged()), this, SLOT(slotCollectAllParameters()));
         } else if (type == "curve") {
@@ -486,14 +752,14 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
             }
             if (!points.isEmpty())
                 curve->setCurve(KisCubicCurve(points));
-            QSpinBox *spinin = new QSpinBox();
+            MySpinBox *spinin = new MySpinBox();
             spinin->setRange(0, 1000);
-            QSpinBox *spinout = new QSpinBox();
+            MySpinBox *spinout = new MySpinBox();
             spinout->setRange(0, 1000);
             curve->setupInOutControls(spinin, spinout, 0, 1000);
-            vbox->addWidget(curve);
-            vbox->addWidget(spinin);
-            vbox->addWidget(spinout);
+            m_vbox->addWidget(curve);
+            m_vbox->addWidget(spinin);
+            m_vbox->addWidget(spinout);
 
             connect(curve, SIGNAL(modified()), this, SLOT(slotCollectAllParameters()));
             m_valueItems[paramName] = curve;
@@ -504,7 +770,7 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
         } else if (type == "bezier_spline") {
             BezierSplineWidget *widget = new BezierSplineWidget(value, parent);
             stretch = false;
-            vbox->addWidget(widget);
+            m_vbox->addWidget(widget);
             m_valueItems[paramName] = widget;
             connect(widget, SIGNAL(modified()), this, SLOT(slotCollectAllParameters()));
             QString depends = pa.attribute("depends");
@@ -519,7 +785,7 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
             connect(roto, SIGNAL(seekToPos(int)), this, SIGNAL(seekTimeline(int)));
             connect(this, SIGNAL(syncEffectsPos(int)), roto, SLOT(slotSyncPosition(int)));
             connect(this, SIGNAL(effectStateChanged(bool)), roto, SLOT(slotShowScene(bool)));
-            vbox->addWidget(roto);
+            m_vbox->addWidget(roto);
             m_valueItems[paramName] = roto;
 #endif
         } else if (type == "wipe") {
@@ -587,17 +853,58 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
             connect(cval->urlwidget, SIGNAL(returnPressed()) , this, SLOT(slotCollectAllParameters()));
             connect(cval->urlwidget, SIGNAL(urlSelected(const KUrl&)) , this, SLOT(slotCollectAllParameters()));
             m_uiItems.append(cval);
+       } else if (type == "keywords") {
+            Keywordval* kval = new Keywordval;
+            kval->setupUi(toFillin);
+            kval->label->setText(paramName);
+            kval->lineeditwidget->setText(value);
+            QDomElement klistelem = pa.firstChildElement("keywords");
+            QDomElement kdisplaylistelem = pa.firstChildElement("keywordsdisplay");
+            QStringList keywordlist;
+            QStringList keyworddisplaylist;
+            if (!klistelem.isNull()) {
+                keywordlist = klistelem.text().split(';');
+                keyworddisplaylist = i18n(kdisplaylistelem.text().toUtf8().data()).split(';');
+            }
+            if (keyworddisplaylist.count() != keywordlist.count()) {
+                keyworddisplaylist = keywordlist;
+            }
+            for (int i = 0; i < keywordlist.count(); i++) {
+                kval->comboboxwidget->addItem(keyworddisplaylist.at(i), keywordlist.at(i));
+            }
+            // Add disabled user prompt at index 0
+            kval->comboboxwidget->insertItem(0, i18n("<select a keyword>"), "");
+            kval->comboboxwidget->model()->setData( kval->comboboxwidget->model()->index(0,0), QVariant(Qt::NoItemFlags), Qt::UserRole -1);
+            kval->comboboxwidget->setCurrentIndex(0);
+            m_valueItems[paramName] = kval;
+            connect(kval->lineeditwidget, SIGNAL(editingFinished()) , this, SLOT(collectAllParameters()));
+            connect(kval->comboboxwidget, SIGNAL(activated (const QString&)), this, SLOT(collectAllParameters()));
+            m_uiItems.append(kval);
+        } else if (type == "fontfamily") {
+            Fontval* fval = new Fontval;
+            fval->setupUi(toFillin);
+            fval->name->setText(paramName);
+            fval->fontfamilywidget->setCurrentFont(QFont(value));
+            m_valueItems[paramName] = fval;
+            connect(fval->fontfamilywidget, SIGNAL(currentFontChanged(const QFont &)), this, SLOT(collectAllParameters())) ;
+            m_uiItems.append(fval);
+        } else if (type == "filterjob") {
+           QVBoxLayout *l= new QVBoxLayout(toFillin);
+           QPushButton *button = new QPushButton(paramName, toFillin);
+           l->addWidget(button);
+            m_valueItems[paramName] = button;
+            connect(button, SIGNAL(pressed()), this, SLOT(slotStartFilterJobAction()));   
         } else {
             delete toFillin;
             toFillin = NULL;
         }
 
         if (toFillin)
-            vbox->addWidget(toFillin);
+            m_vbox->addWidget(toFillin);
     }
 
     if (stretch)
-        vbox->addStretch();
+        m_vbox->addStretch();
 
     if (m_keyframeEditor)
         m_keyframeEditor->checkVisibleParam();
@@ -613,6 +920,11 @@ ParameterContainer::ParameterContainer(QDomElement effect, ItemInfo info, Effect
     }
 }
 
+ParameterContainer::~ParameterContainer()
+{
+    //clearLayout(m_vbox);
+    delete m_vbox;
+}
 
 void ParameterContainer::meetDependency(const QString& name, QString type, QString value)
 {
@@ -711,8 +1023,8 @@ void ParameterContainer::slotCollectAllParameters()
     QLocale locale;
     locale.setNumberOptions(QLocale::OmitGroupSeparator);
     const QDomElement oldparam = m_effect.cloneNode().toElement();
-    QDomElement newparam = oldparam.cloneNode().toElement();
-    QDomNodeList namenode = newparam.elementsByTagName("parameter");
+    //QDomElement newparam = oldparam.cloneNode().toElement();
+    QDomNodeList namenode = m_effect.elementsByTagName("parameter");
 
     for (int i = 0; i < namenode.count() ; i++) {
         QDomNode pa = namenode.item(i);
@@ -744,7 +1056,7 @@ void ParameterContainer::slotCollectAllParameters()
             setValue = box->checkState() == Qt::Checked ? "1" : "0" ;
         } else if (type == "color") {
             ChooseColorWidget *choosecolor = ((ChooseColorWidget*)m_valueItems.value(paramName));
-            setValue = choosecolor->getColor().name();
+            setValue = choosecolor->getColor();
         } else if (type == "complex") {
             ComplexParameter *complex = ((ComplexParameter*)m_valueItems.value(paramName));
             namenode.item(i) = complex->getParamDesc();
@@ -761,23 +1073,23 @@ void ParameterContainer::slotCollectAllParameters()
             PositionEdit *pedit = ((PositionEdit*)m_valueItems.value(paramName));
             int pos = pedit->getPosition();
             setValue = QString::number(pos);
-            if (newparam.attribute("id") == "fadein" || newparam.attribute("id") == "fade_from_black") {
+            if (m_effect.attribute("id") == "fadein" || m_effect.attribute("id") == "fade_from_black") {
                 // Make sure duration is not longer than clip
                 /*if (pos > m_out) {
                     pos = m_out;
                     pedit->setPosition(pos);
                 }*/
-                EffectsList::setParameter(newparam, "in", QString::number(m_in));
-                EffectsList::setParameter(newparam, "out", QString::number(m_in + pos));
+                EffectsList::setParameter(m_effect, "in", QString::number(m_in));
+                EffectsList::setParameter(m_effect, "out", QString::number(m_in + pos));
                 setValue.clear();
-            } else if (newparam.attribute("id") == "fadeout" || newparam.attribute("id") == "fade_to_black") {
+            } else if (m_effect.attribute("id") == "fadeout" || m_effect.attribute("id") == "fade_to_black") {
                 // Make sure duration is not longer than clip
                 /*if (pos > m_out) {
                     pos = m_out;
                     pedit->setPosition(pos);
                 }*/
-                EffectsList::setParameter(newparam, "in", QString::number(m_out - pos));
-                EffectsList::setParameter(newparam, "out", QString::number(m_out));
+                EffectsList::setParameter(m_effect, "in", QString::number(m_out - pos));
+                EffectsList::setParameter(m_effect, "out", QString::number(m_out));
                 setValue.clear();
             }
         } else if (type == "curve") {
@@ -788,24 +1100,24 @@ void ParameterContainer::slotCollectAllParameters()
             QString outName = pa.attributes().namedItem("outpoints").nodeValue();
             int off = pa.attributes().namedItem("min").nodeValue().toInt();
             int end = pa.attributes().namedItem("max").nodeValue().toInt();
-            EffectsList::setParameter(newparam, number, QString::number(points.count()));
+            EffectsList::setParameter(m_effect, number, QString::number(points.count()));
             for (int j = 0; (j < points.count() && j + off <= end); j++) {
                 QString in = inName;
                 in.replace("%i", QString::number(j + off));
                 QString out = outName;
                 out.replace("%i", QString::number(j + off));
-                EffectsList::setParameter(newparam, in, locale.toString(points.at(j).x()));
-                EffectsList::setParameter(newparam, out, locale.toString(points.at(j).y()));
+                EffectsList::setParameter(m_effect, in, locale.toString(points.at(j).x()));
+                EffectsList::setParameter(m_effect, out, locale.toString(points.at(j).y()));
             }
             QString depends = pa.attributes().namedItem("depends").nodeValue();
             if (!depends.isEmpty())
-                meetDependency(paramName, type, EffectsList::parameter(newparam, depends));
+                meetDependency(paramName, type, EffectsList::parameter(m_effect, depends));
         } else if (type == "bezier_spline") {
             BezierSplineWidget *widget = (BezierSplineWidget*)m_valueItems.value(paramName);
             setValue = widget->spline();
             QString depends = pa.attributes().namedItem("depends").nodeValue();
             if (!depends.isEmpty())
-                meetDependency(paramName, type, EffectsList::parameter(newparam, depends));
+                meetDependency(paramName, type, EffectsList::parameter(m_effect, depends));
 #ifdef USE_QJSON
         } else if (type == "roto-spline") {
             RotoWidget *widget = static_cast<RotoWidget *>(m_valueItems.value(paramName));
@@ -856,13 +1168,25 @@ void ParameterContainer::slotCollectAllParameters()
         } else if (type == "url") {
             KUrlRequester *req = ((Urlval*)m_valueItems.value(paramName))->urlwidget;
             setValue = req->url().path();
+       } else if (type == "keywords"){
+            QLineEdit *line = ((Keywordval*)m_valueItems.value(paramName))->lineeditwidget;
+            QComboBox *combo = ((Keywordval*)m_valueItems.value(paramName))->comboboxwidget;
+            if(combo->currentIndex())
+            {
+                QString comboval = combo->itemData(combo->currentIndex()).toString();
+                line->insert(comboval);
+                combo->setCurrentIndex(0);
+            }
+            setValue = line->text();
+        } else if (type == "fontfamily") {
+            QFontComboBox* fontfamily = ((Fontval*)m_valueItems.value(paramName))->fontfamilywidget;
+            setValue = fontfamily->currentFont().family();
         }
-
         if (!setValue.isNull())
             pa.attributes().namedItem("value").setNodeValue(setValue);
 
     }
-    emit parameterChanged(oldparam, newparam, m_index);
+    emit parameterChanged(oldparam, m_effect, m_index);
 }
 
 QString ParameterContainer::getWipeString(wipeInfo info)
@@ -910,9 +1234,22 @@ QString ParameterContainer::getWipeString(wipeInfo info)
     return QString(start + ";-1=" + end);
 }
 
-int ParameterContainer::index()
+int ParameterContainer::index() const
 {
     return m_index;
 }
 
+void ParameterContainer::slotStartFilterJobAction()
+{
+    QDomNodeList namenode = m_effect.elementsByTagName("parameter");
+    for (int i = 0; i < namenode.count() ; i++) {
+        QDomElement pa = namenode.item(i).toElement();
+        QString type = pa.attribute("type");
+        if (type == "filterjob") {
+            emit startFilterJob(pa.attribute("filtertag"), pa.attribute("filterparams"), pa.attribute("finalfilter"), pa.attribute("consumer"), pa.attribute("consumerparams"), pa.attribute("wantedproperties"));
+            kDebug()<<" - - -PROPS:\n"<<pa.attribute("filtertag")<<"-"<< pa.attribute("filterparams")<<"-"<< pa.attribute("consumer")<<"-"<< pa.attribute("consumerparams")<<"-"<< pa.attribute("wantedproperties");
+            break;
+        }
+    }
+}