]> git.sesse.net Git - kdenlive/blobdiff - src/transition.cpp
Move undo commands to their own subdirectory
[kdenlive] / src / transition.cpp
index ae6b6cd2b6546c7cde2a07c8323c02ff32185034..fe3aa518e7c265bbb993e780ea97f5adebadcdb6 100644 (file)
  *                                                                         *
  ***************************************************************************/
 
-#include <QBrush>
-#include <qdom.h>
-#include <QPainter>
-#include <QGraphicsScene>
-#include <QGraphicsView>
-#include <QScrollBar>
-#include <QStyleOptionGraphicsItem>
+#include "transition.h"
+#include "clipitem.h"
+#include "kdenlivesettings.h"
+#include "customtrackscene.h"
+#include "mainwindow.h"
 
 #include <kdebug.h>
 #include <KIcon>
 #include <klocale.h>
 
-#include "transition.h"
-#include "clipitem.h"
-#include "kdenlivesettings.h"
-
+#include <QBrush>
+#include <QDomElement>
+#include <QPainter>
+#include <QStyleOptionGraphicsItem>
 
-Transition::Transition(const QRectF& rect , ClipItem * clipa, const TRANSITIONTYPE & type, const GenTime &startTime, const GenTime &endTime, double fps, bool inverted) : AbstractClipItem(rect) {
-    m_invertTransition = inverted;
-    m_singleClip = true;
-    m_transitionTrack = 0;
-    m_secondClip = NULL;
-    m_transitionType = type;
-    m_transitionName = getTransitionName(m_transitionType);
-    m_fps = fps;
-    m_cropDuration = endTime - startTime;
-    m_startPos = startTime;
-    // Default duration = 2.5 seconds
-    GenTime defaultTransitionDuration = GenTime(2.5);
-
-    m_referenceClip = clipa;
-    if (startTime < m_referenceClip->startPos()) m_transitionStart = GenTime(0.0);
-    else if (startTime > m_referenceClip->endPos()) m_transitionStart = m_referenceClip->duration() - defaultTransitionDuration;
-    else m_transitionStart = startTime - m_referenceClip->startPos();
-
-    if (m_transitionStart + m_cropDuration > m_referenceClip->duration())
-        m_transitionDuration = m_referenceClip->duration() - m_transitionStart;
-    else m_transitionDuration = m_cropDuration;
-    m_secondClip = 0;
-    setFlags(QGraphicsItem::ItemClipsToShape | QGraphicsItem::ItemIsMovable | QGraphicsItem::ItemIsSelectable);
-    setZValue(2);
-}
 
-// create a transition from XML
-Transition::Transition(const QRectF& rect , ClipItem * clip, QDomElement transitionElement, double fps, GenTime offset) : AbstractClipItem(rect) {
-    if (offset == GenTime()) offset = clip->startPos();
-    m_referenceClip = clip;
-    m_singleClip = true;
-    m_secondClip = NULL;
-    m_transitionStart = GenTime(transitionElement.attribute("start", QString::null).toInt(), m_referenceClip->fps());
-    m_transitionDuration = GenTime(transitionElement.attribute("end", QString::null).toInt(), m_referenceClip->fps()) - m_transitionStart;
-    m_transitionTrack = transitionElement.attribute("transition_track", "0").toInt();
-    m_transitionStart = m_transitionStart - offset;
-
-    m_invertTransition = transitionElement.attribute("inverted", "0").toInt();
-    uint transType = transitionElement.attribute("type", "0").toInt();
-    if (transType == LUMA_TRANSITION) m_transitionType = LUMA_TRANSITION;
-    else if (transType == COMPOSITE_TRANSITION) m_transitionType = COMPOSITE_TRANSITION;
-    else if (transType == PIP_TRANSITION) m_transitionType = PIP_TRANSITION;
-    else if (transType == LUMAFILE_TRANSITION) m_transitionType = LUMAFILE_TRANSITION;
-    else if (transType == MIX_TRANSITION) m_transitionType = MIX_TRANSITION;
-
-    // load transition parameters
-    typedef QMap<QString, QString> ParamMap;
-    ParamMap params;
-    for (QDomNode n = transitionElement.firstChild(); !n.isNull(); n = n.nextSibling()) {
-        QDomElement paramElement = n.toElement();
-        params[paramElement.tagName()] = paramElement.attribute("value", QString::null);
+Transition::Transition(const ItemInfo &info, int transitiontrack, double fps, QDomElement params, bool automaticTransition) :
+        AbstractClipItem(info, QRectF(), fps),
+        m_forceTransitionTrack(false),
+        m_automaticTransition(automaticTransition),
+        m_secondClip(NULL),
+        m_transitionTrack(transitiontrack)
+{
+    setZValue(3);
+    m_info.cropDuration = info.endPos - info.startPos;
+    setPos(info.startPos.frames(fps), (int)(info.track * KdenliveSettings::trackheight() + itemOffset() + 1));
+
+#if QT_VERSION >= 0x040600
+    m_startAnimation = new QPropertyAnimation(this, "rect");
+    m_startAnimation->setDuration(200);
+    QRectF r(0, 0, m_info.cropDuration.frames(fps) - 0.02, (qreal) itemHeight() / 2);
+    QRectF r2(0, 0, m_info.cropDuration.frames(fps) - 0.02, (qreal)itemHeight());
+    m_startAnimation->setStartValue(r);
+    m_startAnimation->setEndValue(r2);
+    m_startAnimation->setEasingCurve(QEasingCurve::OutQuad);
+    m_startAnimation->start();
+#else
+    setRect(0, 0, m_info.cropDuration.frames(fps) - 0.02, (qreal) itemHeight());
+#endif
+
+    m_info.cropStart = GenTime();
+    m_maxDuration = GenTime(600);
+
+    if (m_automaticTransition) setBrush(QColor(200, 200, 50, 180));
+    else setBrush(QColor(200, 100, 50, 180));
+
+    //m_referenceClip = clipa;
+    if (params.isNull()) {
+        m_parameters = MainWindow::transitions.getEffectByTag("luma", "dissolve").cloneNode().toElement();
+    } else {
+        m_parameters = params;
     }
-    if (m_invertTransition) params["reverse"] = "1";
-    if (!params.isEmpty()) setTransitionParameters(params);
-
-    // Check if transition is valid (not outside of clip)
-    if (m_transitionStart > clip->duration())
-        m_transitionDuration = GenTime();
+    if (m_automaticTransition) m_parameters.setAttribute("automatic", 1);
+    else if (m_parameters.attribute("automatic") == "1") m_automaticTransition = true;
+    if (m_parameters.attribute("force_track") == "1") m_forceTransitionTrack = true;
+    m_name = i18n(m_parameters.firstChildElement("name").text().toUtf8().data());
+    m_secondClip = 0;
 
+    //m_referenceClip->addTransition(this);
 }
 
-Transition::~Transition() {
+Transition::~Transition()
+{
+    blockSignals(true);
+#if QT_VERSION >= 0x040600
+    delete m_startAnimation;
+#endif
+    if (scene()) scene()->removeItem(this);
 }
 
-void Transition::setTransitionType(TRANSITIONTYPE newType) {
-    m_transitionType = newType;
-    m_transitionName = getTransitionName(m_transitionType);
+Transition *Transition::clone()
+{
+    QDomElement xml = toXML().cloneNode().toElement();
+    Transition *tr = new Transition(info(), transitionEndTrack(), m_fps, xml);
+    return tr;
 }
 
-TRANSITIONTYPE Transition::transitionType() const {
-    return m_transitionType;
+QString Transition::transitionTag() const
+{
+    return m_parameters.attribute("tag");
 }
 
-QString Transition::transitionTag()const {
-    switch (m_transitionType) {
-    case COMPOSITE_TRANSITION:
-        return "composite";
-    case PIP_TRANSITION:
-        return "composite";
-    case MIX_TRANSITION:
-        return "mix";
-    default:
-        return "luma";
-
-    }
+QStringList Transition::transitionInfo() const
+{
+    QStringList info;
+    info << m_name << m_parameters.attribute("tag") << m_parameters.attribute("id");
+    return info;
 }
 
-QString Transition::getTransitionName(const TRANSITIONTYPE & type) {
-    if (type == COMPOSITE_TRANSITION) return i18n("Push");
-    else if (type == PIP_TRANSITION) return i18n("Pip");
-    else if (type == LUMAFILE_TRANSITION) return i18n("Wipe");
-    else if (type == MIX_TRANSITION) return i18n("Audio Fade");
-    return i18n("Crossfade");
+bool Transition::isAutomatic() const
+{
+    return m_automaticTransition;
 }
 
-TRANSITIONTYPE Transition::getTransitionForName(const QString & type) {
-    if (type == i18n("Push")) return COMPOSITE_TRANSITION;
-    else if (type == i18n("Pip")) return PIP_TRANSITION;
-    else if (type == i18n("Wipe")) return LUMAFILE_TRANSITION;
-    return LUMA_TRANSITION;
+void Transition::setAutomatic(bool automatic)
+{
+    m_automaticTransition = automatic;
+    if (automatic) {
+        m_parameters.setAttribute("automatic", 1);
+        setBrush(QColor(200, 200, 50, 180));
+    } else {
+        m_parameters.removeAttribute("automatic");
+        setBrush(QColor(200, 100, 50, 180));
+    }
+    update();
 }
 
-
-QString Transition::transitionName() const {
-    return m_transitionName;
+void Transition::setTransitionParameters(const QDomElement params)
+{
+    m_parameters = params;
+    if (m_parameters.attribute("force_track") == "1") setForcedTrack(true, m_parameters.attribute("transition_btrack").toInt());
+    else if (m_parameters.attribute("force_track") == "0") setForcedTrack(false, m_parameters.attribute("transition_btrack").toInt());
+    m_name = i18n(m_parameters.firstChildElement("name").text().toUtf8().data());
+    update();
 }
 
-void Transition::setTransitionParameters(const QMap < QString, QString > parameters) {
-    m_transitionParameters = parameters;
-}
 
-const QMap < QString, QString > Transition::transitionParameters() const {
-    return m_transitionParameters;
+bool Transition::invertedTransition() const
+{
+    return false; //m_parameters.attribute("reverse").toInt();
 }
 
-bool Transition::invertTransition() const {
-    if (!m_singleClip) {
-        if (m_referenceClip->startPos() < m_secondClip->startPos()) return true;
-        else return false;
-    }
-    return m_invertTransition;
+void Transition::setTransitionDirection(bool /*inv*/)
+{
+    //m_parameters.setAttribute("reverse", inv);
 }
 
-QPixmap Transition::transitionPixmap() const {
-    KIcon icon;
-    if (m_transitionType == LUMA_TRANSITION) {
-        if (invertTransition()) icon = KIcon("kdenlive_trans_down");
-        else icon = KIcon("kdenlive_trans_up");
-    } else if (m_transitionType == COMPOSITE_TRANSITION) {
-        icon = KIcon("kdenlive_trans_wiper");
-    } else if (m_transitionType == LUMAFILE_TRANSITION) {
-        icon = KIcon("kdenlive_trans_luma");
-    } else icon = KIcon("kdenlive_trans_pip");
-    return icon.pixmap(QSize(15, 15));
+int Transition::transitionEndTrack() const
+{
+    return m_transitionTrack;
 }
 
-int Transition::transitionTrack() const {
-    return m_transitionTrack;
+void Transition::updateTransitionEndTrack(int newtrack)
+{
+    if (!m_forceTransitionTrack) m_transitionTrack = newtrack;
 }
 
-void Transition::setTransitionTrack(int track) {
+void Transition::setForcedTrack(bool force, int track)
+{
+    m_forceTransitionTrack = force;
     m_transitionTrack = track;
 }
 
-void Transition::setTransitionDirection(bool inv) {
-    m_invertTransition = inv;
+bool Transition::forcedTrack() const
+{
+    return m_forceTransitionTrack;
 }
 
-int Transition::transitionStartTrack() const {
-    return m_referenceClip->track();
-}
+void Transition::paint(QPainter *painter,
+                       const QStyleOptionGraphicsItem *option,
+                       QWidget */*widget*/)
+{
+    const QRectF exposed = option->exposedRect;
+    painter->setClipRect(exposed);
+    const QRectF br = rect();
+    QPen framePen;
+    const QRectF mapped = painter->worldTransform().mapRect(br);
+
+    painter->fillRect(exposed, brush());
+
+    QPointF p1(br.x(), br.y() + br.height() / 2 - 7);
+    painter->setWorldMatrixEnabled(false);
+    const QString text = m_name + (m_forceTransitionTrack ? "|>" : QString());
+
+    // Draw clip name
+    if (isSelected() || (parentItem() && parentItem()->isSelected())) {
+        framePen.setColor(Qt::red);
+        framePen.setWidthF(2.0);
+    }
+    else {
+        framePen.setColor(brush().color().darker());
+    }
 
-int Transition::transitionEndTrack() const {
-    if (!m_singleClip) return m_secondClip->track();
-    return m_referenceClip->track() + 1;
-    //TODO: calculate next video track
-}
+    const QRectF txtBounding = painter->boundingRect(mapped, Qt::AlignHCenter | Qt::AlignVCenter, ' ' + text + ' ');
+    painter->setBrush(framePen.color());
+    painter->setPen(Qt::NoPen);
+    painter->drawRoundedRect(txtBounding, 3, 3);
+    painter->setBrush(QBrush(Qt::NoBrush));
 
-GenTime Transition::transitionDuration() const {
-    return transitionEndTime() - transitionStartTime();
-}
+    painter->setPen(Qt::white);
+    painter->drawText(txtBounding, Qt::AlignCenter, text);
 
-GenTime Transition::transitionStartTime() const {
-    if (!m_singleClip) {
-        GenTime startb = m_secondClip->startPos();
-        GenTime starta = m_referenceClip->startPos();
-        if (startb > m_referenceClip->endPos()) return m_referenceClip->endPos() - GenTime(0.12);
-        if (startb > starta)
-            return startb;
-        return starta;
-    } else return m_referenceClip->startPos() + m_transitionStart;
+    // Draw frame
+    painter->setPen(framePen);
+    painter->setClipping(false);
+    painter->drawRect(mapped.adjusted(0, 0, -0.5, -0.5));
 }
 
+int Transition::type() const
+{
+    return TRANSITIONWIDGET;
+}
 
-GenTime Transition::transitionEndTime() const {
-    if (!m_singleClip) {
-        GenTime endb = m_secondClip->endPos();
-        GenTime enda = m_referenceClip->endPos();
-        if (m_secondClip->startPos() > enda) return enda;
-        if (endb < m_referenceClip->startPos()) return m_referenceClip->startPos() + GenTime(0.12);
-        else if (endb > enda) return enda;
-        else return endb;
-    } else {
-        if (m_transitionStart + m_transitionDuration > m_referenceClip->duration())
-            return m_referenceClip->endPos();
-        return m_referenceClip->startPos() + m_transitionStart + m_transitionDuration;
+//virtual
+QVariant Transition::itemChange(GraphicsItemChange change, const QVariant &value)
+{
+    if (change == QGraphicsItem::ItemSelectedChange) {
+        if (value.toBool()) setZValue(10);
+        else setZValue(3);
     }
-}
-void Transition::paint(QPainter *painter,
-                       const QStyleOptionGraphicsItem *option,
-                       QWidget *widget) {
-    QRect rectInView;//this is the rect that is visible by the user
-    if (scene()->views().size() > 0) {
-        rectInView = scene()->views()[0]->viewport()->rect();
-        rectInView.moveTo(scene()->views()[0]->horizontalScrollBar()->value(), scene()->views()[0]->verticalScrollBar()->value());
-        rectInView.adjust(-10, -10, 10, 10);//make view rect 10 pixel greater on each site, or repaint after scroll event
-        //kDebug() << scene()->views()[0]->viewport()->rect() << " " <<  scene()->views()[0]->horizontalScrollBar()->value();
+    if (change == ItemPositionChange && scene()) {
+        // calculate new position.
+        QPointF newPos = value.toPointF();
+        int xpos = projectScene()->getSnapPointForPos((int) newPos.x(), KdenliveSettings::snaptopoints());
+        xpos = qMax(xpos, 0);
+        newPos.setX(xpos);
+        int newTrack = newPos.y() / KdenliveSettings::trackheight();
+        newTrack = qMin(newTrack, projectScene()->tracksCount() - 1);
+        newTrack = qMax(newTrack, 0);
+        newPos.setY((int)(newTrack * KdenliveSettings::trackheight() + itemOffset() + 1));
+        // Only one clip is moving
+        QRectF sceneShape = rect();
+        sceneShape.translate(newPos);
+        QList<QGraphicsItem*> items;
+        // TODO: manage transitions in OVERWRITE MODE
+        //if (projectScene()->editMode() == NORMALEDIT)
+        items = scene()->items(sceneShape, Qt::IntersectsItemShape);
+        items.removeAll(this);
+
+        bool forwardMove = newPos.x() > pos().x();
+        int offset = 0;
+        if (!items.isEmpty()) {
+            for (int i = 0; i < items.count(); i++) {
+                if (!items.at(i)->isEnabled()) continue;
+                if (items.at(i)->type() == type()) {
+                    // Collision!
+                    QPointF otherPos = items.at(i)->pos();
+                    if ((int) otherPos.y() != (int) pos().y()) {
+                        return pos();
+                    }
+                    if (forwardMove) {
+                        offset = qMax(offset, (int)(newPos.x() - (static_cast < AbstractClipItem* >(items.at(i))->startPos() - cropDuration()).frames(m_fps)));
+                    } else {
+                        offset = qMax(offset, (int)((static_cast < AbstractClipItem* >(items.at(i))->endPos().frames(m_fps)) - newPos.x()));
+                    }
+
+                    if (offset > 0) {
+                        if (forwardMove) {
+                            sceneShape.translate(QPointF(-offset, 0));
+                            newPos.setX(newPos.x() - offset);
+                        } else {
+                            sceneShape.translate(QPointF(offset, 0));
+                            newPos.setX(newPos.x() + offset);
+                        }
+                        QList<QGraphicsItem*> subitems = scene()->items(sceneShape, Qt::IntersectsItemShape);
+                        subitems.removeAll(this);
+                        for (int j = 0; j < subitems.count(); j++) {
+                            if (!subitems.at(j)->isEnabled()) continue;
+                            if (subitems.at(j)->type() == type()) {
+                                // move was not successful, revert to previous pos
+                                m_info.startPos = GenTime((int) pos().x(), m_fps);
+                                return pos();
+                            }
+                        }
+                    }
+
+                    m_info.track = newTrack;
+                    m_info.startPos = GenTime((int) newPos.x(), m_fps);
+
+                    return newPos;
+                }
+            }
+        }
+       
+        m_info.track = newTrack;
+        m_info.startPos = GenTime((int) newPos.x(), m_fps);
+        //kDebug()<<"// ITEM NEW POS: "<<newPos.x()<<", mapped: "<<mapToScene(newPos.x(), 0).x();
+        return newPos;
     }
-    if (rectInView.isNull())
-        return;
-    QPainterPath clippath;
-    clippath.addRect(rectInView);
-    QRectF br = rect();
-    QPainterPath roundRectPathUpper, roundRectPathLower;
-    double roundingY = 20;
-    double roundingX = 20;
-    double offset = 1;
-    painter->setClipRect(option->exposedRect);
-    if (roundingX > br.width() / 2) roundingX = br.width() / 2;
-
-    int br_endx = (int)(br.x() + br .width() - offset);
-    int br_startx = (int)(br.x() + offset);
-    int br_starty = (int)(br.y());
-    int br_halfy = (int)(br.y() + br.height() / 2 - offset);
-    int br_endy = (int)(br.y() + br.height());
-
-
-    // build path around clip
-    roundRectPathUpper.moveTo(br_endx , br_halfy);
-    roundRectPathUpper.arcTo(br_endx - roundingX  , br_starty , roundingX, roundingY, 0.0, 90.0);
-    roundRectPathUpper.lineTo(br_startx + roundingX , br_starty);
-    roundRectPathUpper.arcTo(br_startx , br_starty , roundingX, roundingY, 90.0, 90.0);
-    roundRectPathUpper.lineTo(br_startx, br_halfy);
-
-    roundRectPathLower.moveTo(br_startx , br_halfy);
-    roundRectPathLower.arcTo(br_startx , br_endy - roundingY , roundingX, roundingY, 180.0, 90.0);
-    roundRectPathLower.lineTo(br_endx - roundingX , br_endy);
-    roundRectPathLower.arcTo(br_endx - roundingX , br_endy - roundingY, roundingX, roundingY, 270.0, 90.0);
-    roundRectPathLower.lineTo(br_endx , br_halfy);
-
-    QPainterPath resultClipPath = roundRectPathUpper.united(roundRectPathLower);
-
-    painter->setClipPath(resultClipPath.intersected(clippath), Qt::IntersectClip);
-    //painter->fillPath(roundRectPath, brush()); //, QBrush(QColor(Qt::red)));
-    painter->fillRect(br.intersected(rectInView), QBrush(QColor(200, 200, 0, 160)));
-    painter->setClipRect(option->exposedRect);
-    painter->drawPath(resultClipPath.intersected(clippath));
+    return QGraphicsItem::itemChange(change, value);
 }
 
-int Transition::type() const {
-    return 70001;
-}
-OPERATIONTYPE Transition::operationMode(QPointF pos, double scale) {
-    if (abs((int)(pos.x() - rect().x())) < 6) return RESIZESTART;
-    else if (abs((int)(pos.x() - (rect().x() + rect().width()))) < 6) return RESIZEEND;
+
+OPERATIONTYPE Transition::operationMode(QPointF pos)
+{
+    if (isItemLocked()) return NONE;
+
+    const double scale = projectScene()->scale().x();
+    double maximumOffset = 6 / scale;
+
+    QRectF rect = sceneBoundingRect();
+    if (qAbs((int)(pos.x() - rect.x())) < maximumOffset) return RESIZESTART;
+    else if (qAbs((int)(pos.x() - (rect.right()))) < maximumOffset) return RESIZEEND;
     return MOVE;
 }
-void Transition::resizeTransitionStart(GenTime time) {
-    if (!m_singleClip) return; //cannot resize automatic transitions
-    if (time < m_referenceClip->startPos()) time = m_referenceClip->startPos();
-    // Transitions shouldn't be shorter than 3 frames, about 0.12 seconds
-    if (transitionEndTime().ms() - time.ms() < 120.0) time = transitionEndTime() - GenTime(0.12);
-    m_transitionDuration = m_transitionDuration - (time - m_referenceClip->startPos() - m_transitionStart);
-    m_transitionStart = time - m_referenceClip->startPos();
-}
 
-void Transition::resizeTransitionEnd(GenTime time) {
-    if (!m_singleClip) return; //cannot resize automatic transitions
-    if (time > m_referenceClip->endPos()) time = m_referenceClip->endPos();
-    // Transitions shouldn't be shorter than 3 frames, about 0.12 seconds
-    if (time.ms() - transitionStartTime().ms() < 120.0) time = transitionStartTime() + GenTime(0.12);
-    m_transitionDuration = time - (m_referenceClip->startPos() + m_transitionStart);
+int Transition::itemHeight()
+{
+    return (int) (KdenliveSettings::trackheight() / 3 * 2 - 1);
 }
 
-void Transition::moveTransition(GenTime time) {
-    if (!m_singleClip) return; //cannot move automatic transitions
-    if (m_transitionStart + time < GenTime(0.0)) m_transitionStart = GenTime(0.0);
-    else if (m_transitionStart + time > m_referenceClip->duration() - m_transitionDuration)
-        m_transitionStart = m_referenceClip->duration() - m_transitionDuration;
-    else m_transitionStart = m_transitionStart + time;
-    if (m_transitionStart < GenTime(0.0)) m_transitionStart = GenTime(0.0);
+int Transition::itemOffset()
+{
+    return (int) (KdenliveSettings::trackheight() / 3 * 2);
 }
 
-bool Transition::hasClip(const ClipItem * clip) const {
+bool Transition::hasClip(const ClipItem * clip) const
+{
     if (clip == m_secondClip) return true;
     return false;
 }
 
-bool Transition::belongsToClip(const ClipItem * clip) const {
+bool Transition::belongsToClip(const ClipItem * clip) const
+{
     if (clip == m_referenceClip) return true;
     return false;
 }
 
+/*
 Transition *Transition::clone() {
-    return new Transition::Transition(rect(), m_referenceClip, this->toXML() , m_fps);
-    /*if (m_singleClip || m_secondClip == 0)
-        return new Transition::Transition(m_referenceClip);
-    else
-        //return new Transition::Transition(m_referenceClip, m_secondClip);
-    return new Transition::Transition(m_referenceClip, this->toXML());
-    */
-}
+    return new Transition::Transition(rect(), m_referenceClip, toXML() , m_fps);
+}*/
 
+/*
 Transition *Transition::reparent(ClipItem * clip) {
-    return new Transition::Transition(rect(), clip, this->toXML(), m_fps, m_referenceClip->startPos());
-}
+    return new Transition::Transition(rect(), clip, toXML(), m_fps, m_referenceClip->startPos());
+}*/
 
-bool Transition::isValid() const {
-    return (m_transitionDuration != GenTime());
+bool Transition::isValid() const
+{
+    return true; //(m_transitionDuration != GenTime());
 }
 
-const ClipItem *Transition::referencedClip() const {
+const ClipItem *Transition::referencedClip() const
+{
     return m_referenceClip;
 }
 
-QDomElement Transition::toXML() {
-    QDomDocument doc;
-    QDomElement effect = doc.createElement("ktransition");
-    effect.setAttribute("type", transitionType());
-    effect.setAttribute("inverted", invertTransition());
-    effect.setAttribute("transition_track", m_transitionTrack);
-    effect.setAttribute("start", transitionStartTime().frames(m_referenceClip->fps()));
-    effect.setAttribute("end", transitionEndTime().frames(m_referenceClip->fps()));
+QDomElement Transition::toXML()
+{
+    m_parameters.setAttribute("type", transitionTag());
+    //m_transitionParameters.setAttribute("inverted", invertTransition());
+    m_parameters.setAttribute("transition_atrack", track());
+    m_parameters.setAttribute("transition_btrack", m_transitionTrack);
+    m_parameters.setAttribute("start", startPos().frames(m_fps));
+    m_parameters.setAttribute("end", endPos().frames(m_fps));
+    m_parameters.setAttribute("force_track", m_forceTransitionTrack);
+    m_parameters.setAttribute("automatic", m_automaticTransition);
 
     if (m_secondClip) {
-        effect.setAttribute("clipb_starttime", m_secondClip->startPos().frames(m_referenceClip->fps()));
-        effect.setAttribute("clipb_track", transitionEndTrack());
+        m_parameters.setAttribute("clipb_starttime", m_secondClip->startPos().frames(m_referenceClip->fps()));
+        m_parameters.setAttribute("clipb_track", transitionEndTrack());
     }
+    return m_parameters.cloneNode().toElement();
+}
 
-
-    QMap<QString, QString>::Iterator it;
-    for (it = m_transitionParameters.begin(); it != m_transitionParameters.end(); ++it) {
-        QDomElement param = doc.createElement(it.key());
-        param.setAttribute("value", it.value());
-        effect.appendChild(param);
+bool Transition::hasGeometry()
+{
+    QDomNodeList namenode = m_parameters.elementsByTagName("parameter");
+    for (int i = 0; i < namenode.count() ; i++) {
+        QDomElement pa = namenode.item(i).toElement();
+        if (pa.attribute("type") == "geometry") return true;
     }
+    return false;
+}
+
+int Transition::defaultZValue() const
+{
+    return 3;
+}
 
-    return effect;
+bool Transition::updateKeyframes()
+{
+    QString keyframes;
+    QDomElement pa;
+    bool modified = false;
+    QDomNodeList namenode = m_parameters.elementsByTagName("parameter");
+    for (int i = 0; i < namenode.count() ; i++) {
+        pa = namenode.item(i).toElement();
+        if (pa.attribute("type") == "geometry") {
+            keyframes = pa.attribute("value");
+            break;
+        }
+    }
+    if (keyframes.isEmpty()) return false;
+    int duration = cropDuration().frames(m_fps) - 1;
+    QStringList values = keyframes.split(";");
+    int frame;
+    int i = 0;
+    foreach(const QString &pos, values) {
+        if (!pos.contains('=')) {
+            i++;
+            continue;
+        }
+        frame = pos.section('=', 0, 0).toInt();
+        if (frame > duration) {
+            modified = true;
+            break;
+        }
+        i++;
+    }
+    if (modified) {
+        if (i > 0) {
+            // Check if there is a keyframe at transition end
+            QString prev = values.at(i-1);
+            bool done = false;
+            if (prev.contains('=')) {
+                int previousKeyframe = prev.section('=', 0, 0).toInt();
+                if (previousKeyframe == duration) {
+                    // Remove the last keyframes
+                    while (values.count() > i) {
+                        values.removeLast();
+                    }
+                    done = true;
+                }
+            }
+            if (!done) {
+                // Add new keyframe at end and remove last keyframes
+                QString last = values.at(i);
+                last = QString::number(duration) + '=' + last.section('=', 1);
+                values[i] = last;
+                while (values.count() > (i + 1)) {
+                    values.removeLast();
+                }
+            }
+        }
+        pa.setAttribute("value", values.join(";"));
+    }
+    
+    return true;
 }