]> git.sesse.net Git - kdenlive/blobdiff - src/clipitem.cpp
Fix several problems with clip crop start, fix undo transition deletion, set monitor...
[kdenlive] / src / clipitem.cpp
index 8f9c346495993ea86c6f67a92f6010eafdc6875f..ef46cbaebe92f8c1b9f071abecd134bdf9645d44 100644 (file)
 
 #include <KDebug>
 
-#include <mlt++/Mlt.h>
-
 #include "clipitem.h"
 #include "customtrackview.h"
 #include "renderer.h"
+#include "docclipbase.h"
+#include "transition.h"
 #include "events.h"
 #include "kdenlivesettings.h"
+#include "kthumb.h"
+
+ClipItem::ClipItem(DocClipBase *clip, ItemInfo info, double scale, double fps)
+        : AbstractClipItem(info, QRectF(), fps), m_clip(clip), m_resizeMode(NONE), m_grabPoint(0), m_maxTrack(0), m_hasThumbs(false), startThumbTimer(NULL), endThumbTimer(NULL), m_effectsCounter(1), audioThumbWasDrawn(false), m_opacity(1.0), m_timeLine(0), m_thumbsRequested(0), m_startFade(0), m_endFade(0), m_hover(false), m_selectedEffect(-1) {
+    QRectF rect((double) info.startPos.frames(fps) * scale, (double)(info.track * KdenliveSettings::trackheight() + 1), (double)(info.endPos - info.startPos).frames(fps) * scale, (double)(KdenliveSettings::trackheight() - 1));
+    setRect(rect);
 
-ClipItem::ClipItem(DocClipBase *clip, int track, int startpos, const QRectF & rect, int duration)
-        : QGraphicsRectItem(rect), m_clip(clip), m_resizeMode(NONE), m_grabPoint(0), m_maxTrack(0), m_track(track), m_startPos(startpos), m_hasThumbs(false), startThumbTimer(NULL), endThumbTimer(NULL), m_startFade(0), m_endFade(0), m_effectsCounter(1), audioThumbWasDrawn(false), m_opacity(1.0), m_timeLine(0), m_thumbsRequested(0) {
-    //setToolTip(name);
-    kDebug() << "*******  CREATING NEW TML CLIP, DUR: " << duration;
-    m_xml = clip->toXML();
     m_clipName = clip->name();
     m_producer = clip->getId();
     m_clipType = clip->clipType();
-    m_cropStart = 0;
-    m_maxDuration = duration;
-    if (duration != -1) m_cropDuration = duration;
-    else m_cropDuration = m_maxDuration;
+    m_cropStart = info.cropStart;
+    m_maxDuration = clip->maxDuration();
     setAcceptDrops(true);
     audioThumbReady = clip->audioThumbCreated();
+
     /*
       m_cropStart = xml.attribute("in", 0).toInt();
       m_maxDuration = xml.attribute("duration", 0).toInt();
@@ -61,11 +61,12 @@ ClipItem::ClipItem(DocClipBase *clip, int track, int startpos, const QRectF & re
       else m_cropDuration = m_maxDuration;*/
 
 
-    setFlags(QGraphicsItem::ItemClipsToShape | QGraphicsItem::ItemClipsChildrenToShape | QGraphicsItem::ItemIsMovable | QGraphicsItem::ItemIsSelectable);
-    connect(this , SIGNAL(prepareAudioThumb(double, QPainterPath, int, int)) , this, SLOT(slotPrepareAudioThumb(double, QPainterPath, int, int)));
+    setFlags(QGraphicsItem::ItemClipsToShape | QGraphicsItem::ItemIsMovable | QGraphicsItem::ItemIsSelectable);
+    setAcceptsHoverEvents(true);
+    connect(this , SIGNAL(prepareAudioThumb(double, QPainterPath, int, int, int)) , this, SLOT(slotPrepareAudioThumb(double, QPainterPath, int, int, int)));
 
-    setBrush(QColor(100, 100, 150));
-    if (m_clipType == VIDEO || m_clipType == AV) {
+    setBrush(QColor(141, 166, 215));
+    if (m_clipType == VIDEO || m_clipType == AV || m_clipType == SLIDESHOW) {
         m_hasThumbs = true;
         connect(this, SIGNAL(getThumb(int, int)), clip->thumbProducer(), SLOT(extractImage(int, int)));
         connect(clip->thumbProducer(), SIGNAL(thumbReady(int, QPixmap)), this, SLOT(slotThumbReady(int, QPixmap)));
@@ -78,13 +79,13 @@ ClipItem::ClipItem(DocClipBase *clip, int track, int startpos, const QRectF & re
         endThumbTimer = new QTimer(this);
         endThumbTimer->setSingleShot(true);
         connect(endThumbTimer, SIGNAL(timeout()), this, SLOT(slotGetEndThumb()));
-
     } else if (m_clipType == COLOR) {
-        QString colour = m_xml.attribute("colour");
+        QString colour = clip->getProperty("colour");
         colour = colour.replace(0, 2, "#");
         setBrush(QColor(colour.left(7)));
-    } else if (m_clipType == IMAGE) {
-        m_startPix = KThumb::getImage(KUrl(m_xml.attribute("resource")), 50 * KdenliveSettings::project_display_ratio(), 50);
+    } else if (m_clipType == IMAGE || m_clipType == TEXT) {
+        m_startPix = KThumb::getImage(KUrl(clip->getProperty("resource")), (int)(50 * KdenliveSettings::project_display_ratio()), 50);
+        m_endPix = m_startPix;
     } else if (m_clipType == AUDIO) {
         connect(clip, SIGNAL(gotAudioData()), this, SLOT(slotGotAudioData()));
     }
@@ -96,180 +97,344 @@ ClipItem::~ClipItem() {
     if (endThumbTimer) delete endThumbTimer;
 }
 
+ClipItem *ClipItem::clone(double scale) const {
+    ClipItem *duplicate = new ClipItem(m_clip, info(), scale, m_fps);
+    duplicate->setEffectList(m_effectList);
+    return duplicate;
+}
+
+void ClipItem::setEffectList(const EffectsList effectList) {
+    //m_effectList = effectList.clone();
+}
+
+int ClipItem::selectedEffectIndex() const {
+    return m_selectedEffect;
+}
+
+void ClipItem::initEffect(QDomElement effect) {
+    // the kdenlive_ix int is used to identify an effect in mlt's playlist, should
+    // not be changed
+    if (effect.attribute("kdenlive_ix").toInt() == 0)
+        effect.setAttribute("kdenlive_ix", QString::number(effectsCounter()));
+    // init keyframes if required
+    QDomNodeList params = effect.elementsByTagName("parameter");
+    for (int i = 0; i < params.count(); i++) {
+        QDomElement e = params.item(i).toElement();
+        if (!e.isNull() && e.attribute("type") == "keyframe") {
+            QString def = e.attribute("default");
+            // Effect has a keyframe type parameter, we need to set the values
+            if (e.attribute("keyframes").isEmpty()) {
+                e.setAttribute("keyframes", QString::number(m_cropStart.frames(m_fps)) + ":" + def + ";" + QString::number((m_cropStart + m_cropDuration).frames(m_fps)) + ":" + def);
+                //kDebug() << "///// EFFECT KEYFRAMES INITED: " << e.attribute("keyframes");
+                break;
+            }
+        }
+    }
+}
+
+void ClipItem::setKeyframes(const int ix, const QString keyframes) {
+    QDomElement effect = effectAt(ix);
+    if (effect.attribute("disabled") == "1") return;
+    QDomNodeList params = effect.elementsByTagName("parameter");
+    for (int i = 0; i < params.count(); i++) {
+        QDomElement e = params.item(i).toElement();
+        if (!e.isNull() && e.attribute("type") == "keyframe") {
+            e.setAttribute("keyframes", keyframes);
+            if (ix == m_selectedEffect) {
+                m_keyframes.clear();
+                double max = e.attribute("max").toDouble();
+                double min = e.attribute("min").toDouble();
+                m_keyframeFactor = 100.0 / (max - min);
+                m_keyframeDefault = e.attribute("default").toDouble();
+                // parse keyframes
+                QStringList keyframes = e.attribute("keyframes").split(";", QString::SkipEmptyParts);
+                foreach(QString str, keyframes) {
+                    int pos = str.section(":", 0, 0).toInt();
+                    double val = str.section(":", 1, 1).toDouble();
+                    m_keyframes[pos] = val;
+                }
+                update();
+                return;
+            }
+            break;
+        }
+    }
+}
+
+
+void ClipItem::setSelectedEffect(const int ix) {
+    m_selectedEffect = ix;
+    QDomElement effect = effectAt(m_selectedEffect);
+    QDomNodeList params = effect.elementsByTagName("parameter");
+    if (effect.attribute("disabled") != "1")
+        for (int i = 0; i < params.count(); i++) {
+            QDomElement e = params.item(i).toElement();
+            if (!e.isNull() && e.attribute("type") == "keyframe") {
+                m_keyframes.clear();
+                double max = e.attribute("max").toDouble();
+                double min = e.attribute("min").toDouble();
+                m_keyframeFactor = 100.0 / (max - min);
+                m_keyframeDefault = e.attribute("default").toDouble();
+                // parse keyframes
+                QStringList keyframes = e.attribute("keyframes").split(";", QString::SkipEmptyParts);
+                foreach(QString str, keyframes) {
+                    int pos = str.section(":", 0, 0).toInt();
+                    double val = str.section(":", 1, 1).toDouble();
+                    m_keyframes[pos] = val;
+                }
+                update();
+                return;
+            }
+        }
+    if (!m_keyframes.isEmpty()) {
+        m_keyframes.clear();
+        update();
+    }
+}
+
+QString ClipItem::keyframes(const int index) {
+    QString result;
+    QDomElement effect = effectAt(index);
+    QDomNodeList params = effect.elementsByTagName("parameter");
+
+    for (int i = 0; i < params.count(); i++) {
+        QDomElement e = params.item(i).toElement();
+        if (!e.isNull() && e.attribute("type") == "keyframe") {
+            result = e.attribute("keyframes");
+            break;
+        }
+    }
+    return result;
+}
+
+void ClipItem::updateKeyframeEffect() {
+    // regenerate xml parameter from the clip keyframes
+    QDomElement effect = effectAt(m_selectedEffect);
+    if (effect.attribute("disabled") == "1") return;
+    QDomNodeList params = effect.elementsByTagName("parameter");
+
+    for (int i = 0; i < params.count(); i++) {
+        QDomElement e = params.item(i).toElement();
+        if (!e.isNull() && e.attribute("type") == "keyframe") {
+            QString keyframes;
+            if (m_keyframes.count() > 1) {
+                QMap<int, double>::const_iterator i = m_keyframes.constBegin();
+                double x1;
+                double y1;
+                while (i != m_keyframes.constEnd()) {
+                    keyframes.append(QString::number(i.key()) + ":" + QString::number(i.value()) + ";");
+                    ++i;
+                }
+            }
+            // Effect has a keyframe type parameter, we need to set the values
+            //kDebug() << ":::::::::::::::   SETTING EFFECT KEYFRAMES: " << keyframes;
+            e.setAttribute("keyframes", keyframes);
+            break;
+        }
+    }
+}
+
+QDomElement ClipItem::selectedEffect() {
+    if (m_selectedEffect == -1 || m_effectList.isEmpty()) return QDomElement();
+    return effectAt(m_selectedEffect);
+}
+
+void ClipItem::resetThumbs() {
+    slotFetchThumbs();
+    audioThumbCachePic.clear();
+}
+
+
+void ClipItem::refreshClip() {
+    m_maxDuration = m_clip->maxDuration();
+    if (m_clipType == VIDEO || m_clipType == AV || m_clipType == SLIDESHOW) slotFetchThumbs();
+    else if (m_clipType == COLOR) {
+        QString colour = m_clip->getProperty("colour");
+        colour = colour.replace(0, 2, "#");
+        setBrush(QColor(colour.left(7)));
+    } else if (m_clipType == IMAGE || m_clipType == TEXT) {
+        m_startPix = KThumb::getImage(KUrl(m_clip->getProperty("resource")), (int)(50 * KdenliveSettings::project_display_ratio()), 50);
+        m_endPix = m_startPix;
+    }
+}
+
 void ClipItem::slotFetchThumbs() {
     m_thumbsRequested += 2;
-    emit getThumb(m_cropStart, m_cropStart + m_cropDuration);
+    emit getThumb((int)m_cropStart.frames(m_fps), (int)(m_cropStart + m_cropDuration).frames(m_fps));
 }
 
 void ClipItem::slotGetStartThumb() {
     m_thumbsRequested++;
-    emit getThumb(m_cropStart, -1);
+    emit getThumb((int)m_cropStart.frames(m_fps), -1);
 }
 
 void ClipItem::slotGetEndThumb() {
     m_thumbsRequested++;
-    emit getThumb(-1, m_cropStart + m_cropDuration);
+    emit getThumb(-1, (int)(m_cropStart + m_cropDuration).frames(m_fps));
 }
 
 void ClipItem::slotThumbReady(int frame, QPixmap pix) {
     if (m_thumbsRequested == 0) return;
-    if (frame == m_cropStart) m_startPix = pix;
-    else m_endPix = pix;
-    update();
+    if (frame == m_cropStart.frames(m_fps)) {
+        m_startPix = pix;
+        QRectF r = boundingRect();
+        r.setRight(pix.width() + 2);
+        update(r);
+    } else {
+        m_endPix = pix;
+        QRectF r = boundingRect();
+        r.setLeft(r.right() - pix.width() - 2);
+        update(r);
+    }
     m_thumbsRequested--;
 }
 
 void ClipItem::slotGotAudioData() {
     audioThumbReady = true;
-    update();
+    if (m_clipType == AV) {
+        QRectF r = boundingRect();
+        r.setTop(r.top() + r.height() / 2 - 1);
+        update(r);
+    } else update();
 }
 
 int ClipItem::type() const {
-    return 70000;
+    return AVWIDGET;
 }
 
-DocClipBase *ClipItem::baseClip() {
+DocClipBase *ClipItem::baseClip() const {
     return m_clip;
 }
 
 QDomElement ClipItem::xml() const {
-    return m_xml;
+    return m_clip->toXML();
 }
 
-int ClipItem::clipType() {
+int ClipItem::clipType() const {
     return m_clipType;
 }
 
-QString ClipItem::clipName() {
+QString ClipItem::clipName() const {
     return m_clipName;
 }
 
-int ClipItem::clipProducer() {
+int ClipItem::clipProducer() const {
     return m_producer;
 }
 
-int ClipItem::maxDuration() {
-    return m_maxDuration;
-}
-
-int ClipItem::duration() {
-    return m_cropDuration;
-}
-
-int ClipItem::startPos() {
-    return m_startPos;
-}
-
-int ClipItem::cropStart() {
-    return m_cropStart;
-}
-
-int ClipItem::endPos() {
-    return m_startPos + m_cropDuration;
-}
-
 void ClipItem::flashClip() {
     if (m_timeLine == 0) {
         m_timeLine = new QTimeLine(750, this);
+        m_timeLine->setCurveShape(QTimeLine::EaseInOutCurve);
         connect(m_timeLine, SIGNAL(valueChanged(qreal)), this, SLOT(animate(qreal)));
     }
     m_timeLine->start();
 }
 
 void ClipItem::animate(qreal value) {
-    m_opacity = value;
-    update();
+    QRectF r = boundingRect();
+    r.setHeight(20);
+    update(r);
 }
 
 // virtual
 void ClipItem::paint(QPainter *painter,
                      const QStyleOptionGraphicsItem *option,
-                     QWidget *widget) {
+                     QWidget *) {
     painter->setOpacity(m_opacity);
     QBrush paintColor = brush();
-    if (isSelected()) paintColor = QBrush(QColor(150, 50, 100));
+    if (isSelected()) paintColor = QBrush(QColor(79, 93, 121));
     QRectF br = rect();
-    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 (rectInView.isNull())
-        return;
-    QPainterPath clippath;
-    clippath.addRect(rectInView);
-    int startpixel = rectInView.x() - rect().x();//start and endpixel that is viewable from rect()
+    double scale = br.width() / m_cropDuration.frames(m_fps);
+
+    // kDebug()<<"///   EXPOSED RECT: "<<option->exposedRect.x()<<" X "<<option->exposedRect.right();
+
+    int startpixel = (int)option->exposedRect.x() - rect().x();
+
     if (startpixel < 0)
         startpixel = 0;
-    int endpixel = rectInView.width() + rectInView.x();
+    int endpixel = (int)option->exposedRect.right();
     if (endpixel < 0)
         endpixel = 0;
 
-    painter->setRenderHints(QPainter::Antialiasing);
-    QPainterPath roundRectPathUpper, roundRectPathLower;
-    double roundingY = 20;
-    double roundingX = 20;
-    double offset = 1;
+    //painter->setRenderHints(QPainter::Antialiasing);
+
+    QPainterPath roundRectPathUpper = upperRectPart(br), roundRectPathLower = lowerRectPart(br);
     painter->setClipRect(option->exposedRect);
-    if (roundingX > br.width() / 2) roundingX = br.width() / 2;
-    //kDebug()<<"-----PAINTING, SCAL: "<<scale<<", height: "<<br.height();
-    roundRectPathUpper.moveTo(br.x() + br .width() - offset, br.y() + br.height() / 2 - offset);
-    roundRectPathUpper.arcTo(br.x() + br .width() - roundingX - offset, br.y(), roundingX, roundingY, 0.0, 90.0);
-    roundRectPathUpper.lineTo(br.x() + roundingX, br.y());
-    roundRectPathUpper.arcTo(br.x() + offset, br.y(), roundingX, roundingY, 90.0, 90.0);
-    roundRectPathUpper.lineTo(br.x() + offset, br.y() + br.height() / 2 - offset);
-    roundRectPathUpper.closeSubpath();
-
-    roundRectPathLower.moveTo(br.x() + offset, br.y() + br.height() / 2 - offset);
-    roundRectPathLower.arcTo(br.x() + offset, br.y() + br.height() - roundingY - offset, roundingX, roundingY, 180.0, 90.0);
-    roundRectPathLower.lineTo(br.x() + br .width() - roundingX, br.y() + br.height() - offset);
-    roundRectPathLower.arcTo(br.x() + br .width() - roundingX - offset, br.y() + br.height() - roundingY - offset, roundingX, roundingY, 270.0, 90.0);
-    roundRectPathLower.lineTo(br.x() + br .width() - offset, br.y() + br.height() / 2 - offset);
-    roundRectPathLower.closeSubpath();
-
-    painter->setClipPath(roundRectPathUpper.united(roundRectPathLower).intersected(clippath), Qt::IntersectClip);
-    //painter->fillPath(roundRectPath, brush()); //, QBrush(QColor(Qt::red)));
-    painter->fillRect(br.intersected(rectInView), paintColor);
-    //painter->fillRect(QRectF(br.x() + br.width() - m_endPix.width(), br.y(), m_endPix.width(), br.height()), QBrush(QColor(Qt::black)));
 
+    // build path around clip
+    QPainterPath resultClipPath = roundRectPathUpper.united(roundRectPathLower);
+    painter->fillPath(resultClipPath, paintColor);
+
+    painter->setClipPath(resultClipPath, Qt::IntersectClip);
     // draw thumbnails
     if (!m_startPix.isNull() && KdenliveSettings::videothumbnails()) {
         if (m_clipType == IMAGE) {
-            painter->drawPixmap(QPointF(br.x() + br.width() - m_startPix.width(), br.y()), m_startPix);
-            QLineF l(br.x() + br.width() - m_startPix.width(), br.y(), br.x() + br.width() - m_startPix.width(), br.y() + br.height());
+            painter->drawPixmap(QPointF(br.right() - m_startPix.width(), br.y()), m_startPix);
+            QLine l(br.right() - m_startPix.width(), br.y(), br.right() - m_startPix.width(), br.y() + br.height());
             painter->drawLine(l);
         } else {
-            painter->drawPixmap(QPointF(br.x() + br.width() - m_endPix.width(), br.y()), m_endPix);
-            QLineF l(br.x() + br.width() - m_endPix.width(), br.y(), br.x() + br.width() - m_endPix.width(), br.y() + br.height());
+            painter->drawPixmap(QPointF(br.right() - m_endPix.width(), br.y()), m_endPix);
+            QLine l(br.right() - m_endPix.width(), br.y(), br.right() - m_endPix.width(), br.y() + br.height());
             painter->drawLine(l);
         }
 
         painter->drawPixmap(QPointF(br.x(), br.y()), m_startPix);
-        QLineF l2(br.x() + m_startPix.width(), br.y(), br.x() + m_startPix.width(), br.y() + br.height());
+        QLine l2(br.x() + m_startPix.width(), br.y(), br.x() + m_startPix.width(), br.y() + br.height());
         painter->drawLine(l2);
     }
-    if ((m_clipType == AV || m_clipType == AUDIO) && audioThumbReady && KdenliveSettings::audiothumbnails()) {
 
-        QPainterPath path = m_clipType == AV ? roundRectPathLower : roundRectPathUpper.united(roundRectPathLower);
-        painter->fillPath(path, QBrush(QColor(200, 200, 200, 127)));
+    // draw audio thumbnails
+    if (KdenliveSettings::audiothumbnails() && ((m_clipType == AV && option->exposedRect.bottom() > (br.y() + br.height() / 2)) || m_clipType == AUDIO) && audioThumbReady) {
 
-        int channels = 2;
-        double pixelForOneFrame = (double)br.width() / duration();
-        if (pixelForOneFrame != framePixelWidth)
-            audioThumbCachePic.clear();
-        emit prepareAudioThumb(pixelForOneFrame, path, startpixel, endpixel + 200);//200 more for less missing parts before repaint after scrolling
+        QPainterPath path = m_clipType == AV ? roundRectPathLower : resultClipPath;
+        if (m_clipType == AV) painter->fillPath(path, QBrush(QColor(200, 200, 200, 140)));
 
+        int channels = baseClip()->getProperty("channels").toInt();
+        if (scale != framePixelWidth)
+            audioThumbCachePic.clear();
+        emit prepareAudioThumb(scale, path, startpixel, endpixel + 200, channels);//200 more for less missing parts before repaint after scrolling
+        int cropLeft = (int)((m_cropStart).frames(m_fps) * scale);
         for (int startCache = startpixel - startpixel % 100; startCache < endpixel + 300;startCache += 100) {
             if (audioThumbCachePic.contains(startCache) && !audioThumbCachePic[startCache].isNull())
-                painter->drawPixmap(path.boundingRect().x() + startCache, path.boundingRect().y(), audioThumbCachePic[startCache]);
+                painter->drawPixmap((int)(roundRectPathUpper.united(roundRectPathLower).boundingRect().x() + startCache - cropLeft), (int)(path.boundingRect().y()), audioThumbCachePic[startCache]);
         }
+    }
 
+    // draw markers
+    QList < CommentedTime > markers = baseClip()->commentedSnapMarkers();
+    QList < CommentedTime >::Iterator it = markers.begin();
+    GenTime pos;
+    double framepos;
+    const int markerwidth = 4;
+    QBrush markerBrush;
+    markerBrush = QBrush(QColor(120, 120, 0, 100));
+    QPen pen = painter->pen();
+    pen.setColor(QColor(255, 255, 255, 200));
+    pen.setStyle(Qt::DotLine);
+    painter->setPen(pen);
+    for (; it != markers.end(); ++it) {
+        pos = (*it).time() - cropStart();
+        if (pos > GenTime()) {
+            if (pos > duration()) break;
+            framepos = scale * pos.frames(m_fps);
+            QLineF l(br.x() + framepos, br.y() + 5, br.x() + framepos, br.y() + br.height() - 5);
+            painter->drawLine(l);
+            if (KdenliveSettings::showmarkers()) {
+                const QRectF txtBounding = painter->boundingRect(br.x() + framepos + 1, br.y() + 10, br.width() - framepos - 2, br.height() - 10, Qt::AlignLeft | Qt::AlignTop, " " + (*it).comment() + " ");
+                QPainterPath path;
+                path.addRoundedRect(txtBounding, 3, 3);
+                painter->fillPath(path, markerBrush);
+                painter->drawText(txtBounding, Qt::AlignCenter, (*it).comment());
+            }
+            //painter->fillRect(QRect(br.x() + framepos, br.y(), 10, br.height()), QBrush(QColor(0, 0, 0, 150)));
+        }
     }
+    pen.setColor(Qt::black);
+    pen.setStyle(Qt::SolidLine);
+    painter->setPen(pen);
 
     // draw start / end fades
-    double scale = br.width() / m_cropDuration;
     QBrush fades;
     if (isSelected()) {
         fades = QBrush(QColor(200, 50, 50, 150));
@@ -277,100 +442,187 @@ void ClipItem::paint(QPainter *painter,
 
     if (m_startFade != 0) {
         QPainterPath fadeInPath;
-        fadeInPath.moveTo(br.x() - offset, br.y());
-        fadeInPath.lineTo(br.x() - offset, br.y() + br.height());
+        fadeInPath.moveTo(br.x() , br.y());
+        fadeInPath.lineTo(br.x() , br.bottom());
         fadeInPath.lineTo(br.x() + m_startFade * scale, br.y());
         fadeInPath.closeSubpath();
-        painter->fillPath(fadeInPath, fades);
+        painter->fillPath(fadeInPath/*.intersected(resultClipPath)*/, fades);
         if (isSelected()) {
-            QLineF l(br.x() + m_startFade * scale, br.y(), br.x(), br.y() + br.height());
+            QLineF l(br.x() + m_startFade * scale, br.y(), br.x(), br.bottom());
             painter->drawLine(l);
         }
     }
     if (m_endFade != 0) {
         QPainterPath fadeOutPath;
-        fadeOutPath.moveTo(br.x() + br.width(), br.y());
-        fadeOutPath.lineTo(br.x() + br.width(), br.y() + br.height());
-        fadeOutPath.lineTo(br.x() + br.width() - m_endFade * scale, br.y());
+        fadeOutPath.moveTo(br.right(), br.y());
+        fadeOutPath.lineTo(br.right(), br.bottom());
+        fadeOutPath.lineTo(br.right() - m_endFade * scale, br.y());
         fadeOutPath.closeSubpath();
-        painter->fillPath(fadeOutPath, fades);
+        painter->fillPath(fadeOutPath/*.intersected(resultClipPath)*/, fades);
         if (isSelected()) {
-            QLineF l(br.x() + br.width() - m_endFade * scale, br.y(), br.x() + br.width(), br.y() + br.height());
+            QLineF l(br.right() - m_endFade * scale, br.y(), br.x() + br.width(), br.bottom());
             painter->drawLine(l);
         }
     }
 
-    QPen pen = painter->pen();
-    pen.setColor(Qt::white);
-    //pen.setStyle(Qt::DashDotDotLine); //Qt::DotLine);
+    // Draw effects names
+    if (!m_effectNames.isEmpty() && br.width() > 30) {
+        QRectF txtBounding = painter->boundingRect(br, Qt::AlignLeft | Qt::AlignTop, m_effectNames);
+        txtBounding.setRight(txtBounding.right() + 15);
+        painter->setPen(Qt::white);
+        QBrush markerBrush(Qt::SolidPattern);
+        if (m_timeLine && m_timeLine->state() == QTimeLine::Running) {
+            qreal value = m_timeLine->currentValue();
+            txtBounding.setWidth(txtBounding.width() * value);
+            markerBrush.setColor(QColor(50 + 200 * (1.0 - value), 50, 50, 100 + 50 * value));
+        } else markerBrush.setColor(QColor(50, 50, 50, 150));
+        QPainterPath path;
+        path.addRoundedRect(txtBounding, 4, 4);
+        painter->fillPath(path/*.intersected(resultClipPath)*/, markerBrush);
+        painter->drawText(txtBounding, Qt::AlignCenter, m_effectNames);
+        painter->setPen(Qt::black);
+    }
+
     // Draw clip name
-    QString effects = effectNames().join(" / ");
-    if (!effects.isEmpty()) {
-        painter->setPen(pen);
-        QFont font = painter->font();
-        QFont smallFont = font;
-        smallFont.setPointSize(8);
-        painter->setFont(smallFont);
-        QRectF txtBounding = painter->boundingRect(br, Qt::AlignLeft | Qt::AlignTop, " " + effects + " ");
-        painter->fillRect(txtBounding, QBrush(QColor(0, 0, 0, 150)));
-        painter->drawText(txtBounding, Qt::AlignCenter, effects);
+    QRectF txtBounding = painter->boundingRect(br, Qt::AlignHCenter | Qt::AlignTop, " " + m_clipName + " ");
+    //painter->fillRect(txtBounding, QBrush(QColor(255, 255, 255, 150)));
+    painter->setPen(QColor(0, 0, 0, 180));
+    painter->drawText(txtBounding, Qt::AlignCenter, m_clipName);
+    txtBounding.translate(QPointF(1, 1));
+    painter->setPen(QColor(255, 255, 255, 255));
+    painter->drawText(txtBounding, Qt::AlignCenter, m_clipName);
+    // draw frame around clip
+    if (isSelected()) {
+        pen.setColor(Qt::red);
+        //pen.setWidth(2);
+    } else {
         pen.setColor(Qt::black);
-        painter->setPen(pen);
-        painter->setFont(font);
+        //pen.setWidth(1);
     }
 
-    pen.setColor(Qt::red);
-    //pen.setStyle(Qt::DashDotDotLine); //Qt::DotLine);
-    if (isSelected()) painter->setPen(pen);
-    painter->setClipRect(option->exposedRect);
-    painter->drawPath(roundRectPathUpper.united(roundRectPathLower).intersected(clippath));
 
-    QRectF txtBounding = painter->boundingRect(br, Qt::AlignCenter, " " + m_clipName + " ");
-    painter->fillRect(txtBounding, QBrush(QColor(255, 255, 255, 150)));
-    painter->drawText(txtBounding, Qt::AlignCenter, m_clipName);
+    // draw effect or transition keyframes
+    if (br.width() > 20) drawKeyFrames(painter, option->exposedRect);
 
+    // draw clip border
+    painter->setClipRect(option->exposedRect);
+    painter->setPen(pen);
+    //painter->setClipRect(option->exposedRect);
+    painter->drawPath(resultClipPath);
+
+    if (m_hover && br.width() > 30) {
+        painter->setBrush(QColor(180, 180, 50, 180)); //gradient);
+
+        // draw transitions handles
+        QPainterPath transitionHandle;
+        const int handle_size = 4;
+        transitionHandle.moveTo(0, 0);
+        transitionHandle.lineTo(handle_size, handle_size);
+        transitionHandle.lineTo(handle_size * 2, 0);
+        transitionHandle.lineTo(handle_size * 3, handle_size);
+        transitionHandle.lineTo(handle_size * 2, handle_size * 2);
+        transitionHandle.lineTo(handle_size * 3, handle_size * 3);
+        transitionHandle.lineTo(0, handle_size * 3);
+        transitionHandle.closeSubpath();
+        int pointy = (int)(br.y() + br.height() / 2);
+        int pointx1 = (int)(br.x() + 10);
+        int pointx2 = (int)(br.x() + br.width() - (10 + handle_size * 3));
+#if 0
+        painter->setPen(QPen(Qt::black));
+        painter->setBrush(QBrush(QColor(50, 50, 0)));
+#else
+        /*QRadialGradient gradient(pointx1 + 5, pointy + 5 , 5, 2, 2);
+        gradient.setColorAt(0.2, Qt::white);
+        gradient.setColorAt(0.8, Qt::yellow);
+        gradient.setColorAt(1, Qt::black);*/
+
+#endif
+        painter->translate(pointx1, pointy);
+        painter->drawPath(transitionHandle); //Ellipse(0, 0 , 10, 10);
+        painter->translate(-pointx1, -pointy);
+
+        /*        QRadialGradient gradient1(pointx2 + 5, pointy + 5 , 5, 2, 2);
+                gradient1.setColorAt(0.2, Qt::white);
+                gradient1.setColorAt(0.8, Qt::yellow);
+                gradient1.setColorAt(1, Qt::black);
+                painter->setBrush(gradient1);*/
+        painter->translate(pointx2, pointy);
+        QMatrix m;
+        m.scale(-1.0, 1.0);
+        //painter->setMatrix(m);
+        painter->drawPath(transitionHandle); // Ellipse(0, 0, 10, 10);
+        //painter->setMatrix(m);
+        painter->translate(- pointx2, -pointy);
+    }
+}
 
-    //painter->fillRect(startpixel,0,startpixel+endpixel,(int)br.height(),  QBrush(QColor(255,255,255,150)));
-    //painter->fillRect(QRect(br.x(), br.y(), roundingX, roundingY), QBrush(QColor(Qt::green)));
 
-    /*QRectF recta(rect().x(), rect().y(), scale,rect().height());
-    painter->drawRect(recta);
-    painter->drawLine(rect().x() + 1, rect().y(), rect().x() + 1, rect().y() + rect().height());
-    painter->drawLine(rect().x() + rect().width(), rect().y(), rect().x() + rect().width(), rect().y() + rect().height());
-    painter->setPen(QPen(Qt::black, 1.0));
-    painter->drawLine(rect().x(), rect().y(), rect().x() + rect().width(), rect().y());
-    painter->drawLine(rect().x(), rect().y() + rect().height(), rect().x() + rect().width(), rect().y() + rect().height());*/
+OPERATIONTYPE ClipItem::operationMode(QPointF pos, double scale) {
+    if (isSelected()) {
+        m_editedKeyframe = mouseOverKeyFrames(pos);
+        if (m_editedKeyframe != -1) return KEYFRAME;
+    }
+    if (qAbs((int)(pos.x() - (rect().x() + scale * m_startFade))) < 6 && qAbs((int)(pos.y() - rect().y())) < 6) {
+        if (m_startFade == 0) setToolTip(i18n("Add audio fade"));
+        else setToolTip(i18n("Audio fade duration: %1s", GenTime(m_startFade, m_fps).seconds()));
+        return FADEIN;
+    } else if (qAbs((int)(pos.x() - rect().x())) < 6) {
+        setToolTip(i18n("Crop from start: %1s", cropStart().seconds()));
+        return RESIZESTART;
+    } else if (qAbs((int)(pos.x() - (rect().x() + rect().width() - scale * m_endFade))) < 6 && qAbs((int)(pos.y() - rect().y())) < 6) {
+        if (m_endFade == 0) setToolTip(i18n("Add audio fade"));
+        else setToolTip(i18n("Audio fade duration: %1s", GenTime(m_endFade, m_fps).seconds()));
+        return FADEOUT;
+    } else if (qAbs((int)(pos.x() - (rect().x() + rect().width()))) < 6) {
+        setToolTip(i18n("Clip duration: %1s", duration().seconds()));
+        return RESIZEEND;
+    } else if (qAbs((int)(pos.x() - (rect().x() + 16))) < 10 && qAbs((int)(pos.y() - (rect().y() + rect().height() / 2 + 5))) < 8) {
+        setToolTip(i18n("Add transition"));
+        return TRANSITIONSTART;
+    } else if (qAbs((int)(pos.x() - (rect().x() + rect().width() - 21))) < 10 && qAbs((int)(pos.y() - (rect().y() + rect().height() / 2 + 5))) < 8) {
+        setToolTip(i18n("Add transition"));
+        return TRANSITIONEND;
+    }
+    setToolTip(QString());
+    return MOVE;
+}
+
+QList <GenTime> ClipItem::snapMarkers() const {
+    QList < GenTime > snaps;
+    QList < GenTime > markers = baseClip()->snapMarkers();
+    GenTime pos;
+    double framepos;
 
-    //QGraphicsRectItem::paint(painter, option, widget);
-    //QPen pen(Qt::green, 1.0 / size.x() + 0.5);
-    //painter->setPen(pen);
-    //painter->drawLine(rect().x(), rect().y(), rect().x() + rect().width(), rect().y());
-    //kDebug()<<"ITEM REPAINT RECT: "<<boundingRect().width();
-    //painter->drawText(rect(), Qt::AlignCenter, m_name);
-    // painter->drawRect(boundingRect());
-    //painter->drawRoundRect(-10, -10, 20, 20);
-    painter->setPen(QPen(Qt::black));
-    painter->setBrush(QBrush(Qt::yellow));
-    painter->drawEllipse(br.x() + 10, br.y() + br.height() / 2 - 5 , 10, 10);
-    painter->drawEllipse(br.x() + br.width() - 20, br.y() + br.height() / 2 - 5, 10, 10);
+    for (int i = 0; i < markers.size(); i++) {
+        pos = markers.at(i) - cropStart();
+        if (pos > GenTime()) {
+            if (pos > duration()) break;
+            else snaps.append(pos + startPos());
+        }
+    }
+    return snaps;
 }
 
+QList <CommentedTime> ClipItem::commentedSnapMarkers() const {
+    QList < CommentedTime > snaps;
+    QList < CommentedTime > markers = baseClip()->commentedSnapMarkers();
+    GenTime pos;
+    double framepos;
 
-OPERATIONTYPE ClipItem::operationMode(QPointF pos, double scale) {
-    if (abs(pos.x() - (rect().x() + scale * m_startFade)) < 6 && abs(pos.y() - rect().y()) < 6) return FADEIN;
-    else if (abs(pos.x() - rect().x()) < 6) return RESIZESTART;
-    else if (abs(pos.x() - (rect().x() + rect().width() - scale * m_endFade)) < 6 && abs(pos.y() - rect().y()) < 6) return FADEOUT;
-    else if (abs(pos.x() - (rect().x() + rect().width())) < 6) return RESIZEEND;
-    else if (abs(pos.x() - (rect().x() + 10)) < 6 && abs(pos.y() - (rect().y() + rect().height() / 2 - 5)) < 6) return TRANSITIONSTART;
-    else if (abs(pos.x() - (rect().x() + rect().width() - 20)) < 6 && abs(pos.y() - (rect().y() + rect().height() / 2 - 5)) < 6) return TRANSITIONEND;
-    return MOVE;
+    for (int i = 0; i < markers.size(); i++) {
+        pos = markers.at(i).time() - cropStart();
+        if (pos > GenTime()) {
+            if (pos > duration()) break;
+            else snaps.append(CommentedTime(pos + startPos(), markers.at(i).comment()));
+        }
+    }
+    return snaps;
 }
 
-void ClipItem::slotPrepareAudioThumb(double pixelForOneFrame, QPainterPath path, int startpixel, int endpixel) {
-    int channels = 2;
+void ClipItem::slotPrepareAudioThumb(double pixelForOneFrame, QPainterPath path, int startpixel, int endpixel, int channels) {
 
     QRectF re = path.boundingRect();
-
+    //kDebug() << "// PREP AUDIO THMB FRMO : " << startpixel << ", to: " << endpixel;
     //if ( (!audioThumbWasDrawn || framePixelWidth!=pixelForOneFrame ) && !baseClip()->audioFrameChache.isEmpty()){
 
     for (int startCache = startpixel - startpixel % 100;startCache + 100 < endpixel ;startCache += 100) {
@@ -379,8 +631,8 @@ void ClipItem::slotPrepareAudioThumb(double pixelForOneFrame, QPainterPath path,
         if (framePixelWidth == pixelForOneFrame && audioThumbCachePic.contains(startCache))
             continue;
         if (audioThumbCachePic[startCache].isNull() || framePixelWidth != pixelForOneFrame) {
-            audioThumbCachePic[startCache] = QPixmap(100, re.height());
-            audioThumbCachePic[startCache].fill(QColor(200, 200, 200, 127));
+            audioThumbCachePic[startCache] = QPixmap(100, (int)(re.height()));
+            audioThumbCachePic[startCache].fill(QColor(200, 200, 200, 0));
         }
         bool fullAreaDraw = pixelForOneFrame < 10;
         QMap<int, QPainterPath > positiveChannelPaths;
@@ -401,7 +653,7 @@ void ClipItem::slotPrepareAudioThumb(double pixelForOneFrame, QPainterPath path,
 
         for (int samples = 0;samples <= 100;samples++) {
             double frame = (double)(samples + startCache - 0) / pixelForOneFrame;
-            int sample = (frame - (int)(frame)) * 20 ;// AUDIO_FRAME_SIZE
+            int sample = (int)((frame - (int)(frame)) * 20);   // AUDIO_FRAME_SIZE
             if (frame < 0 || sample < 0 || sample > 19)
                 continue;
             QMap<int, QByteArray> frame_channel_data = baseClip()->audioFrameChache[(int)frame];
@@ -427,10 +679,14 @@ void ClipItem::slotPrepareAudioThumb(double pixelForOneFrame, QPainterPath path,
                 }
 
         }
+        if (m_clipType != AV) pixpainter.setBrush(QBrush(QColor(200, 200, 100)));
+        else {
+            pixpainter.setPen(QPen(QColor(0, 0, 0)));
+            pixpainter.setBrush(QBrush(QColor(60, 60, 60)));
+        }
         for (int i = 0;i < channels;i++) {
             if (fullAreaDraw) {
                 //pixpainter.fillPath(positiveChannelPaths[i].united(negativeChannelPaths[i]),QBrush(Qt::SolidPattern));//or singleif looks better
-                pixpainter.setBrush(QBrush(QColor(200, 200, 100, 200)));
                 pixpainter.drawPath(positiveChannelPaths[i].united(negativeChannelPaths[i]));//or singleif looks better
             } else
                 pixpainter.drawPath(positiveChannelPaths[i]);
@@ -442,34 +698,32 @@ void ClipItem::slotPrepareAudioThumb(double pixelForOneFrame, QPainterPath path,
     //}
 }
 
-int ClipItem::fadeIn() const {
+uint ClipItem::fadeIn() const {
     return m_startFade;
 }
 
-int ClipItem::fadeOut() const {
+uint ClipItem::fadeOut() const {
     return m_endFade;
 }
 
+
 void ClipItem::setFadeIn(int pos, double scale) {
     int oldIn = m_startFade;
     if (pos < 0) pos = 0;
-    if (pos > m_cropDuration) pos = m_cropDuration / 2;
+    if (pos > m_cropDuration.frames(m_fps)) pos = (int)(m_cropDuration.frames(m_fps) / 2);
     m_startFade = pos;
-    if (oldIn > pos) update(rect().x(), rect().y(), oldIn * scale, rect().height());
-    else update(rect().x(), rect().y(), pos * scale, rect().height());
+    update(rect().x(), rect().y(), qMax(oldIn, pos) * scale, rect().height());
 }
 
 void ClipItem::setFadeOut(int pos, double scale) {
     int oldOut = m_endFade;
     if (pos < 0) pos = 0;
-    if (pos > m_cropDuration) pos = m_cropDuration / 2;
+    if (pos > m_cropDuration.frames(m_fps)) pos = (int)(m_cropDuration.frames(m_fps) / 2);
     m_endFade = pos;
-    if (oldOut > pos) update(rect().x() + rect().width() - pos * scale, rect().y(), pos * scale, rect().height());
-    else update(rect().x() + rect().width() - oldOut * scale, rect().y(), oldOut * scale, rect().height());
+    update(rect().x() + rect().width() - qMax(oldOut, pos) * scale, rect().y(), pos * scale, rect().height());
 
 }
 
-
 // virtual
 void ClipItem::mousePressEvent(QGraphicsSceneMouseEvent * event) {
     /*m_resizeMode = operationMode(event->pos());
@@ -486,109 +740,76 @@ void ClipItem::mouseReleaseEvent(QGraphicsSceneMouseEvent * event) {
     QGraphicsRectItem::mouseReleaseEvent(event);
 }
 
-void ClipItem::moveTo(int x, double scale, double offset, int newTrack) {
-    double origX = rect().x();
-    double origY = rect().y();
-    bool success = true;
-    if (x < 0) return;
-    setRect(x * scale, origY + offset, rect().width(), rect().height());
-    QList <QGraphicsItem *> collisionList = collidingItems(Qt::IntersectsItemBoundingRect);
-    if (collisionList.size() == 0) m_track = newTrack;
-    for (int i = 0; i < collisionList.size(); ++i) {
-        QGraphicsItem *item = collisionList.at(i);
-        if (item->type() == 70000) {
-            if (offset == 0) {
-                QRectF other = ((QGraphicsRectItem *)item)->rect();
-                if (x < m_startPos) {
-                    kDebug() << "COLLISION, MOVING TO------";
-                    m_startPos = ((ClipItem *)item)->endPos() + 1;
-                    origX = m_startPos * scale;
-                } else {
-                    kDebug() << "COLLISION, MOVING TO+++";
-                    m_startPos = ((ClipItem *)item)->startPos() - m_cropDuration;
-                    origX = m_startPos * scale;
-                }
-            }
-            setRect(origX, origY, rect().width(), rect().height());
-            offset = 0;
-            origX = rect().x();
-            success = false;
-            break;
-        }
-    }
-    if (success) {
-        m_track = newTrack;
-        m_startPos = x;
-    }
-    /*    QList <QGraphicsItem *> childrenList = QGraphicsItem::children();
-        for (int i = 0; i < childrenList.size(); ++i) {
-          childrenList.at(i)->moveBy(rect().x() - origX , offset);
-        }*/
+//virtual
+void ClipItem::hoverEnterEvent(QGraphicsSceneHoverEvent *) {
+    m_hover = true;
+    QRectF r = boundingRect();
+    qreal width = qMin(25.0, r.width());
+    update(r.x(), r.y(), width, r.height());
+    update(r.right() - width, r.y(), width, r.height());
+}
+
+//virtual
+void ClipItem::hoverLeaveEvent(QGraphicsSceneHoverEvent *) {
+    m_hover = false;
+    QRectF r = boundingRect();
+    qreal width = qMin(25.0, r.width());
+    update(r.x(), r.y(), width, r.height());
+    update(r.right() - width, r.y(), width, r.height());
 }
 
 void ClipItem::resizeStart(int posx, double scale) {
-    int durationDiff = posx - m_startPos;
-    if (durationDiff == 0) return;
-    kDebug() << "-- RESCALE: CROP=" << m_cropStart << ", DIFF = " << durationDiff;
-    if (m_cropStart + durationDiff < 0) {
-        durationDiff = -m_cropStart;
-    } else if (durationDiff >= m_cropDuration) {
-        durationDiff = m_cropDuration - 3;
-    }
-    m_startPos += durationDiff;
-    m_cropStart += durationDiff;
-    m_cropDuration -= durationDiff;
-    setRect(m_startPos * scale, rect().y(), m_cropDuration * scale, rect().height());
-    QList <QGraphicsItem *> collisionList = collidingItems(Qt::IntersectsItemBoundingRect);
-    for (int i = 0; i < collisionList.size(); ++i) {
-        QGraphicsItem *item = collisionList.at(i);
-        if (item->type() == 70000) {
-            int diff = ((ClipItem *)item)->endPos() + 1 - m_startPos;
-            setRect((m_startPos + diff) * scale, rect().y(), (m_cropDuration - diff) * scale, rect().height());
-            m_startPos += diff;
-            m_cropStart += diff;
-            m_cropDuration -= diff;
-            break;
-        }
-    }
-    if (m_hasThumbs) startThumbTimer->start(100);
+    const int previous = cropStart().frames(m_fps);
+    AbstractClipItem::resizeStart(posx, scale);
+    checkEffectsKeyframesPos(previous, cropStart().frames(m_fps), true);
+    if (m_hasThumbs && KdenliveSettings::videothumbnails()) startThumbTimer->start(100);
 }
 
 void ClipItem::resizeEnd(int posx, double scale) {
-    int durationDiff = posx - endPos();
-    if (durationDiff == 0) return;
-    kDebug() << "-- RESCALE: CROP=" << m_cropStart << ", DIFF = " << durationDiff;
-    if (m_cropDuration + durationDiff <= 0) {
-        durationDiff = - (m_cropDuration - 3);
-    } else if (m_cropDuration + durationDiff >= m_maxDuration) {
-        durationDiff = m_maxDuration - m_cropDuration;
-    }
-    m_cropDuration += durationDiff;
-    setRect(m_startPos * scale, rect().y(), m_cropDuration * scale, rect().height());
-    QList <QGraphicsItem *> collisionList = collidingItems(Qt::IntersectsItemBoundingRect);
-    for (int i = 0; i < collisionList.size(); ++i) {
-        QGraphicsItem *item = collisionList.at(i);
-        if (item->type() == 70000) {
-            int diff = ((ClipItem *)item)->startPos() - 1 - startPos();
-            m_cropDuration = diff;
-            setRect(m_startPos * scale, rect().y(), m_cropDuration * scale, rect().height());
-            break;
+    const int previous = (cropStart() + duration()).frames(m_fps);
+    AbstractClipItem::resizeEnd(posx, scale);
+    checkEffectsKeyframesPos(previous, (cropStart() + duration()).frames(m_fps), false);
+    if (m_hasThumbs && KdenliveSettings::videothumbnails()) endThumbTimer->start(100);
+}
+
+
+void ClipItem::checkEffectsKeyframesPos(const int previous, const int current, bool fromStart) {
+    for (int i = 0; i < m_effectList.size(); i++) {
+        QDomElement effect = m_effectList.at(i);
+        QDomNodeList params = effect.elementsByTagName("parameter");
+        for (int j = 0; j < params.count(); j++) {
+            QDomElement e = params.item(i).toElement();
+            if (e.attribute("type") == "keyframe") {
+                // parse keyframes and adjust values
+                QStringList keyframes = e.attribute("keyframes").split(";", QString::SkipEmptyParts);
+                QMap <int, double> kfr;
+                foreach(QString str, keyframes) {
+                    int pos = str.section(":", 0, 0).toInt();
+                    double val = str.section(":", 1, 1).toDouble();
+                    if (pos == previous) kfr[current] = val;
+                    else {
+                        if (fromStart && pos >= current) kfr[pos] = val;
+                        else if (!fromStart && pos <= current) kfr[pos] = val;
+                    }
+                }
+                QString newkfr;
+                QMap<int, double>::const_iterator k = kfr.constBegin();
+                while (k != kfr.constEnd()) {
+                    newkfr.append(QString::number(k.key()) + ":" + QString::number(k.value()) + ";");
+                    ++k;
+                }
+                e.setAttribute("keyframes", newkfr);
+                break;
+            }
         }
     }
-    if (m_hasThumbs) endThumbTimer->start(100);
-}
-
-// virtual
-void ClipItem::mouseMoveEvent(QGraphicsSceneMouseEvent * event) {
+    if (m_selectedEffect >= 0) setSelectedEffect(m_selectedEffect);
 }
 
-int ClipItem::track() {
-    return  m_track;
-}
 
-void ClipItem::setTrack(int track) {
-    m_track = track;
-}
+// virtual
+/*void ClipItem::mouseMoveEvent(QGraphicsSceneMouseEvent * event) {
+}*/
 
 int ClipItem::effectsCounter() {
     return m_effectsCounter++;
@@ -603,6 +824,7 @@ QStringList ClipItem::effectNames() {
 }
 
 QDomElement ClipItem::effectAt(int ix) {
+    if (ix > m_effectList.count() - 1 || ix < 0) return QDomElement();
     return m_effectList.at(ix);
 }
 
@@ -610,28 +832,77 @@ void ClipItem::setEffectAt(int ix, QDomElement effect) {
     kDebug() << "CHange EFFECT AT: " << ix << ", CURR: " << m_effectList.at(ix).attribute("tag") << ", NEW: " << effect.attribute("tag");
     m_effectList.insert(ix, effect);
     m_effectList.removeAt(ix + 1);
-    update(boundingRect());
+    m_effectNames = m_effectList.effectNames().join(" / ");
+    if (effect.attribute("id") == "fadein" || effect.attribute("id") == "fadeout") update(boundingRect());
+    else {
+        QRectF r = boundingRect();
+        r.setHeight(20);
+        update(r);
+    }
 }
 
-QMap <QString, QString> ClipItem::addEffect(QDomElement effect) {
+QMap <QString, QString> ClipItem::addEffect(QDomElement effect, bool animate) {
     QMap <QString, QString> effectParams;
+    bool needRepaint = false;
+    /*QDomDocument doc;
+    doc.appendChild(doc.importNode(effect, true));
+    kDebug() << "///////  CLIP ADD EFFECT: "<< doc.toString();*/
     m_effectList.append(effect);
     effectParams["tag"] = effect.attribute("tag");
+    QString effectId = effect.attribute("id");
+    if (effectId.isEmpty()) effectId = effect.attribute("tag");
+    effectParams["id"] = effectId;
     effectParams["kdenlive_ix"] = effect.attribute("kdenlive_ix");
     QString state = effect.attribute("disabled");
     if (!state.isEmpty()) effectParams["disabled"] = state;
     QDomNodeList params = effect.elementsByTagName("parameter");
+    int fade = 0;
     for (int i = 0; i < params.count(); i++) {
         QDomElement e = params.item(i).toElement();
         if (!e.isNull()) {
-            effectParams[e.attribute("name")] = e.attribute("value");
-        }
-        if (!e.attribute("factor").isEmpty()) {
-            effectParams[e.attribute("name")] =  QString::number(effectParams[e.attribute("name")].toDouble() / e.attribute("factor").toDouble());
+            if (e.attribute("type") == "keyframe") {
+                effectParams["keyframes"] = e.attribute("keyframes");
+                effectParams["min"] = e.attribute("min");
+                effectParams["max"] = e.attribute("max");
+                effectParams["factor"] = e.attribute("factor", "1");
+                effectParams["starttag"] = e.attribute("starttag", "start");
+                effectParams["endtag"] = e.attribute("endtag", "end");
+            }
+
+            double f = e.attribute("factor", "1").toDouble();
+
+            if (f == 1) {
+                effectParams[e.attribute("name")] = e.attribute("value");
+                // check if it is a fade effect
+                if (effectId == "fadein") {
+                    needRepaint = true;
+                    if (e.attribute("name") == "out") fade += e.attribute("value").toInt();
+                    else if (e.attribute("name") == "in") fade -= e.attribute("value").toInt();
+                } else if (effectId == "fadeout") {
+                    needRepaint = true;
+                    if (e.attribute("name") == "out") fade -= e.attribute("value").toInt();
+                    else if (e.attribute("name") == "in") fade += e.attribute("value").toInt();
+                }
+            } else {
+                effectParams[e.attribute("name")] =  QString::number(effectParams[e.attribute("name")].toDouble() / f);
+            }
         }
     }
-    flashClip();
-    update(boundingRect());
+    m_effectNames = m_effectList.effectNames().join(" / ");
+    if (fade > 0) m_startFade = fade;
+    else if (fade < 0) m_endFade = -fade;
+    if (needRepaint) update(boundingRect());
+    if (animate) {
+        flashClip();
+    } else if (!needRepaint) {
+        QRectF r = boundingRect();
+        r.setHeight(20);
+        update(r);
+    }
+    if (m_selectedEffect == -1) {
+        m_selectedEffect = 0;
+        setSelectedEffect(m_selectedEffect);
+    }
     return effectParams;
 }
 
@@ -639,12 +910,22 @@ QMap <QString, QString> ClipItem::getEffectArgs(QDomElement effect) {
     QMap <QString, QString> effectParams;
     effectParams["tag"] = effect.attribute("tag");
     effectParams["kdenlive_ix"] = effect.attribute("kdenlive_ix");
+    effectParams["id"] = effect.attribute("id");
     QString state = effect.attribute("disabled");
     if (!state.isEmpty()) effectParams["disabled"] = state;
     QDomNodeList params = effect.elementsByTagName("parameter");
     for (int i = 0; i < params.count(); i++) {
         QDomElement e = params.item(i).toElement();
-        if (e.attribute("name").contains(";")) {
+        kDebug() << "/ / / /SENDING EFFECT PARAM: " << e.attribute("type") << ", NAME_ " << e.attribute("tag");
+        if (e.attribute("type") == "keyframe") {
+            kDebug() << "/ / / /SENDING KEYFR EFFECT TYPE";
+            effectParams["keyframes"] = e.attribute("keyframes");
+            effectParams["max"] = e.attribute("max");
+            effectParams["min"] = e.attribute("min");
+            effectParams["factor"] = e.attribute("factor", "1");
+            effectParams["starttag"] = e.attribute("starttag", "start");
+            effectParams["endtag"] = e.attribute("endtag", "end");
+        } else if (e.attribute("namedesc").contains(";")) {
             QString format = e.attribute("format");
             QStringList separators = format.split("%d", QString::SkipEmptyParts);
             QStringList values = e.attribute("value").split(QRegExp("[,:;x]"));
@@ -657,26 +938,33 @@ QMap <QString, QString> ClipItem::getEffectArgs(QDomElement effect) {
                 txtNeu << (int)(values[i+1].toDouble());
             }
             effectParams["start"] = neu;
-        } else
-            if (!e.isNull()) {
-                effectParams[e.attribute("name")] = e.attribute("value");
-            }
-        if (!e.attribute("factor").isEmpty()) {
-            effectParams[e.attribute("name")] =  QString::number(effectParams[e.attribute("name")].toDouble() / e.attribute("factor").toDouble());
+        } else {
+            if (e.attribute("factor", "1") != "1")
+                effectParams[e.attribute("name")] =  QString::number(e.attribute("value").toDouble() / e.attribute("factor").toDouble());
+            else effectParams[e.attribute("name")] = e.attribute("value");
         }
     }
     return effectParams;
 }
 
 void ClipItem::deleteEffect(QString index) {
+    bool needRepaint = false;
     for (int i = 0; i < m_effectList.size(); ++i) {
         if (m_effectList.at(i).attribute("kdenlive_ix") == index) {
+            if (m_effectList.at(i).attribute("id") == "fadein") {
+                m_startFade = 0;
+                needRepaint = true;
+            } else if (m_effectList.at(i).attribute("id") == "fadeout") {
+                m_endFade = 0;
+                needRepaint = true;
+            }
             m_effectList.removeAt(i);
             break;
         }
     }
+    m_effectNames = m_effectList.effectNames().join(" / ");
+    if (needRepaint) update(boundingRect());
     flashClip();
-    update(boundingRect());
 }
 
 //virtual
@@ -686,7 +974,7 @@ void ClipItem::dropEvent(QGraphicsSceneDragDropEvent * event) {
     doc.setContent(effects, true);
     QDomElement e = doc.documentElement();
     CustomTrackView *view = (CustomTrackView *) scene()->views()[0];
-    if (view) view->slotAddEffect(e, GenTime(m_startPos, 25), m_track);
+    if (view) view->slotAddEffect(e, m_startPos, track());
 }
 
 //virtual
@@ -697,7 +985,13 @@ void ClipItem::dragEnterEvent(QGraphicsSceneDragDropEvent *event) {
 void ClipItem::dragLeaveEvent(QGraphicsSceneDragDropEvent *event) {
     Q_UNUSED(event);
 }
-
+void ClipItem::addTransition(Transition* t) {
+    m_transitionsList.append(t);
+    CustomTrackView *view = (CustomTrackView *) scene()->views()[0];
+    QDomDocument doc;
+    QDomElement e = doc.documentElement();
+    //if (view) view->slotAddTransition(this, t->toXML() , t->startPos(), track());
+}
 // virtual
 /*
 void CustomTrackView::mousePressEvent ( QMouseEvent * event )