]> git.sesse.net Git - kdenlive/blobdiff - src/clipitem.cpp
projects / kdenlive / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Fix fade effects
[kdenlive] / src / clipitem.cpp
diff --git a/src/clipitem.cpp b/src/clipitem.cpp
index d6be7a43ec0892520f2aeecfa9ad029058d6e0ac..3f342969f8c88c9054a23dd59a90a829addab518 100644 (file)
--- a/src/clipitem.cpp
+++ b/src/clipitem.cpp
@@ -226,7 +226,7 @@ int ClipItem::selectedEffectIndex() const
     return m_selectedEffect;
 }
 
-void ClipItem::initEffect(QDomElement effect, int diff)
+void ClipItem::initEffect(QDomElement effect, int diff, int offset)
 {
     // the kdenlive_ix int is used to identify an effect in mlt's playlist, should
     // not be changed
@@ -258,10 +258,24 @@ void ClipItem::initEffect(QDomElement effect, int diff)
                 e.setAttribute("value", "1");
         }
 
-        if ((e.attribute("type") == "keyframe" || e.attribute("type") == "simplekeyframe") && e.attribute("keyframes").isEmpty()) {
-            // Effect has a keyframe type parameter, we need to set the values
-            e.setAttribute("keyframes", QString::number(cropStart().frames(m_fps)) + ':' + e.attribute("default"));
+        if (e.attribute("type") == "keyframe" || e.attribute("type") == "simplekeyframe") {
+           if (e.attribute("keyframes").isEmpty()) {
+               // Effect has a keyframe type parameter, we need to set the values
+               e.setAttribute("keyframes", QString::number(cropStart().frames(m_fps)) + ':' + e.attribute("default"));
+           }
+           else if (offset != 0) {
+               // adjust keyframes to this clip
+               QString adjusted = adjustKeyframes(e.attribute("keyframes"), offset - cropStart().frames(m_fps));
+               e.setAttribute("keyframes", adjusted);
+           }
         }
+
+        if (e.attribute("type") == "geometry" && !e.hasAttribute("fixed")) {
+            // Effects with a geometry parameter need to sync in / out with parent clip
+           effect.setAttribute("in", QString::number(cropStart().frames(m_fps)));
+           effect.setAttribute("out", QString::number((cropStart() + cropDuration()).frames(m_fps) - 1));
+           effect.setAttribute("_sync_in_out", "1");
+       }
     }
     if (effect.attribute("tag") == "volume" || effect.attribute("tag") == "brightness") {
         if (effect.attribute("id") == "fadeout" || effect.attribute("id") == "fade_to_black") {
@@ -298,6 +312,7 @@ void ClipItem::initEffect(QDomElement effect, int diff)
             if (effect.attribute("id") == "fadein") {
                 if (m_effectList.hasEffect(QString(), "fade_from_black") == -1) {
                     int effectDuration = EffectsList::parameter(effect, "out").toInt();
+                   if (offset != 0) effectDuration -= offset - cropStart().frames(m_fps);
                     if (effectDuration > cropDuration().frames(m_fps)) {
                         effectDuration = cropDuration().frames(m_fps) / 2;
                     }
@@ -307,6 +322,7 @@ void ClipItem::initEffect(QDomElement effect, int diff)
             } else if (effect.attribute("id") == "fade_from_black") {
                 if (m_effectList.hasEffect(QString(), "fadein") == -1) {
                     int effectDuration = EffectsList::parameter(effect, "out").toInt();
+                   if (offset != 0) effectDuration -= offset - cropStart().frames(m_fps);
                     if (effectDuration > cropDuration().frames(m_fps)) {
                         effectDuration = cropDuration().frames(m_fps) / 2;
                     }
@@ -320,6 +336,19 @@ void ClipItem::initEffect(QDomElement effect, int diff)
     }
 }
 
+const QString ClipItem::adjustKeyframes(QString keyframes, int offset)
+{
+    QStringList result;
+    // Simple keyframes
+    const QStringList list = keyframes.split(';', QString::SkipEmptyParts);
+    foreach(const QString &keyframe, list) {
+       int pos = keyframe.section(':', 0, 0).toInt() - offset;
+       QString newKey = QString::number(pos) + ":" + keyframe.section(':', 1);
+       result.append(newKey);
+    }
+    return result.join(";");
+}
+
 bool ClipItem::checkKeyFrames()
 {
     bool clipEffectsModified = false;
@@ -433,7 +462,7 @@ void ClipItem::setSelectedEffect(const int ix)
 {
     m_selectedEffect = ix;
     QLocale locale;
-    QDomElement effect = effectAt(m_selectedEffect);
+    QDomElement effect = effectAtIndex(m_selectedEffect);
     if (!effect.isNull() && effect.attribute("disable") != "1") {
         QDomNodeList params = effect.elementsByTagName("parameter");
         for (int i = 0; i < params.count(); i++) {
@@ -487,7 +516,7 @@ QStringList ClipItem::keyframes(const int index)
 void ClipItem::updateKeyframeEffect()
 {
     // regenerate xml parameter from the clip keyframes
-    QDomElement effect = getEffectAt(m_selectedEffect);
+    QDomElement effect = getEffectAtIndex(m_selectedEffect);
     if (effect.attribute("disable") == "1") return;
     QDomNodeList params = effect.elementsByTagName("parameter");
     QDomElement e = params.item(m_visibleParam).toElement();
@@ -509,7 +538,7 @@ void ClipItem::updateKeyframeEffect()
 QDomElement ClipItem::selectedEffect()
 {
     if (m_selectedEffect == -1 || m_effectList.isEmpty()) return QDomElement();
-    return effectAt(m_selectedEffect);
+    return effectAtIndex(m_selectedEffect);
 }
 
 void ClipItem::resetThumbs(bool clearExistingThumbs)
@@ -1002,11 +1031,11 @@ OPERATIONTYPE ClipItem::operationMode(QPointF pos)
 
     if (qAbs((int)(pos.x() - (rect.x() + m_startFade))) < maximumOffset  && qAbs((int)(pos.y() - rect.y())) < 6) {
         return FADEIN;
-    } else if (pos.x() - rect.x() < maximumOffset && (rect.bottom() - pos.y() > addtransitionOffset)) {
+    } else if ((pos.x() <= rect.x() + rect.width() / 2) && pos.x() - rect.x() < maximumOffset && (rect.bottom() - pos.y() > addtransitionOffset)) {
         return RESIZESTART;
     } else if (qAbs((int)(pos.x() - (rect.x() + rect.width() - m_endFade))) < maximumOffset && qAbs((int)(pos.y() - rect.y())) < 6) {
         return FADEOUT;
-    } else if ((rect.right() - pos.x() < maximumOffset) && (rect.bottom() - pos.y() > addtransitionOffset)) {
+    } else if ((pos.x() >= rect.x() + rect.width() / 2) && (rect.right() - pos.x() < maximumOffset) && (rect.bottom() - pos.y() > addtransitionOffset)) {
         return RESIZEEND;
     } else if ((pos.x() - rect.x() < 16 / scale) && (rect.bottom() - pos.y() <= addtransitionOffset)) {
         return TRANSITIONSTART;
@@ -1359,13 +1388,19 @@ QStringList ClipItem::effectNames()
     return m_effectList.effectNames();
 }
 
-QDomElement ClipItem::effectAt(int ix) const
+QDomElement ClipItem::effect(int ix) const
+{
+    if (ix >= m_effectList.count() || ix < 0) return QDomElement();
+    return m_effectList.at(ix).cloneNode().toElement();
+}
+
+QDomElement ClipItem::effectAtIndex(int ix) const
 {
     if (ix > m_effectList.count() || ix <= 0) return QDomElement();
     return m_effectList.itemFromIndex(ix).cloneNode().toElement();
 }
 
-QDomElement ClipItem::getEffectAt(int ix) const
+QDomElement ClipItem::getEffectAtIndex(int ix) const
 {
     if (ix > m_effectList.count() || ix <= 0) return QDomElement();
     return m_effectList.itemFromIndex(ix);
@@ -1386,6 +1421,11 @@ void ClipItem::updateEffect(QDomElement effect)
     }
 }
 
+void ClipItem::enableEffects(QList <int> indexes, bool disable)
+{
+    m_effectList.enableEffects(indexes, disable);
+}
+
 bool ClipItem::moveEffect(QDomElement effect, int ix)
 {
     if (ix <= 0 || ix > (m_effectList.count()) || effect.isNull()) {
@@ -1424,6 +1464,18 @@ EffectsParameterList ClipItem::addEffect(QDomElement effect, bool /*animate*/)
     
     // Update index to the real one
     effect.setAttribute("kdenlive_ix", insertedEffect.attribute("kdenlive_ix"));
+    int effectIn;
+    int effectOut;
+
+    if (effect.attribute("tag") == "affine") {
+       // special case: the affine effect needs in / out points
+       effectIn = effect.attribute("in").toInt();
+       effectOut = effect.attribute("out").toInt();
+    }
+    else {
+       effectIn = EffectsList::parameter(effect, "in").toInt();
+       effectOut = EffectsList::parameter(effect, "out").toInt();
+    }
     
     EffectsParameterList parameters;
     parameters.addParam("tag", insertedEffect.attribute("tag"));
@@ -1435,11 +1487,50 @@ EffectsParameterList ClipItem::addEffect(QDomElement effect, bool /*animate*/)
     if (effectId.isEmpty()) effectId = insertedEffect.attribute("tag");
     parameters.addParam("id", effectId);
 
-    // special case: the affine effect needs in / out points
-
     QDomNodeList params = insertedEffect.elementsByTagName("parameter");
     int fade = 0;
     bool needInOutSync = false;
+
+    // check if it is a fade effect
+    if (effectId == "fadein") {
+       needRepaint = true;
+        if (m_effectList.hasEffect(QString(), "fade_from_black") == -1) {
+           fade = effectOut - effectIn;
+        }/* else {
+           QDomElement fadein = m_effectList.getEffectByTag(QString(), "fade_from_black");
+            if (fadein.attribute("name") == "out") fade += fadein.attribute("value").toInt();
+            else if (fadein.attribute("name") == "in") fade -= fadein.attribute("value").toInt();
+        }*/
+    } else if (effectId == "fade_from_black") {
+       kDebug()<<"// FOUND FTB:"<<effectOut<<" - "<<effectIn;
+       needRepaint = true;
+        if (m_effectList.hasEffect(QString(), "fadein") == -1) {
+           fade = effectOut - effectIn;
+        }/* else {
+           QDomElement fadein = m_effectList.getEffectByTag(QString(), "fadein");
+            if (fadein.attribute("name") == "out") fade += fadein.attribute("value").toInt();
+            else if (fadein.attribute("name") == "in") fade -= fadein.attribute("value").toInt();
+        }*/
+     } else if (effectId == "fadeout") {
+       needRepaint = true;
+        if (m_effectList.hasEffect(QString(), "fade_to_black") == -1) {
+           fade = effectIn - effectOut;
+        } /*else {
+           QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fade_to_black");
+            if (fadeout.attribute("name") == "out") fade -= fadeout.attribute("value").toInt();
+            else if (fadeout.attribute("name") == "in") fade += fadeout.attribute("value").toInt();
+        }*/
+    } else if (effectId == "fade_to_black") {
+       needRepaint = true;
+        if (m_effectList.hasEffect(QString(), "fadeout") == -1) {
+           fade = effectIn - effectOut;
+        }/* else {
+           QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fadeout");
+            if (fadeout.attribute("name") == "out") fade -= fadeout.attribute("value").toInt();
+            else if (fadeout.attribute("name") == "in") fade += fadeout.attribute("value").toInt();
+        }*/
+    }
+
     for (int i = 0; i < params.count(); i++) {
         QDomElement e = params.item(i).toElement();
         if (!e.isNull()) {
@@ -1473,48 +1564,6 @@ EffectsParameterList ClipItem::addEffect(QDomElement effect, bool /*animate*/)
             } else if (e.attribute("factor", "1") == "1" && e.attribute("offset", "0") == "0") {
                 parameters.addParam(e.attribute("name"), e.attribute("value"));
 
-                // check if it is a fade effect
-                if (effectId == "fadein") {
-                    needRepaint = true;
-                    if (m_effectList.hasEffect(QString(), "fade_from_black") == -1) {
-                        if (e.attribute("name") == "out") fade += e.attribute("value").toInt();
-                        else if (e.attribute("name") == "in") fade -= e.attribute("value").toInt();
-                    } else {
-                        QDomElement fadein = m_effectList.getEffectByTag(QString(), "fade_from_black");
-                        if (fadein.attribute("name") == "out") fade += fadein.attribute("value").toInt();
-                        else if (fadein.attribute("name") == "in") fade -= fadein.attribute("value").toInt();
-                    }
-                } else if (effectId == "fade_from_black") {
-                    needRepaint = true;
-                    if (m_effectList.hasEffect(QString(), "fadein") == -1) {
-                        if (e.attribute("name") == "out") fade += e.attribute("value").toInt();
-                        else if (e.attribute("name") == "in") fade -= e.attribute("value").toInt();
-                    } else {
-                        QDomElement fadein = m_effectList.getEffectByTag(QString(), "fadein");
-                        if (fadein.attribute("name") == "out") fade += fadein.attribute("value").toInt();
-                        else if (fadein.attribute("name") == "in") fade -= fadein.attribute("value").toInt();
-                    }
-                } else if (effectId == "fadeout") {
-                    needRepaint = true;
-                    if (m_effectList.hasEffect(QString(), "fade_to_black") == -1) {
-                        if (e.attribute("name") == "out") fade -= e.attribute("value").toInt();
-                        else if (e.attribute("name") == "in") fade += e.attribute("value").toInt();
-                    } else {
-                        QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fade_to_black");
-                        if (fadeout.attribute("name") == "out") fade -= fadeout.attribute("value").toInt();
-                        else if (fadeout.attribute("name") == "in") fade += fadeout.attribute("value").toInt();
-                    }
-                } else if (effectId == "fade_to_black") {
-                    needRepaint = true;
-                    if (m_effectList.hasEffect(QString(), "fadeout") == -1) {
-                        if (e.attribute("name") == "out") fade -= e.attribute("value").toInt();
-                        else if (e.attribute("name") == "in") fade += e.attribute("value").toInt();
-                    } else {
-                        QDomElement fadeout = m_effectList.getEffectByTag(QString(), "fadeout");
-                        if (fadeout.attribute("name") == "out") fade -= fadeout.attribute("value").toInt();
-                        else if (fadeout.attribute("name") == "in") fade += fadeout.attribute("value").toInt();
-                    }
-                }
             } else {
                 double fact;
                 if (e.attribute("factor").contains('%')) {
@@ -1845,9 +1894,9 @@ QMap<int, QDomElement> ClipItem::adjustEffectsToDuration(int width, int height,
             if (id == "fade_from_black" || id == "fadein") {
                 if (in != cropStart().frames(m_fps)) {
                     effects[i] = effect.cloneNode().toElement();
-                    int diff = in - cropStart().frames(m_fps);
-                    in -= diff;
-                    out -= diff;
+                    int duration = out - in;
+                    in = cropStart().frames(m_fps);
+                    out = in + duration;
                     EffectsList::setParameter(effect, "in", QString::number(in));
                     EffectsList::setParameter(effect, "out", QString::number(out));
                 }
Unnamed repository; edit this file 'description' to name the repository.