]> git.sesse.net Git - kdenlive/blobdiff - src/customtrackview.cpp
Cleaning code style of Definitions.
[kdenlive] / src / customtrackview.cpp
index ebeb71946c1f5c3ac2303570b9d0b24888520f70..65454aa23bff2672cd91b71d7cf6511493f82674 100644 (file)
@@ -62,7 +62,6 @@
 #include "tracksconfigdialog.h"
 #include "commands/configtrackscommand.h"
 #include "commands/rebuildgroupcommand.h"
-#include "commands/razorgroupcommand.h"
 #include "commands/refreshmonitorcommand.h"
 #include "profilesdialog.h"
 
@@ -70,7 +69,7 @@
 #include "lib/audio/audioCorrelation.h"
 
 #include <KDebug>
-#include <KLocale>
+#include <KLocalizedString>
 #include <KUrl>
 #include <KIcon>
 #include <KCursor>
@@ -110,35 +109,44 @@ bool sortGuidesList(const Guide *g1 , const Guide *g2)
 // const int duration = animate ? 1500 : 1;
 
 CustomTrackView::CustomTrackView(KdenliveDoc *doc, CustomTrackScene* projectscene, QWidget *parent) :
-    QGraphicsView(projectscene, parent),
-    m_tracksHeight(KdenliveSettings::trackheight()),
-    m_projectDuration(0),
-    m_cursorPos(0),
-    m_document(doc),
-    m_scene(projectscene),
-    m_cursorLine(NULL),
-    m_operationMode(NONE),
-    m_moveOpMode(NONE),
-    m_dragItem(NULL),
-    m_dragGuide(NULL),
-    m_visualTip(NULL),
-    m_animation(NULL),
-    m_clickPoint(),
-    m_autoScroll(KdenliveSettings::autoscroll()),
-    m_pasteEffectsAction(NULL),
-    m_ungroupAction(NULL),
-    m_scrollOffset(0),
-    m_clipDrag(false),
-    m_findIndex(0),
-    m_tool(SELECTTOOL),
-    m_copiedItems(),
-    m_menuPosition(),
-    m_blockRefresh(false),
-    m_selectionGroup(NULL),
-    m_selectedTrack(0),
-    m_audioCorrelator(NULL),
-    m_audioAlignmentReference(NULL),
-    m_controlModifier(false)
+    QGraphicsView(projectscene, parent)
+  , m_tracksHeight(KdenliveSettings::trackheight())
+  , m_projectDuration(0)
+  , m_cursorPos(0)
+  , m_document(doc)
+  , m_scene(projectscene)
+  , m_cursorLine(NULL)
+  , m_operationMode(None)
+  , m_moveOpMode(None)
+  , m_dragItem(NULL)
+  , m_dragGuide(NULL)
+  , m_visualTip(NULL)
+  , m_animation(NULL)
+  , m_clickPoint()
+  , m_autoScroll(KdenliveSettings::autoscroll())
+  , m_timelineContextMenu(NULL)
+  , m_timelineContextClipMenu(NULL)
+  , m_timelineContextTransitionMenu(NULL)
+  , m_markerMenu(NULL)
+  , m_autoTransition(NULL)
+  , m_pasteEffectsAction(NULL)
+  , m_ungroupAction(NULL)
+  , m_editGuide(NULL)
+  , m_deleteGuide(NULL)
+  , m_clipTypeGroup(NULL)
+  , m_scrollOffset(0)
+  , m_clipDrag(false)
+  , m_findIndex(0)
+  , m_tool(SelectTool)
+  , m_copiedItems()
+  , m_menuPosition()
+  , m_blockRefresh(false)
+  , m_selectionGroup(NULL)
+  , m_selectedTrack(0)
+  , m_spacerOffset(0)
+  , m_audioCorrelator(NULL)
+  , m_audioAlignmentReference(NULL)
+  , m_controlModifier(false)
 {
     if (doc) {
         m_commandStack = doc->commandStack();
@@ -231,16 +239,18 @@ void CustomTrackView::setContextMenu(QMenu *timeline, QMenu *clip, QMenu *transi
     m_markerMenu = new QMenu(i18n("Go to marker..."), this);
     m_markerMenu->setEnabled(false);
     markermenu->addMenu(m_markerMenu);
-    connect(m_markerMenu, SIGNAL(triggered(QAction *)), this, SLOT(slotGoToMarker(QAction *)));
+    connect(m_markerMenu, SIGNAL(triggered(QAction*)), this, SLOT(slotGoToMarker(QAction*)));
     QList <QAction *> list = m_timelineContextClipMenu->actions();
-    for (int i = 0; i < list.count(); i++) {
+    for (int i = 0; i < list.count(); ++i) {
         if (list.at(i)->data().toString() == "paste_effects") m_pasteEffectsAction = list.at(i);
         else if (list.at(i)->data().toString() == "ungroup_clip") m_ungroupAction = list.at(i);
+        else if (list.at(i)->data().toString() == "A") m_audioActions.append(list.at(i));
+        else if (list.at(i)->data().toString() == "A+V") m_avActions.append(list.at(i));
     }
 
     m_timelineContextTransitionMenu = transition;
     list = m_timelineContextTransitionMenu->actions();
-    for (int i = 0; i < list.count(); i++) {
+    for (int i = 0; i < list.count(); ++i) {
         if (list.at(i)->data().toString() == "auto") {
             m_autoTransition = list.at(i);
             break;
@@ -278,7 +288,7 @@ void CustomTrackView::checkAutoScroll()
 }*/
 
 
-int CustomTrackView::getFrameWidth()
+int CustomTrackView::getFrameWidth() const
 {
     return (int) (m_tracksHeight * m_document->mltProfile().display_aspect_num / m_document->mltProfile().display_aspect_den + 0.5);
 }
@@ -288,8 +298,8 @@ void CustomTrackView::updateSceneFrameWidth()
     int frameWidth = getFrameWidth();
     QList<QGraphicsItem *> itemList = items();
     ClipItem *item;
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             item = (ClipItem*) itemList.at(i);
             item->resetFrameWidth(frameWidth);
             item->resetThumbs(true);
@@ -308,30 +318,27 @@ bool CustomTrackView::checkTrackHeight()
     int frameWidth = getFrameWidth();
     bool snap = KdenliveSettings::snaptopoints();
     KdenliveSettings::setSnaptopoints(false);
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             item = (ClipItem*) itemList.at(i);
             item->setRect(0, 0, item->rect().width(), m_tracksHeight - 1);
             item->setPos((qreal) item->startPos().frames(m_document->fps()), (qreal) item->track() * m_tracksHeight + 1);
             item->resetFrameWidth(frameWidth);
             item->resetThumbs(true);
-        } else if (itemList.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (itemList.at(i)->type() == TransitionWidget) {
             transitionitem = (Transition*) itemList.at(i);
             transitionitem->setRect(0, 0, transitionitem->rect().width(), m_tracksHeight / 3 * 2 - 1);
             transitionitem->setPos((qreal) transitionitem->startPos().frames(m_document->fps()), (qreal) transitionitem->track() * m_tracksHeight + m_tracksHeight / 3 * 2);
         }
     }
     double newHeight = m_tracksHeight * m_document->tracksCount() * matrix().m22();
-    m_cursorLine->setLine(m_cursorLine->line().x1(), 0, m_cursorLine->line().x1(), newHeight - 1);
-
-    for (int i = 0; i < m_guides.count(); i++) {
-        QLineF l = m_guides.at(i)->line();
-        l.setP2(QPointF(l.x2(), newHeight));
-        m_guides.at(i)->setLine(l);
+    m_cursorLine->setLine(0, 0, 0, newHeight - 1);
+    for (int i = 0; i < m_guides.count(); ++i) {
+        m_guides.at(i)->setLine(0, 0, 0, newHeight - 1);
     }
 
     setSceneRect(0, 0, sceneRect().width(), m_tracksHeight * m_document->tracksCount());
-//     verticalScrollBar()->setMaximum(m_tracksHeight * m_document->tracksCount());
+    //     verticalScrollBar()->setMaximum(m_tracksHeight * m_document->tracksCount());
     KdenliveSettings::setSnaptopoints(snap);
     viewport()->update();
     return true;
@@ -360,8 +367,8 @@ int CustomTrackView::getPreviousVideoTrack(int track)
 {
     track = m_document->tracksCount() - track - 1;
     track --;
-    for (int i = track; i > -1; i--) {
-        if (m_document->trackInfoAt(i).type == VIDEOTRACK) return i + 1;
+    for (int i = track; i > -1; --i) {
+        if (m_document->trackInfoAt(i).type == VideoTrack) return i + 1;
     }
     return 0;
 }
@@ -392,9 +399,9 @@ void CustomTrackView::slotCheckMouseScrolling()
 void CustomTrackView::slotCheckPositionScrolling()
 {
     // If mouse is at a border of the view, scroll
-    if (m_moveOpMode != SEEK) return;
+    if (m_moveOpMode != Seek) return;
     if (mapFromScene(m_cursorPos, 0).x() < 3) {
-       if (horizontalScrollBar()->value() == 0) return;
+        if (horizontalScrollBar()->value() == 0) return;
         horizontalScrollBar()->setValue(horizontalScrollBar()->value() - 2);
         QTimer::singleShot(200, this, SLOT(slotCheckPositionScrolling()));
         seekCursorPos(mapToScene(QPoint(-2, 0)).x());
@@ -407,12 +414,17 @@ void CustomTrackView::slotCheckPositionScrolling()
 
 void CustomTrackView::slotAlignPlayheadToMousePos()
 {
-       /* get curser point ref in screen coord */
-       QPoint ps = QCursor::pos();
-       /* get xPos in scene coord */
-       int mappedXPos = qMax((int)(mapToScene(mapFromGlobal(ps)).x() + 0.5), 0);
-       /* move playhead to new xPos*/
-       seekCursorPos(mappedXPos);
+    /* get curser point ref in screen coord */
+    QPoint ps = QCursor::pos();
+    /* get xPos in scene coord */
+    int mappedXPos = qMax((int)(mapToScene(mapFromGlobal(ps)).x() + 0.5), 0);
+    /* move playhead to new xPos*/
+    seekCursorPos(mappedXPos);
+}
+
+int CustomTrackView::getMousePos() const
+{
+    return qMax((int)(mapToScene(mapFromGlobal(QCursor::pos())).x() + 0.5), 0);
 }
 
 // virtual
@@ -423,24 +435,24 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
     double snappedPos = getSnapPointForPos(mappedXPos);
     emit mousePosition(mappedXPos);
 
-    if (m_operationMode == SCROLLTIMELINE) {
-       QGraphicsView::mouseMoveEvent(event);
-       return;
+    if (m_operationMode == ScrollTimeline) {
+        QGraphicsView::mouseMoveEvent(event);
+        return;
     }
 
     if (event->buttons() & Qt::MidButton) return;
-    if (dragMode() == QGraphicsView::RubberBandDrag || (event->modifiers() == Qt::ControlModifier && m_tool != SPACERTOOL && m_operationMode != RESIZESTART && m_operationMode != RESIZEEND)) {
+    if (dragMode() == QGraphicsView::RubberBandDrag || (event->modifiers() == Qt::ControlModifier && m_tool != SpacerTool && m_operationMode != ResizeStart && m_operationMode != ResizeEnd)) {
         event->setAccepted(true);
-        m_moveOpMode = NONE;
+        m_moveOpMode = None;
         if (event->modifiers() != Qt::ControlModifier || dragMode() == QGraphicsView::RubberBandDrag) QGraphicsView::mouseMoveEvent(event);
         return;
     }
 
     if (event->buttons() != Qt::NoButton) {
         bool move = (event->pos() - m_clickEvent).manhattanLength() >= QApplication::startDragDistance();
-       if (m_dragItem && move) m_clipDrag = true;
-        if (m_dragItem && m_tool == SELECTTOOL) {
-            if (m_operationMode == MOVE && m_clipDrag) {
+        if (m_dragItem && move) m_clipDrag = true;
+        if (m_dragItem && m_tool == SelectTool) {
+            if (m_operationMode == MoveOperation && m_clipDrag) {
                 QGraphicsView::mouseMoveEvent(event);
                 // If mouse is at a border of the view, scroll
                 if (pos < 5) {
@@ -452,9 +464,9 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
                 } else if (m_scrollTimer.isActive()) {
                     m_scrollTimer.stop();
                 }
-            } else if (m_operationMode == RESIZESTART && move) {
+            } else if (m_operationMode == ResizeStart && move) {
                 m_document->renderer()->pause();
-                if (!m_controlModifier && m_dragItem->type() == AVWIDGET && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
+                if (!m_controlModifier && m_dragItem->type() == AVWidget && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
                     AbstractGroupItem *parent = static_cast <AbstractGroupItem *>(m_dragItem->parentItem());
                     if (parent)
                         parent->resizeStart((int)(snappedPos - m_dragItemInfo.startPos.frames(m_document->fps())));
@@ -465,9 +477,9 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
                 QString duration = m_document->timecode().getDisplayTimecode(m_dragItem->cropDuration(), KdenliveSettings::frametimecode());
                 QString offset = m_document->timecode().getDisplayTimecode(m_dragItem->cropStart() - m_dragItemInfo.cropStart, KdenliveSettings::frametimecode());
                 emit displayMessage(i18n("Crop from start:") + ' ' + crop + ' ' + i18n("Duration:") + ' ' + duration + ' ' + i18n("Offset:") + ' ' + offset, InformationMessage);
-            } else if (m_operationMode == RESIZEEND && move) {
+            } else if (m_operationMode == ResizeEnd && move) {
                 m_document->renderer()->pause();
-                if (!m_controlModifier && m_dragItem->type() == AVWIDGET && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
+                if (!m_controlModifier && m_dragItem->type() == AVWidget && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
                     AbstractGroupItem *parent = static_cast <AbstractGroupItem *>(m_dragItem->parentItem());
                     if (parent)
                         parent->resizeEnd((int)(snappedPos - m_dragItemInfo.endPos.frames(m_document->fps())));
@@ -477,11 +489,11 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
                 QString duration = m_document->timecode().getDisplayTimecode(m_dragItem->cropDuration(), KdenliveSettings::frametimecode());
                 QString offset = m_document->timecode().getDisplayTimecode(m_dragItem->cropDuration() - m_dragItemInfo.cropDuration, KdenliveSettings::frametimecode());
                 emit displayMessage(i18n("Duration:") + ' ' + duration + ' ' + i18n("Offset:") + ' ' + offset, InformationMessage);
-            } else if (m_operationMode == FADEIN && move) {
+            } else if (m_operationMode == FadeIn && move) {
                 ((ClipItem*) m_dragItem)->setFadeIn((int)(mappedXPos - m_dragItem->startPos().frames(m_document->fps())));
-            } else if (m_operationMode == FADEOUT && move) {
+            } else if (m_operationMode == FadeOut && move) {
                 ((ClipItem*) m_dragItem)->setFadeOut((int)(m_dragItem->endPos().frames(m_document->fps()) - mappedXPos));
-            } else if (m_operationMode == KEYFRAME && move) {
+            } else if (m_operationMode == KeyFrame && move) {
                 GenTime keyFramePos = GenTime(mappedXPos, m_document->fps()) - m_dragItem->startPos() + m_dragItem->cropStart();
                 double pos = mapToScene(event->pos()).toPoint().y();
                 QRectF br = m_dragItem->sceneBoundingRect();
@@ -493,22 +505,22 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
             }
             removeTipAnimation();
             return;
-        } else if (m_operationMode == MOVEGUIDE) {
+        } else if (m_operationMode == MoveGuide) {
             removeTipAnimation();
             QGraphicsView::mouseMoveEvent(event);
             return;
-        } else if (m_operationMode == SPACER && move && m_selectionGroup) {
+        } else if (m_operationMode == Spacer && move && m_selectionGroup) {
             // spacer tool
             snappedPos = getSnapPointForPos(mappedXPos + m_spacerOffset);
             if (snappedPos < 0) snappedPos = 0;
 
             // Make sure there is no collision
             QList<QGraphicsItem *> children = m_selectionGroup->childItems();
-            QPainterPath shape = m_selectionGroup->clipGroupShape(QPointF(snappedPos - m_selectionGroup->sceneBoundingRect().left(), 0));
+            QPainterPath shape = m_selectionGroup->clipGroupSpacerShape(QPointF(snappedPos - m_selectionGroup->sceneBoundingRect().left(), 0));
             QList<QGraphicsItem*> collidingItems = scene()->items(shape, Qt::IntersectsItemShape);
             collidingItems.removeAll(m_selectionGroup);
-            for (int i = 0; i < children.count(); i++) {
-                if (children.at(i)->type() == GROUPWIDGET) {
+            for (int i = 0; i < children.count(); ++i) {
+                if (children.at(i)->type() == GroupWidget) {
                     QList<QGraphicsItem *> subchildren = children.at(i)->childItems();
                     for (int j = 0; j < subchildren.count(); j++)
                         collidingItems.removeAll(subchildren.at(j));
@@ -517,9 +529,9 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
             }
             bool collision = false;
             int offset = 0;
-            for (int i = 0; i < collidingItems.count(); i++) {
+            for (int i = 0; i < collidingItems.count(); ++i) {
                 if (!collidingItems.at(i)->isEnabled()) continue;
-                if (collidingItems.at(i)->type() == AVWIDGET && snappedPos < m_selectionGroup->sceneBoundingRect().left()) {
+                if (collidingItems.at(i)->type() == AVWidget && snappedPos < m_selectionGroup->sceneBoundingRect().left()) {
                     AbstractClipItem *item = static_cast <AbstractClipItem *>(collidingItems.at(i));
                     // Moving backward, determine best pos
                     QPainterPath clipPath;
@@ -530,11 +542,11 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
             }
             snappedPos += offset;
             // make sure we have no collision
-            shape = m_selectionGroup->clipGroupShape(QPointF(snappedPos - m_selectionGroup->sceneBoundingRect().left(), 0));
+            shape = m_selectionGroup->clipGroupSpacerShape(QPointF(snappedPos - m_selectionGroup->sceneBoundingRect().left(), 0));
             collidingItems = scene()->items(shape, Qt::IntersectsItemShape);
             collidingItems.removeAll(m_selectionGroup);
-            for (int i = 0; i < children.count(); i++) {
-                if (children.at(i)->type() == GROUPWIDGET) {
+            for (int i = 0; i < children.count(); ++i) {
+                if (children.at(i)->type() == GroupWidget) {
                     QList<QGraphicsItem *> subchildren = children.at(i)->childItems();
                     for (int j = 0; j < subchildren.count(); j++)
                         collidingItems.removeAll(subchildren.at(j));
@@ -542,9 +554,9 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
                 collidingItems.removeAll(children.at(i));
             }
 
-            for (int i = 0; i < collidingItems.count(); i++) {
+            for (int i = 0; i < collidingItems.count(); ++i) {
                 if (!collidingItems.at(i)->isEnabled()) continue;
-                if (collidingItems.at(i)->type() == AVWIDGET) {
+                if (collidingItems.at(i)->type() == AVWidget) {
                     collision = true;
                     break;
                 }
@@ -556,8 +568,8 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
                 shape = m_selectionGroup->transitionGroupShape(QPointF(snappedPos - m_selectionGroup->sceneBoundingRect().left(), 0));
                 collidingItems = scene()->items(shape, Qt::IntersectsItemShape);
                 collidingItems.removeAll(m_selectionGroup);
-                for (int i = 0; i < children.count(); i++) {
-                    if (children.at(i)->type() == GROUPWIDGET) {
+                for (int i = 0; i < children.count(); ++i) {
+                    if (children.at(i)->type() == GroupWidget) {
                         QList<QGraphicsItem *> subchildren = children.at(i)->childItems();
                         for (int j = 0; j < subchildren.count(); j++)
                             collidingItems.removeAll(subchildren.at(j));
@@ -566,8 +578,8 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
                 }
                 offset = 0;
 
-                for (int i = 0; i < collidingItems.count(); i++) {
-                    if (collidingItems.at(i)->type() == TRANSITIONWIDGET && snappedPos < m_selectionGroup->sceneBoundingRect().left()) {
+                for (int i = 0; i < collidingItems.count(); ++i) {
+                    if (collidingItems.at(i)->type() == TransitionWidget && snappedPos < m_selectionGroup->sceneBoundingRect().left()) {
                         AbstractClipItem *item = static_cast <AbstractClipItem *>(collidingItems.at(i));
                         // Moving backward, determine best pos
                         QPainterPath clipPath;
@@ -582,16 +594,16 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
                 shape = m_selectionGroup->transitionGroupShape(QPointF(snappedPos - m_selectionGroup->sceneBoundingRect().left(), 0));
                 collidingItems = scene()->items(shape, Qt::IntersectsItemShape);
                 collidingItems.removeAll(m_selectionGroup);
-                for (int i = 0; i < children.count(); i++) {
-                    if (children.at(i)->type() == GROUPWIDGET) {
+                for (int i = 0; i < children.count(); ++i) {
+                    if (children.at(i)->type() == GroupWidget) {
                         QList<QGraphicsItem *> subchildren = children.at(i)->childItems();
                         for (int j = 0; j < subchildren.count(); j++)
                             collidingItems.removeAll(subchildren.at(j));
                     }
                     collidingItems.removeAll(children.at(i));
                 }
-                for (int i = 0; i < collidingItems.count(); i++) {
-                    if (collidingItems.at(i)->type() == TRANSITIONWIDGET) {
+                for (int i = 0; i < collidingItems.count(); ++i) {
+                    if (collidingItems.at(i)->type() == TransitionWidget) {
                         collision = true;
                         break;
                     }
@@ -604,36 +616,36 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
         }
     }
 
-    if (m_tool == RAZORTOOL) {
+    if (m_tool == RazorTool) {
         setCursor(m_razorCursor);
-    } else if (m_tool == SPACERTOOL) {
+    } else if (m_tool == SpacerTool) {
         setCursor(m_spacerCursor);
     }
 
     QList<QGraphicsItem *> itemList = items(event->pos());
     QGraphicsRectItem *item = NULL;
-    OPERATIONTYPE opMode = NONE;
+    OperationType opMode = None;
 
     if (itemList.count() == 1 && itemList.at(0)->type() == GUIDEITEM) {
-        opMode = MOVEGUIDE;
-       setCursor(Qt::SplitHCursor);
-    } else for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET || itemList.at(i)->type() == TRANSITIONWIDGET) {
+        opMode = MoveGuide;
+        setCursor(Qt::SplitHCursor);
+    } else for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget || itemList.at(i)->type() == TransitionWidget) {
             item = (QGraphicsRectItem*) itemList.at(i);
-           break;
-       }
+            break;
+        }
     }
 
-    if (m_tool == SPACERTOOL) {
+    if (m_tool == SpacerTool) {
         event->accept();
         return;
     }
 
     if (item && event->buttons() == Qt::NoButton) {
         AbstractClipItem *clip = static_cast <AbstractClipItem*>(item);
-        if (m_tool == RAZORTOOL) {
+        if (m_tool == RazorTool) {
             // razor tool over a clip, display current frame in monitor
-            if (false && !m_blockRefresh && item->type() == AVWIDGET) {
+            if (false && !m_blockRefresh && item->type() == AVWidget) {
                 //TODO: solve crash when showing frame when moving razor over clip
                 emit showClipFrame(((ClipItem *) item)->baseClip(), QPoint(), false, mappedXPos - (clip->startPos() - clip->cropStart()).frames(m_document->fps()));
             }
@@ -643,12 +655,12 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
 
         if (m_selectionGroup && clip->parentItem() == m_selectionGroup) {
             // all other modes break the selection, so the user probably wants to move it
-            opMode = MOVE;
+            opMode = MoveOperation;
         } else {
-           if (clip->rect().width() * transform().m11() < 15) {
-               // If the item is very small, only allow move
-               opMode = MOVE;
-           }
+            if (clip->rect().width() * transform().m11() < 15) {
+                // If the item is very small, only allow move
+                opMode = MoveOperation;
+            }
             else opMode = clip->operationMode(mapToScene(event->pos()));
         }
 
@@ -662,55 +674,55 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
         m_moveOpMode = opMode;
         setTipAnimation(clip, opMode, size);
         ClipItem *ci = NULL;
-        if (item->type() == AVWIDGET)
+        if (item->type() == AVWidget)
             ci = static_cast <ClipItem *>(item);
         QString message;
-        if (opMode == MOVE) {
+        if (opMode == MoveOperation) {
             setCursor(Qt::OpenHandCursor);
             if (ci) {
                 message = ci->clipName() + i18n(":");
                 message.append(i18n(" Position:") + m_document->timecode().getDisplayTimecode(ci->info().startPos, KdenliveSettings::frametimecode()));
                 message.append(i18n(" Duration:") + m_document->timecode().getDisplayTimecode(ci->cropDuration(),  KdenliveSettings::frametimecode()));
-                if (clip->parentItem() && clip->parentItem()->type() == GROUPWIDGET) {
+                if (clip->parentItem() && clip->parentItem()->type() == GroupWidget) {
                     AbstractGroupItem *parent = static_cast <AbstractGroupItem *>(clip->parentItem());
                     if (clip->parentItem() == m_selectionGroup)
                         message.append(i18n(" Selection duration:"));
                     else
                         message.append(i18n(" Group duration:"));
                     message.append(m_document->timecode().getDisplayTimecode(parent->duration(), KdenliveSettings::frametimecode()));
-                    if (parent->parentItem() && parent->parentItem()->type() == GROUPWIDGET) {
+                    if (parent->parentItem() && parent->parentItem()->type() == GroupWidget) {
                         AbstractGroupItem *parent2 = static_cast <AbstractGroupItem *>(parent->parentItem());
                         message.append(i18n(" Selection duration:") + m_document->timecode().getDisplayTimecode(parent2->duration(), KdenliveSettings::frametimecode()));
                     }
                 }
             }
-        } else if (opMode == RESIZESTART) {
+        } else if (opMode == ResizeStart) {
             setCursor(KCursor("left_side", Qt::SizeHorCursor));
             if (ci)
                 message = i18n("Crop from start: ") + m_document->timecode().getDisplayTimecode(ci->cropStart(), KdenliveSettings::frametimecode());
-            if (item->type() == AVWIDGET && item->parentItem() && item->parentItem() != m_selectionGroup)
+            if (item->type() == AVWidget && item->parentItem() && item->parentItem() != m_selectionGroup)
                 message.append(i18n("Use Ctrl to resize only current item, otherwise all items in this group will be resized at once."));
-        } else if (opMode == RESIZEEND) {
+        } else if (opMode == ResizeEnd) {
             setCursor(KCursor("right_side", Qt::SizeHorCursor));
             if (ci)
                 message = i18n("Duration: ") + m_document->timecode().getDisplayTimecode(ci->cropDuration(), KdenliveSettings::frametimecode());
-            if (item->type() == AVWIDGET && item->parentItem() && item->parentItem() != m_selectionGroup)
+            if (item->type() == AVWidget && item->parentItem() && item->parentItem() != m_selectionGroup)
                 message.append(i18n("Use Ctrl to resize only current item, otherwise all items in this group will be resized at once."));
-        } else if (opMode == FADEIN || opMode == FADEOUT) {
+        } else if (opMode == FadeIn || opMode == FadeOut) {
             setCursor(Qt::PointingHandCursor);
-            if (ci && opMode == FADEIN && ci->fadeIn()) {
+            if (ci && opMode == FadeIn && ci->fadeIn()) {
                 message = i18n("Fade in duration: ");
                 message.append(m_document->timecode().getDisplayTimecodeFromFrames(ci->fadeIn(), KdenliveSettings::frametimecode()));
-            } else if (ci && opMode == FADEOUT && ci->fadeOut()) {
+            } else if (ci && opMode == FadeOut && ci->fadeOut()) {
                 message = i18n("Fade out duration: ");
                 message.append(m_document->timecode().getDisplayTimecodeFromFrames(ci->fadeOut(), KdenliveSettings::frametimecode()));
             } else {
                 message = i18n("Drag to add or resize a fade effect.");
             }
-        } else if (opMode == TRANSITIONSTART || opMode == TRANSITIONEND) {
+        } else if (opMode == TransitionStart || opMode == TransitionEnd) {
             setCursor(Qt::PointingHandCursor);
             message = i18n("Click to add a transition.");
-        } else if (opMode == KEYFRAME) {
+        } else if (opMode == KeyFrame) {
             setCursor(Qt::PointingHandCursor);
             emit displayMessage(i18n("Move keyframe above or below clip to remove it, double click to add a new one."), InformationMessage);
         }
@@ -718,21 +730,23 @@ void CustomTrackView::mouseMoveEvent(QMouseEvent * event)
         if (!message.isEmpty())
             emit displayMessage(message, InformationMessage);
     } // no clip under mouse
-    else if (m_tool == RAZORTOOL) {
+    else if (m_tool == RazorTool) {
         event->accept();
         return;
-    } else if (opMode == MOVEGUIDE) {
+    } else if (opMode == MoveGuide) {
         m_moveOpMode = opMode;
     } else {
         removeTipAnimation();
         setCursor(Qt::ArrowCursor);
         if (event->buttons() != Qt::NoButton && event->modifiers() == Qt::NoModifier) {
             QGraphicsView::mouseMoveEvent(event);
-            m_moveOpMode = SEEK;
-            seekCursorPos(mappedXPos);
-            slotCheckPositionScrolling();
+            m_moveOpMode = Seek;
+            if (mappedXPos != m_document->renderer()->getCurrentSeekPosition() && mappedXPos != cursorPos()) {
+                seekCursorPos(mappedXPos);
+                slotCheckPositionScrolling();
+            }
             return;
-        } else m_moveOpMode = NONE;
+        } else m_moveOpMode = None;
     }
     QGraphicsView::mouseMoveEvent(event);
 }
@@ -748,7 +762,7 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
     if (event->button() == Qt::MidButton) {
         emit playMonitor();
         m_blockRefresh = false;
-        m_operationMode = NONE;
+        m_operationMode = None;
         return;
     }
 
@@ -758,14 +772,14 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
         setDragMode(QGraphicsView::RubberBandDrag);
         if (!(event->modifiers() & Qt::ControlModifier)) {
             resetSelectionGroup();
-           if (m_dragItem) {
-               emit clipItemSelected(NULL);
-               m_dragItem = NULL;
-           }
+            if (m_dragItem) {
+                emit clipItemSelected(NULL);
+                m_dragItem = NULL;
+            }
             scene()->clearSelection();
         }
         m_blockRefresh = false;
-        m_operationMode = RUBBERSELECTION;
+        m_operationMode = RubberSelection;
         QGraphicsView::mousePressEvent(event);
         return;
     }
@@ -773,8 +787,8 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
     m_blockRefresh = true;
     m_dragGuide = NULL;
 
-    if (m_tool != RAZORTOOL) activateMonitor();
-    else if (m_document->renderer()->playSpeed() != 0.0) {
+    if (m_tool != RazorTool) activateMonitor();
+    else if (m_document->renderer()->isPlaying()) {
         m_document->renderer()->pause();
         return;
     }
@@ -782,15 +796,14 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
 
     // check item under mouse
     QList<QGraphicsItem *> collisionList = items(m_clickEvent);
-    if (event->modifiers() == Qt::ControlModifier && m_tool != SPACERTOOL && collisionList.count() == 0) {
+    if (event->modifiers() == Qt::ControlModifier && m_tool != SpacerTool && collisionList.count() == 0) {
         // Pressing Ctrl + left mouse button in an empty area scrolls the timeline
         setDragMode(QGraphicsView::ScrollHandDrag);
         m_blockRefresh = false;
-        m_operationMode = SCROLLTIMELINE;
-       QGraphicsView::mousePressEvent(event);
+        m_operationMode = ScrollTimeline;
+        QGraphicsView::mousePressEvent(event);
         return;
     }
-
     // if a guide and a clip were pressed, just select the guide
     for (int i = 0; i < collisionList.count(); ++i) {
         if (collisionList.at(i)->type() == GUIDEITEM) {
@@ -798,7 +811,7 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
             m_dragGuide = (Guide *) collisionList.at(i);
             if (event->button() == Qt::LeftButton) { // move it
                 m_dragGuide->setFlag(QGraphicsItem::ItemIsMovable, true);
-                m_operationMode = MOVEGUIDE;
+                m_operationMode = MoveGuide;
                 // deselect all clips so that only the guide will move
                 m_scene->clearSelection();
                 resetSelectionGroup(false);
@@ -815,27 +828,42 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
     AbstractGroupItem *dragGroup = NULL;
     AbstractClipItem *collisionClip = NULL;
     bool found = false;
+    QStringList lockedTracks;
+    double yOffset = 0;
+    m_selectionMutex.lock();
     while (!m_dragGuide && ct < collisionList.count()) {
-        if (collisionList.at(ct)->type() == AVWIDGET || collisionList.at(ct)->type() == TRANSITIONWIDGET) {
+        if (collisionList.at(ct)->type() == AVWidget || collisionList.at(ct)->type() == TransitionWidget) {
             collisionClip = static_cast <AbstractClipItem *>(collisionList.at(ct));
             if (collisionClip->isItemLocked() || !collisionClip->isEnabled()) {
-               ct++;
+                ct++;
                 continue;
-           }
+            }
             if (collisionClip == m_dragItem) {
                 collisionClip = NULL;
-           }
+            }
             else {
                 m_dragItem = collisionClip;
-           }
+            }
             found = true;
-           
-           m_dragItem->setProperty("y_absolute", mapToScene(m_clickEvent).y() - m_dragItem->scenePos().y());
+            for (int i = 0; i < m_document->tracksCount(); ++i) {
+                if (m_document->trackInfoAt(i).isLocked) lockedTracks << QString::number(m_document->tracksCount() - i - 1);
+            }
+            yOffset = mapToScene(m_clickEvent).y() - m_dragItem->scenePos().y();
+            m_dragItem->setProperty("y_absolute", yOffset);
+            m_dragItem->setProperty("locked_tracks", lockedTracks);
             m_dragItemInfo = m_dragItem->info();
-           if (m_selectionGroup) m_selectionGroup->setProperty("y_absolute", mapToScene(m_clickEvent).y() - m_dragItem->scenePos().y());
-            if (m_dragItem->parentItem() && m_dragItem->parentItem()->type() == GROUPWIDGET && m_dragItem->parentItem() != m_selectionGroup) {
-                dragGroup = static_cast <AbstractGroupItem *>(m_dragItem->parentItem());
-               dragGroup->setProperty("y_absolute", mapToScene(m_clickEvent).y() - m_dragItem->scenePos().y());
+            if (m_selectionGroup) {
+                m_selectionGroup->setProperty("y_absolute", yOffset);
+                m_selectionGroup->setProperty("locked_tracks", lockedTracks);
+            }
+            if (m_dragItem->parentItem() && m_dragItem->parentItem()->type() == GroupWidget && m_dragItem->parentItem() != m_selectionGroup) {
+                QGraphicsItem *topGroup = m_dragItem->parentItem();
+                while (topGroup->parentItem() && topGroup->parentItem()->type() == GroupWidget && topGroup->parentItem() != m_selectionGroup) {
+                    topGroup = topGroup->parentItem();
+                }
+                dragGroup = static_cast <AbstractGroupItem *>(topGroup);
+                dragGroup->setProperty("y_absolute", yOffset);
+                dragGroup->setProperty("locked_tracks", lockedTracks);
             }
             break;
         }
@@ -849,17 +877,17 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
 #if QT_VERSION >= 0x040800
     // Add shadow to dragged item, currently disabled because of painting artifacts
     /*if (m_dragItem) {
-       QGraphicsDropShadowEffect *eff = new QGraphicsDropShadowEffect();
-       eff->setBlurRadius(5);
-       eff->setOffset(3, 3);
-       m_dragItem->setGraphicsEffect(eff);
+    QGraphicsDropShadowEffect *eff = new QGraphicsDropShadowEffect();
+    eff->setBlurRadius(5);
+    eff->setOffset(3, 3);
+    m_dragItem->setGraphicsEffect(eff);
     }*/
 #endif
-    if (m_dragItem && m_dragItem->type() == TRANSITIONWIDGET && m_dragItem->isEnabled()) {
+    if (m_dragItem && m_dragItem->type() == TransitionWidget && m_dragItem->isEnabled()) {
         // update transition menu action
         m_autoTransition->setChecked(static_cast<Transition *>(m_dragItem)->isAutomatic());
         m_autoTransition->setEnabled(true);
-       // A transition is selected
+        // A transition is selected
         QPoint p;
         ClipItem *transitionClip = getClipItemAt(m_dragItemInfo.startPos, m_dragItemInfo.track);
         if (transitionClip && transitionClip->baseClip()) {
@@ -871,15 +899,15 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
         }
         emit transitionItemSelected(static_cast <Transition *>(m_dragItem), getPreviousVideoTrack(m_dragItem->track()), p);
     } else {
-       emit transitionItemSelected(NULL);
-       m_autoTransition->setEnabled(false);
+        emit transitionItemSelected(NULL);
+        m_autoTransition->setEnabled(false);
     }
     // context menu requested
     if (event->button() == Qt::RightButton) {
         if (!m_dragItem && !m_dragGuide) {
             // check if there is a guide close to mouse click
             QList<QGraphicsItem *> guidesCollisionList = items(event->pos().x() - 5, event->pos().y(), 10, 2); // a rect of height < 2 does not always collide with the guide
-            for (int i = 0; i < guidesCollisionList.count(); i++) {
+            for (int i = 0; i < guidesCollisionList.count(); ++i) {
                 if (guidesCollisionList.at(i)->type() == GUIDEITEM) {
                     m_dragGuide = static_cast <Guide *>(guidesCollisionList.at(i));
                     break;
@@ -893,18 +921,22 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
             m_dragGuide = static_cast <Guide *>(guidesCollisionList.at(0));*/
         }
 
-        m_operationMode = NONE;
+        m_operationMode = None;
+        if (dragGroup == NULL) {
+            if (m_dragItem && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup)
+                dragGroup = static_cast<AbstractGroupItem*> (m_dragItem->parentItem());
+        }
         displayContextMenu(event->globalPos(), m_dragItem, dragGroup);
         m_menuPosition = m_clickEvent;
     }
-
+    m_selectionMutex.unlock();
     // No item under click
-    if (m_dragItem == NULL || m_tool == SPACERTOOL) {
+    if (m_dragItem == NULL || m_tool == SpacerTool) {
         resetSelectionGroup(false);
         m_scene->clearSelection();
         //event->accept();
         updateClipTypeActions(NULL);
-        if (m_tool == SPACERTOOL) {
+        if (m_tool == SpacerTool) {
             QList<QGraphicsItem *> selection;
             if (event->modifiers() == Qt::ControlModifier) {
                 // Ctrl + click, select all items on track after click position
@@ -934,17 +966,18 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
 
             QList <GenTime> offsetList;
             // create group to hold selected items
+            m_selectionMutex.lock();
             m_selectionGroup = new AbstractGroupItem(m_document->fps());
             scene()->addItem(m_selectionGroup);
 
-            for (int i = 0; i < selection.count(); i++) {
-                if (selection.at(i)->parentItem() == 0 && (selection.at(i)->type() == AVWIDGET || selection.at(i)->type() == TRANSITIONWIDGET)) {
+            for (int i = 0; i < selection.count(); ++i) {
+                if (selection.at(i)->parentItem() == 0 && (selection.at(i)->type() == AVWidget || selection.at(i)->type() == TransitionWidget)) {
                     AbstractClipItem *item = static_cast<AbstractClipItem *>(selection.at(i));
                     if (item->isItemLocked()) continue;
                     offsetList.append(item->startPos());
                     offsetList.append(item->endPos());
                     m_selectionGroup->addItem(selection.at(i));
-                } else if (/*selection.at(i)->parentItem() == 0 && */selection.at(i)->type() == GROUPWIDGET) {
+                } else if (/*selection.at(i)->parentItem() == 0 && */selection.at(i)->type() == GroupWidget) {
                     if (static_cast<AbstractGroupItem *>(selection.at(i))->isItemLocked()) continue;
                     QList<QGraphicsItem *> children = selection.at(i)->childItems();
                     for (int j = 0; j < children.count(); j++) {
@@ -959,6 +992,7 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
                 }
             }
             m_spacerOffset = m_selectionGroup->sceneBoundingRect().left() - (int)(mapToScene(m_clickEvent).x());
+            m_selectionMutex.unlock();
             if (!offsetList.isEmpty()) {
                 qSort(offsetList);
                 QList <GenTime> cleandOffsetList;
@@ -971,9 +1005,9 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
                 }
                 updateSnapPoints(NULL, cleandOffsetList, true);
             }
-            m_operationMode = SPACER;
+            m_operationMode = Spacer;
         } else if (event->button() != Qt::RightButton) {
-           setCursor(Qt::ArrowCursor);
+            setCursor(Qt::ArrowCursor);
             seekCursorPos((int)(mapToScene(event->x(), 0).x()));
         }
         QGraphicsView::mousePressEvent(event);
@@ -982,17 +1016,17 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
     }
 
     // Razor tool
-    if (m_tool == RAZORTOOL && m_dragItem) {
+    if (m_tool == RazorTool && m_dragItem) {
         GenTime cutPos = GenTime((int)(mapToScene(event->pos()).x()), m_document->fps());
-        if (m_dragItem->type() == TRANSITIONWIDGET) {
+        if (m_dragItem->type() == TransitionWidget) {
             emit displayMessage(i18n("Cannot cut a transition"), ErrorMessage);
         } else {
             m_document->renderer()->pause();
             if (m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
                 razorGroup((AbstractGroupItem *)m_dragItem->parentItem(), cutPos);
             } else {
-                AbstractClipItem *clip = static_cast <AbstractClipItem *>(m_dragItem);
-                RazorClipCommand* command = new RazorClipCommand(this, clip->info(), cutPos);
+                ClipItem *clip = static_cast <ClipItem *>(m_dragItem);
+                RazorClipCommand* command = new RazorClipCommand(this, clip->info(), clip->effectList(), cutPos);
                 m_commandStack->push(command);
             }
             setDocumentModified();
@@ -1020,51 +1054,69 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
             // A refresh seems necessary otherwise in zoomed mode, some clips disappear
             viewport()->update();
         } else {
-           resetSelectionGroup();
-       }
+            resetSelectionGroup();
+        }
+        /*if () {
         dragGroup = NULL;
         if (m_dragItem->parentItem() && m_dragItem->parentItem()->type() == GROUPWIDGET) {
-            dragGroup = static_cast <AbstractGroupItem *>(m_dragItem->parentItem());
+        dragGroup = static_cast <AbstractGroupItem *>(m_dragItem->parentItem());
         }
+    }*/
 
         bool selected = !m_dragItem->isSelected();
-       QGraphicsView::mousePressEvent(event);
-         
+        m_dragItem->setZValue(99);
+        if (m_dragItem->parentItem()) m_dragItem->parentItem()->setZValue(99);
+        QGraphicsView::mousePressEvent(event);
+
         if (dragGroup) {
             dragGroup->setSelected(selected);
-           if (dragGroup->parentItem())
-               dragGroup->parentItem()->setSelected(selected);
-       }
+            QList<QGraphicsItem *> children = dragGroup->childItems();
+            for (int i = 0; i < children.count(); ++i) {
+                children.at(i)->setSelected(selected);
+            }
+            if (dragGroup->parentItem()) {
+                dragGroup->parentItem()->setSelected(selected);
+            }
+        }
         else
             m_dragItem->setSelected(selected);
-       if (selected == false) {
-           m_dragItem = NULL;
-       }
-        groupSelectedItems(QList <QGraphicsItem*>(), false, false, true);
-       if (m_dragItem) { 
-           ClipItem *clip = static_cast <ClipItem *>(m_dragItem);
-           updateClipTypeActions(dragGroup == NULL ? clip : NULL);
-           m_pasteEffectsAction->setEnabled(m_copiedItems.count() == 1);
-       }
-       else updateClipTypeActions(NULL);
+        if (selected == false) {
+            m_dragItem = NULL;
+        }
+        groupSelectedItems(QList <QGraphicsItem*>(), false, true);
+        m_selectionMutex.lock();
+        if (m_selectionGroup) {
+            m_selectionGroup->setProperty("y_absolute", yOffset);
+            m_selectionGroup->setProperty("locked_tracks", lockedTracks);
+        }
+        m_selectionMutex.unlock();
+        if (m_dragItem) {
+            ClipItem *clip = static_cast <ClipItem *>(m_dragItem);
+            updateClipTypeActions(dragGroup == NULL ? clip : NULL);
+            m_pasteEffectsAction->setEnabled(m_copiedItems.count() == 1);
+        }
+        else updateClipTypeActions(NULL);
     }
     else {
-       QGraphicsView::mousePressEvent(event);
-       if (m_selectionGroup) {
-           QList<QGraphicsItem *> children = m_selectionGroup->childItems();
-           for (int i = 0; i < children.count(); i++) {
-               children.at(i)->setSelected(itemSelected);
-           }
+        QGraphicsView::mousePressEvent(event);
+        m_selectionMutex.lock();
+        if (m_selectionGroup) {
+            QList<QGraphicsItem *> children = m_selectionGroup->childItems();
+            for (int i = 0; i < children.count(); ++i) {
+                children.at(i)->setSelected(itemSelected);
+            }
             m_selectionGroup->setSelected(itemSelected);
-           
-       }
-       if (dragGroup)
+
+        }
+        if (dragGroup) {
             dragGroup->setSelected(itemSelected);
-       m_dragItem->setSelected(itemSelected);
+        }
+        m_dragItem->setSelected(itemSelected);
+        m_selectionMutex.unlock();
     }
 
     if (collisionClip != NULL || m_dragItem == NULL) {
-        if (m_dragItem && m_dragItem->type() == AVWIDGET && !m_dragItem->isItemLocked()) {
+        if (m_dragItem && m_dragItem->type() == AVWidget && !m_dragItem->isItemLocked()) {
             ClipItem *selected = static_cast <ClipItem*>(m_dragItem);
             emit clipItemSelected(selected);
         } else {
@@ -1077,31 +1129,32 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
     //QGraphicsView::mousePressEvent(event);
 
     if (m_dragItem) {
-       m_clickPoint = QPoint((int)(mapToScene(event->pos()).x() - m_dragItem->startPos().frames(m_document->fps())), (int)(event->pos().y() - m_dragItem->pos().y()));
-    if (m_selectionGroup && m_dragItem->parentItem() == m_selectionGroup) {
-        // all other modes break the selection, so the user probably wants to move it
-        m_operationMode = MOVE;
-    } else {
-       if (m_dragItem->rect().width() * transform().m11() < 15) {
-           // If the item is very small, only allow move
-           m_operationMode = MOVE;
-       }
-        else m_operationMode = m_dragItem->operationMode(mapToScene(event->pos()));
-    }
-    } else m_operationMode = NONE;
+        m_clickPoint = QPoint((int)(mapToScene(event->pos()).x() - m_dragItem->startPos().frames(m_document->fps())), (int)(event->pos().y() - m_dragItem->pos().y()));
+        if (m_selectionGroup && m_dragItem->parentItem() == m_selectionGroup) {
+            // all other modes break the selection, so the user probably wants to move it
+            m_operationMode = MoveOperation;
+        } else {
+            if (m_dragItem->rect().width() * transform().m11() < 15) {
+                // If the item is very small, only allow move
+                m_operationMode = MoveOperation;
+            }
+            else m_operationMode = m_dragItem->operationMode(mapToScene(event->pos()));
+        }
+    } else m_operationMode = None;
     m_controlModifier = (event->modifiers() == Qt::ControlModifier);
 
     // Update snap points
     if (m_selectionGroup == NULL) {
-        if (m_operationMode == RESIZEEND || m_operationMode == RESIZESTART)
+        if (m_operationMode == ResizeEnd || m_operationMode == ResizeStart)
             updateSnapPoints(NULL);
         else
             updateSnapPoints(m_dragItem);
     } else {
+        m_selectionMutex.lock();
         QList <GenTime> offsetList;
         QList<QGraphicsItem *> children = m_selectionGroup->childItems();
-        for (int i = 0; i < children.count(); i++) {
-            if (children.at(i)->type() == AVWIDGET || children.at(i)->type() == TRANSITIONWIDGET) {
+        for (int i = 0; i < children.count(); ++i) {
+            if (children.at(i)->type() == AVWidget || children.at(i)->type() == TransitionWidget) {
                 AbstractClipItem *item = static_cast <AbstractClipItem *>(children.at(i));
                 offsetList.append(item->startPos());
                 offsetList.append(item->endPos());
@@ -1119,15 +1172,16 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
             }
             updateSnapPoints(NULL, cleandOffsetList, true);
         }
+        m_selectionMutex.unlock();
     }
 
-    if (m_operationMode == KEYFRAME) {
+    if (m_operationMode == KeyFrame) {
         m_dragItem->updateSelectedKeyFrame();
         m_blockRefresh = false;
         return;
-    } else if (m_operationMode == MOVE) {
+    } else if (m_operationMode == MoveOperation) {
         setCursor(Qt::ClosedHandCursor);
-    } else if (m_operationMode == TRANSITIONSTART && event->modifiers() != Qt::ControlModifier) {
+    } else if (m_operationMode == TransitionStart && event->modifiers() != Qt::ControlModifier) {
         ItemInfo info;
         info.startPos = m_dragItem->startPos();
         info.track = m_dragItem->track();
@@ -1149,8 +1203,8 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
         QRectF r(info.startPos.frames(m_document->fps()), startY, (info.endPos - info.startPos).frames(m_document->fps()), m_tracksHeight / 2);
         QList<QGraphicsItem *> selection = m_scene->items(r);
         bool transitionAccepted = true;
-        for (int i = 0; i < selection.count(); i++) {
-            if (selection.at(i)->type() == TRANSITIONWIDGET) {
+        for (int i = 0; i < selection.count(); ++i) {
+            if (selection.at(i)->type() == TransitionWidget) {
                 Transition *tr = static_cast <Transition *>(selection.at(i));
                 if (tr->startPos() - info.startPos > GenTime(5, m_document->fps())) {
                     if (tr->startPos() < info.endPos) info.endPos = tr->startPos();
@@ -1159,7 +1213,7 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
         }
         if (transitionAccepted) slotAddTransition((ClipItem *) m_dragItem, info, transitiontrack);
         else emit displayMessage(i18n("Cannot add transition"), ErrorMessage);
-    } else if (m_operationMode == TRANSITIONEND && event->modifiers() != Qt::ControlModifier) {
+    } else if (m_operationMode == TransitionEnd && event->modifiers() != Qt::ControlModifier) {
         ItemInfo info;
         info.endPos = GenTime(m_dragItem->endPos().frames(m_document->fps()), m_document->fps());
         info.track = m_dragItem->track();
@@ -1170,8 +1224,10 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
             info.startPos = transitionClip->startPos();
         } else {
             GenTime transitionDuration(65, m_document->fps());
-            if (m_dragItem->cropDuration() < transitionDuration) info.startPos = m_dragItem->startPos();
-            else info.startPos = info.endPos - transitionDuration;
+            if (m_dragItem->cropDuration() < transitionDuration)
+                info.startPos = m_dragItem->startPos();
+            else
+                info.startPos = info.endPos - transitionDuration;
         }
         if (info.endPos == info.startPos) info.startPos = info.endPos - GenTime(65, m_document->fps());
         QDomElement transition = MainWindow::transitions.getEffectByTag("luma", "dissolve").cloneNode().toElement();
@@ -1182,8 +1238,8 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
         QRectF r(info.startPos.frames(m_document->fps()), startY, (info.endPos - info.startPos).frames(m_document->fps()), m_tracksHeight / 2);
         QList<QGraphicsItem *> selection = m_scene->items(r);
         bool transitionAccepted = true;
-        for (int i = 0; i < selection.count(); i++) {
-            if (selection.at(i)->type() == TRANSITIONWIDGET) {
+        for (int i = 0; i < selection.count(); ++i) {
+            if (selection.at(i)->type() == TransitionWidget) {
                 Transition *tr = static_cast <Transition *>(selection.at(i));
                 if (info.endPos - tr->endPos() > GenTime(5, m_document->fps())) {
                     if (tr->endPos() > info.startPos) info.startPos = tr->endPos();
@@ -1193,21 +1249,21 @@ void CustomTrackView::mousePressEvent(QMouseEvent * event)
         if (transitionAccepted) slotAddTransition((ClipItem *) m_dragItem, info, transitiontrack, transition);
         else emit displayMessage(i18n("Cannot add transition"), ErrorMessage);
 
-    } else if ((m_operationMode == RESIZESTART || m_operationMode == RESIZEEND) && m_selectionGroup) {
+    } else if ((m_operationMode == ResizeStart || m_operationMode == ResizeEnd) && m_selectionGroup) {
         resetSelectionGroup(false);
         m_dragItem->setSelected(true);
     }
     m_blockRefresh = false;
 }
 
-void CustomTrackView::rebuildGroup(int childTrack, GenTime childPos)
+void CustomTrackView::rebuildGroup(int childTrack, const GenTime &childPos)
 {
     const QPointF p((int)childPos.frames(m_document->fps()), childTrack * m_tracksHeight + m_tracksHeight / 2);
     QList<QGraphicsItem *> list = scene()->items(p);
     AbstractGroupItem *group = NULL;
-    for (int i = 0; i < list.size(); i++) {
+    for (int i = 0; i < list.size(); ++i) {
         if (!list.at(i)->isEnabled()) continue;
-        if (list.at(i)->type() == GROUPWIDGET) {
+        if (list.at(i)->type() == GroupWidget) {
             group = static_cast <AbstractGroupItem *>(list.at(i));
             break;
         }
@@ -1218,18 +1274,22 @@ void CustomTrackView::rebuildGroup(int childTrack, GenTime childPos)
 void CustomTrackView::rebuildGroup(AbstractGroupItem *group)
 {
     if (group) {
+        m_selectionMutex.lock();
+        if (group == m_selectionGroup) m_selectionGroup = NULL;
         QList <QGraphicsItem *> children = group->childItems();
         m_document->clipManager()->removeGroup(group);
-       /*for (int i = 0; i < children.count(); i++) {
-           group->removeFromGroup(children.at(i));
-       }*/
-       scene()->destroyItemGroup(group);
-        groupSelectedItems(children, false, true, true);
+        for (int i = 0; i < children.count(); ++i) {
+            group->removeFromGroup(children.at(i));
+        }
+        scene()->destroyItemGroup(group);
+        m_selectionMutex.unlock();
+        groupSelectedItems(children, group != m_selectionGroup, true);
     }
 }
 
 void CustomTrackView::resetSelectionGroup(bool selectItems)
 {
+    QMutexLocker lock(&m_selectionMutex);
     if (m_selectionGroup) {
         // delete selection group
         bool snap = KdenliveSettings::snaptopoints();
@@ -1237,24 +1297,27 @@ void CustomTrackView::resetSelectionGroup(bool selectItems)
 
         QList<QGraphicsItem *> children = m_selectionGroup->childItems();
         scene()->destroyItemGroup(m_selectionGroup);
-       m_selectionGroup = NULL;
-        for (int i = 0; i < children.count(); i++) {
-            if (children.at(i)->parentItem() == 0 && (children.at(i)->type() == AVWIDGET || children.at(i)->type() == TRANSITIONWIDGET)) {
-                if (!static_cast <AbstractClipItem *>(children.at(i))->isItemLocked()) {
+        m_selectionGroup = NULL;
+        for (int i = 0; i < children.count(); ++i) {
+            if (children.at(i)->parentItem() == 0) {
+                if ((children.at(i)->type() == AVWidget || children.at(i)->type() == TransitionWidget)) {
+                    if (!static_cast <AbstractClipItem *>(children.at(i))->isItemLocked()) {
+                        children.at(i)->setFlag(QGraphicsItem::ItemIsMovable, true);
+                        children.at(i)->setSelected(selectItems);
+                    }
+                } else if (children.at(i)->type() == GroupWidget) {
                     children.at(i)->setFlag(QGraphicsItem::ItemIsMovable, true);
                     children.at(i)->setSelected(selectItems);
                 }
-            } else if (children.at(i)->type() == GROUPWIDGET) {
-                children.at(i)->setFlag(QGraphicsItem::ItemIsMovable, true);
-                children.at(i)->setSelected(selectItems);
             }
         }
         KdenliveSettings::setSnaptopoints(snap);
     }
 }
 
-void CustomTrackView::groupSelectedItems(QList <QGraphicsItem *> selection, bool force, bool createNewGroup, bool selectNewGroup)
+void CustomTrackView::groupSelectedItems(QList <QGraphicsItem *> selection, bool createNewGroup, bool selectNewGroup)
 {
+    QMutexLocker lock(&m_selectionMutex);
     if (m_selectionGroup) {
         kDebug() << "///// ERROR, TRYING TO OVERRIDE EXISTING GROUP";
         return;
@@ -1264,29 +1327,43 @@ void CustomTrackView::groupSelectedItems(QList <QGraphicsItem *> selection, bool
     QSet <QGraphicsItemGroup *> groupsList;
     QSet <QGraphicsItem *> itemsList;
 
-    for (int i = 0; i < selection.count(); i++) {
-       if (selection.at(i)->type() == GROUPWIDGET) {
-           groupsList.insert(static_cast<AbstractGroupItem*> (selection.at(i)));
-       }
-    }
-    for (int i = 0; i < selection.count(); i++) {
-       if (selection.at(i)->type() == AVWIDGET || selection.at(i)->type() == TRANSITIONWIDGET) {
-           if (selection.at(i)->parentItem() && selection.at(i)->parentItem()->type() == GROUPWIDGET) {
-               groupsList.insert(static_cast <QGraphicsItemGroup *> (selection.at(i)->parentItem()));
-           }
-           else {
-               itemsList.insert(selection.at(i));
-           }
-       }
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selectNewGroup) selection.at(i)->setSelected(true);
+        if (selection.at(i)->type() == GroupWidget) {
+            AbstractGroupItem *it = static_cast <AbstractGroupItem *> (selection.at(i));
+            while (it->parentItem() && it->parentItem()->type() == GroupWidget) {
+                it = static_cast <AbstractGroupItem *>(it->parentItem());
+            }
+            if (!it || it->isItemLocked()) continue;
+            groupsList.insert(it);
+        }
+    }
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget || selection.at(i)->type() == TransitionWidget) {
+            if (selection.at(i)->parentItem() && selection.at(i)->parentItem()->type() == GroupWidget) {
+                AbstractGroupItem *it = static_cast <AbstractGroupItem *> (selection.at(i)->parentItem());
+                while (it->parentItem() && it->parentItem()->type() == GroupWidget) {
+                    it = static_cast <AbstractGroupItem *>(it->parentItem());
+                }
+                if (!it || it->isItemLocked()) continue;
+                groupsList.insert(it);
+            }
+            else {
+                AbstractClipItem *it = static_cast<AbstractClipItem *> (selection.at(i));
+                if (!it || it->isItemLocked()) continue;
+                itemsList.insert(selection.at(i));
+            }
+        }
     }
     if (itemsList.isEmpty() && groupsList.isEmpty()) return;
-    if (itemsList.count() == 1) {
-       // only one item selected:
-       QSetIterator<QGraphicsItem *> it(itemsList);
-       m_dragItem = static_cast<AbstractClipItem *>(it.next());
-       m_dragItem->setSelected(true);
+    if (itemsList.count() == 1 && groupsList.isEmpty()) {
+        // only one item selected:
+        QSetIterator<QGraphicsItem *> it(itemsList);
+        m_dragItem = static_cast<AbstractClipItem *>(it.next());
+        m_dragItem->setSelected(true);
+        return;
     }
-    
+
     QRectF rectUnion;
     // Find top left position of selection
     foreach (const QGraphicsItemGroup *value, groupsList) {
@@ -1295,56 +1372,59 @@ void CustomTrackView::groupSelectedItems(QList <QGraphicsItem *> selection, bool
     foreach (const QGraphicsItem *value, itemsList) {
         rectUnion = rectUnion.united(value->sceneBoundingRect());
     }
-    if (force || selection.count() > 1) {
-        bool snap = KdenliveSettings::snaptopoints();
-        KdenliveSettings::setSnaptopoints(false);
-        if (createNewGroup) {
-            AbstractGroupItem *newGroup = m_document->clipManager()->createGroup();
-            newGroup->setPos(rectUnion.left(), rectUnion.top() - 1);
-            QPointF diff = newGroup->pos();
-            newGroup->translate(-diff.x(), -diff.y());
-            //newGroup->translate((int) -rectUnion.left(), (int) -rectUnion.top() + 1);
-
-            scene()->addItem(newGroup);
-            // Check if we are trying to include a group in a group
-           foreach (QGraphicsItemGroup *value, groupsList) {
-               QList<QGraphicsItem *> children = value->childItems();
-               for (int i = 0; i < children.count(); i++) {
-                   if (children.at(i)->type() == AVWIDGET || children.at(i)->type() == TRANSITIONWIDGET)
-                       itemsList.insert(children.at(i));
-               }
-               AbstractGroupItem *grp = static_cast<AbstractGroupItem *>(value);
-               m_document->clipManager()->removeGroup(grp);
-               scene()->destroyItemGroup(grp);
-           }
-
-           foreach (QGraphicsItem *value, itemsList) {
-                newGroup->addItem(value);
-            }
-            KdenliveSettings::setSnaptopoints(snap);
-           if (selectNewGroup) newGroup->setSelected(true);
-        } else {
-            m_selectionGroup = new AbstractGroupItem(m_document->fps());
-            m_selectionGroup->setPos(rectUnion.left(), rectUnion.top() - 1);
-            QPointF diff = m_selectionGroup->pos();
-            //m_selectionGroup->translate((int) - rectUnion.left(), (int) -rectUnion.top() + 1);
-            m_selectionGroup->translate(- diff.x(), -diff.y());
+    bool snap = KdenliveSettings::snaptopoints();
+    KdenliveSettings::setSnaptopoints(false);
+    if (createNewGroup) {
+        AbstractGroupItem *newGroup = m_document->clipManager()->createGroup();
+        newGroup->setPos(rectUnion.left(), rectUnion.top() - 1);
+        QPointF diff = newGroup->pos();
+        newGroup->translate(-diff.x(), -diff.y());
+        //newGroup->translate((int) -rectUnion.left(), (int) -rectUnion.top() + 1);
 
-            scene()->addItem(m_selectionGroup);
-            foreach (QGraphicsItemGroup *value, groupsList) {
-                m_selectionGroup->addItem(value);
-            }
-            foreach (QGraphicsItem *value, itemsList) {
-                m_selectionGroup->addItem(value);
-            }
-            KdenliveSettings::setSnaptopoints(snap);
-            if (m_selectionGroup) {
-                m_selectionGroupInfo.startPos = GenTime(m_selectionGroup->scenePos().x(), m_document->fps());
-                m_selectionGroupInfo.track = m_selectionGroup->track();
-               if (selectNewGroup) m_selectionGroup->setSelected(true);
+        // Check if we are trying to include a group in a group
+        foreach (QGraphicsItemGroup *value, groupsList) {
+            newGroup->addItem(value);
+        }
+
+        foreach (QGraphicsItemGroup *value, groupsList) {
+            QList<QGraphicsItem *> children = value->childItems();
+            for (int i = 0; i < children.count(); ++i) {
+                if (children.at(i)->type() == AVWidget || children.at(i)->type() == TransitionWidget)
+                    itemsList.insert(children.at(i));
             }
+            AbstractGroupItem *grp = static_cast<AbstractGroupItem *>(value);
+            m_document->clipManager()->removeGroup(grp);
+            if (grp == m_selectionGroup) m_selectionGroup = NULL;
+            scene()->destroyItemGroup(grp);
+        }
+
+        foreach (QGraphicsItem *value, itemsList) {
+            newGroup->addItem(value);
+        }
+        scene()->addItem(newGroup);
+        KdenliveSettings::setSnaptopoints(snap);
+        if (selectNewGroup) newGroup->setSelected(true);
+    } else {
+        m_selectionGroup = new AbstractGroupItem(m_document->fps());
+        m_selectionGroup->setPos(rectUnion.left(), rectUnion.top() - 1);
+        QPointF diff = m_selectionGroup->pos();
+        //m_selectionGroup->translate((int) - rectUnion.left(), (int) -rectUnion.top() + 1);
+        m_selectionGroup->translate(- diff.x(), -diff.y());
+
+        scene()->addItem(m_selectionGroup);
+        foreach (QGraphicsItemGroup *value, groupsList) {
+            m_selectionGroup->addItem(value);
         }
-    } else resetSelectionGroup();
+        foreach (QGraphicsItem *value, itemsList) {
+            m_selectionGroup->addItem(value);
+        }
+        KdenliveSettings::setSnaptopoints(snap);
+        if (m_selectionGroup) {
+            m_selectionGroupInfo.startPos = GenTime(m_selectionGroup->scenePos().x(), m_document->fps());
+            m_selectionGroupInfo.track = m_selectionGroup->track();
+            if (selectNewGroup) m_selectionGroup->setSelected(true);
+        }
+    }
 }
 
 void CustomTrackView::mouseDoubleClickEvent(QMouseEvent *event)
@@ -1375,10 +1455,14 @@ void CustomTrackView::mouseDoubleClickEvent(QMouseEvent *event)
         } else*/  {
             // add keyframe
             GenTime keyFramePos = GenTime((int)(mapToScene(event->pos()).x()), m_document->fps()) - m_dragItem->startPos() + m_dragItem->cropStart();
+            int single = m_dragItem->checkForSingleKeyframe();
             int val = m_dragItem->addKeyFrame(keyFramePos, mapToScene(event->pos()).toPoint().y());
             ClipItem * item = static_cast <ClipItem *>(m_dragItem);
-            //QString previous = item->keyframes(item->selectedEffectIndex());
             QDomElement oldEffect = item->selectedEffect().cloneNode().toElement();
+            if (single > -1) {
+                item->insertKeyframe(item->getEffectAtIndex(item->selectedEffectIndex()), (item->cropStart() + item->cropDuration()).frames(m_document->fps()) - 1, single);
+            }
+            //QString previous = item->keyframes(item->selectedEffectIndex());
             item->insertKeyframe(item->getEffectAtIndex(item->selectedEffectIndex()), keyFramePos.frames(m_document->fps()), val);
             //item->updateKeyframeEffect();
             //QString next = item->keyframes(item->selectedEffectIndex());
@@ -1422,7 +1506,7 @@ void CustomTrackView::editItemDuration()
         return;
     }
 
-    if (item->type() == GROUPWIDGET || (item->parentItem() && item->parentItem()->type() == GROUPWIDGET)) {
+    if (item->type() == GroupWidget || (item->parentItem() && item->parentItem()->type() == GroupWidget)) {
         emit displayMessage(i18n("Cannot edit an item in a group"), ErrorMessage);
         return;
     }
@@ -1430,17 +1514,17 @@ void CustomTrackView::editItemDuration()
     if (!item->isItemLocked()) {
         GenTime minimum;
         GenTime maximum;
-        if (item->type() == TRANSITIONWIDGET)
+        if (item->type() == TransitionWidget)
             getTransitionAvailableSpace(item, minimum, maximum);
         else
             getClipAvailableSpace(item, minimum, maximum);
 
         QPointer<ClipDurationDialog> d = new ClipDurationDialog(item,
-                               m_document->timecode(), minimum, maximum, this);
+                                                                m_document->timecode(), minimum, maximum, this);
         if (d->exec() == QDialog::Accepted) {
             ItemInfo clipInfo = item->info();
             ItemInfo startInfo = clipInfo;
-            if (item->type() == TRANSITIONWIDGET) {
+            if (item->type() == TransitionWidget) {
                 // move & resize transition
                 clipInfo.startPos = d->startPos();
                 clipInfo.endPos = clipInfo.startPos + d->duration();
@@ -1517,14 +1601,14 @@ void CustomTrackView::displayContextMenu(QPoint pos, AbstractClipItem *clip, Abs
         m_timelineContextClipMenu->popup(pos);
     } else {
         m_ungroupAction->setEnabled(false);
-        if (clip->type() == AVWIDGET) {
+        if (clip->type() == AVWidget) {
             ClipItem *item = static_cast <ClipItem*>(clip);
             //build go to marker menu
             if (item->baseClip()) {
                 QList <CommentedTime> markers = item->baseClip()->commentedSnapMarkers();
-                int offset = item->startPos().frames(m_document->fps());
+                int offset = (item->startPos()- item->cropStart()).frames(m_document->fps());
                 if (!markers.isEmpty()) {
-                    for (int i = 0; i < markers.count(); i++) {
+                    for (int i = 0; i < markers.count(); ++i) {
                         int pos = (int) markers.at(i).time().frames(m_document->timecode().fps());
                         QString position = m_document->timecode().getTimecode(markers.at(i).time()) + ' ' + markers.at(i).comment();
                         QAction *go = m_markerMenu->addAction(position);
@@ -1536,7 +1620,7 @@ void CustomTrackView::displayContextMenu(QPoint pos, AbstractClipItem *clip, Abs
             updateClipTypeActions(item);
             m_pasteEffectsAction->setEnabled(m_copiedItems.count() == 1);
             m_timelineContextClipMenu->popup(pos);
-        } else if (clip->type() == TRANSITIONWIDGET) {
+        } else if (clip->type() == TransitionWidget) {
             m_timelineContextTransitionMenu->popup(pos);
         }
     }
@@ -1562,22 +1646,28 @@ void CustomTrackView::insertClipCut(DocClipBase *clip, int in, int out)
     pasteInfo.startPos = GenTime(m_cursorPos, m_document->fps());
     pasteInfo.endPos = pasteInfo.startPos + info.endPos;
     pasteInfo.track = selectedTrack();
-    bool ok = canBePastedTo(pasteInfo, AVWIDGET);
+    bool ok = canBePastedTo(pasteInfo, AVWidget);
     if (!ok) {
-       // Cannot be inserted at cursor pos, insert at end of track
-       int duration = m_document->renderer()->mltTrackDuration(m_document->tracksCount() - pasteInfo.track) + 1;
-       pasteInfo.startPos = GenTime(duration, m_document->fps());
-       pasteInfo.endPos = pasteInfo.startPos + info.endPos;
-       ok = canBePastedTo(pasteInfo, AVWIDGET);
+        // Cannot be inserted at cursor pos, insert at end of track
+        int duration = m_document->renderer()->mltTrackDuration(m_document->tracksCount() - pasteInfo.track) + 1;
+        pasteInfo.startPos = GenTime(duration, m_document->fps());
+        pasteInfo.endPos = pasteInfo.startPos + info.endPos;
+        ok = canBePastedTo(pasteInfo, AVWidget);
     }
     if (!ok) {
         emit displayMessage(i18n("Cannot insert clip in timeline"), ErrorMessage);
         return;
     }
 
-    AddTimelineClipCommand *command = new AddTimelineClipCommand(this, clip->toXML(), clip->getId(), pasteInfo, EffectsList(), m_scene->editMode() == OVERWRITEEDIT, m_scene->editMode() == INSERTEDIT, true, false);
-    updateTrackDuration(pasteInfo.track, command);
-    m_commandStack->push(command);
+    // Add refresh command for undo
+    QUndoCommand *addCommand = new QUndoCommand();
+    addCommand->setText(i18n("Add timeline clip"));
+    new RefreshMonitorCommand(this, false, true, addCommand);
+    new AddTimelineClipCommand(this, clip->toXML(), clip->getId(), pasteInfo, EffectsList(), m_scene->editMode() == OVERWRITEEDIT, m_scene->editMode() == INSERTEDIT, true, false, addCommand);
+    new RefreshMonitorCommand(this, true, false, addCommand);
+    updateTrackDuration(pasteInfo.track, addCommand);
+    
+    m_commandStack->push(addCommand);
 
     selectClip(true, false);
     // Automatic audio split
@@ -1587,10 +1677,17 @@ void CustomTrackView::insertClipCut(DocClipBase *clip, int in, int out)
 
 bool CustomTrackView::insertDropClips(const QMimeData *data, const QPoint &pos)
 {
+    QPointF framePos = mapToScene(pos);
+    int track = framePos.y() / KdenliveSettings::trackheight();
+    m_clipDrag = data->hasFormat("kdenlive/clip") || data->hasFormat("kdenlive/producerslist");
+    // This is not a clip drag, maybe effect or other...
+    if (!m_clipDrag) return false;
+    m_scene->clearSelection();
+    m_dragItem = NULL;
+    resetSelectionGroup(false);
+    QMutexLocker lock(&m_selectionMutex);
+    if (track < 0 || track > m_document->tracksCount() - 1 || m_document->trackInfoAt(m_document->tracksCount() - track - 1).isLocked) return true;
     if (data->hasFormat("kdenlive/clip")) {
-        m_clipDrag = true;
-        m_scene->clearSelection();
-        resetSelectionGroup(false);
         QStringList list = QString(data->data("kdenlive/clip")).split(';');
         DocClipBase *clip = m_document->getBaseClip(list.at(0));
         if (clip == NULL) {
@@ -1601,7 +1698,6 @@ bool CustomTrackView::insertDropClips(const QMimeData *data, const QPoint &pos)
             emit displayMessage(i18n("Clip not ready"), ErrorMessage);
             return false;
         }
-        QPointF framePos = mapToScene(pos);
         ItemInfo info;
         info.startPos = GenTime();
         info.cropStart = GenTime(list.at(1).toInt(), m_document->fps());
@@ -1616,7 +1712,7 @@ bool CustomTrackView::insertDropClips(const QMimeData *data, const QPoint &pos)
         pasteInfo.track = (int)(framePos.y() / m_tracksHeight);
         framePos.setX((int)(framePos.x() + 0.5));
         framePos.setY(pasteInfo.track * m_tracksHeight);
-        if (!canBePastedTo(pasteInfo, AVWIDGET)) {
+        if (!canBePastedTo(pasteInfo, AVWidget)) {
             return true;
         }
         m_selectionGroup = new AbstractGroupItem(m_document->fps());
@@ -1626,19 +1722,19 @@ bool CustomTrackView::insertDropClips(const QMimeData *data, const QPoint &pos)
         QList <GenTime> offsetList;
         offsetList.append(info.endPos);
         updateSnapPoints(NULL, offsetList);
+        QStringList lockedTracks;
+        for (int i = 0; i < m_document->tracksCount(); ++i) {
+            if (m_document->trackInfoAt(i).isLocked) lockedTracks << QString::number(m_document->tracksCount() - i - 1);
+        }
+        m_selectionGroup->setProperty("locked_tracks", lockedTracks);
         m_selectionGroup->setPos(framePos);
         scene()->addItem(m_selectionGroup);
         m_selectionGroup->setSelected(true);
-        return true;
     } else if (data->hasFormat("kdenlive/producerslist")) {
-        m_clipDrag = true;
         QStringList ids = QString(data->data("kdenlive/producerslist")).split(';');
-        m_scene->clearSelection();
-        resetSelectionGroup(false);
 
         QList <GenTime> offsetList;
         QList <ItemInfo> infoList;
-        QPointF framePos = mapToScene(pos);
         GenTime start = GenTime((int)(framePos.x() + 0.5), m_document->fps());
         int track = (int)(framePos.y() / m_tracksHeight);
         framePos.setX((int)(framePos.x() + 0.5));
@@ -1646,7 +1742,7 @@ bool CustomTrackView::insertDropClips(const QMimeData *data, const QPoint &pos)
 
         // Check if clips can be inserted at that position
         for (int i = 0; i < ids.size(); ++i) {
-           QString clipData = ids.at(i);
+            QString clipData = ids.at(i);
             DocClipBase *clip = m_document->getBaseClip(clipData.section('/', 0, 0));
             if (clip == NULL) {
                 kDebug() << " WARNING))))))))) CLIP NOT FOUND : " << ids.at(i);
@@ -1658,62 +1754,72 @@ bool CustomTrackView::insertDropClips(const QMimeData *data, const QPoint &pos)
             }
             ItemInfo info;
             info.startPos = start;
-           if (clipData.contains('/')) {
-               // this is a clip zone, set in / out
-               int in = clipData.section('/', 1, 1).toInt();
-               int out = clipData.section('/', 2, 2).toInt();
-               info.cropStart = GenTime(in, m_document->fps());
-               info.cropDuration = GenTime(out - in, m_document->fps());
-           }
+            if (clipData.contains('/')) {
+                // this is a clip zone, set in / out
+                int in = clipData.section('/', 1, 1).toInt();
+                int out = clipData.section('/', 2, 2).toInt();
+                info.cropStart = GenTime(in, m_document->fps());
+                info.cropDuration = GenTime(out - in, m_document->fps());
+            }
             else {
-               info.cropDuration = clip->duration();
-           }
-           info.endPos = info.startPos + info.cropDuration;
+                info.cropDuration = clip->duration();
+            }
+            info.endPos = info.startPos + info.cropDuration;
             info.track = track;
             infoList.append(info);
             start += info.cropDuration;
         }
-        if (!canBePastedTo(infoList, AVWIDGET)) {
+        if (!canBePastedTo(infoList, AVWidget)) {
             return true;
         }
-        m_selectionGroup = new AbstractGroupItem(m_document->fps());
+        if (ids.size() > 1) m_selectionGroup = new AbstractGroupItem(m_document->fps());
         start = GenTime();
         for (int i = 0; i < ids.size(); ++i) {
             QString clipData = ids.at(i);
             DocClipBase *clip = m_document->getBaseClip(clipData.section('/', 0, 0));
             ItemInfo info;
             info.startPos = start;
-           if (clipData.contains('/')) {
-               // this is a clip zone, set in / out
-               int in = clipData.section('/', 1, 1).toInt();
-               int out = clipData.section('/', 2, 2).toInt();
-               info.cropStart = GenTime(in, m_document->fps());
-               info.cropDuration = GenTime(out - in, m_document->fps());
-           }
+            if (clipData.contains('/')) {
+                // this is a clip zone, set in / out
+                int in = clipData.section('/', 1, 1).toInt();
+                int out = clipData.section('/', 2, 2).toInt();
+                info.cropStart = GenTime(in, m_document->fps());
+                info.cropDuration = GenTime(out - in, m_document->fps());
+            }
             else {
-               info.cropDuration = clip->duration();
-           }
-           info.endPos = info.startPos + info.cropDuration;
+                info.cropDuration = clip->duration();
+            }
+            info.endPos = info.startPos + info.cropDuration;
             info.track = 0;
             start += info.cropDuration;
             offsetList.append(start);
             ClipItem *item = new ClipItem(clip, info, m_document->fps(), 1.0, 1, getFrameWidth(), false);
-            m_selectionGroup->addItem(item);
+            if (ids.size() > 1) m_selectionGroup->addItem(item);
+            else m_dragItem = item;
+            item->setSelected(true);
             if (!clip->isPlaceHolder()) m_waitingThumbs.append(item);
         }
 
         updateSnapPoints(NULL, offsetList);
-        m_selectionGroup->setPos(framePos);
-        scene()->addItem(m_selectionGroup);
+        QStringList lockedTracks;
+        for (int i = 0; i < m_document->tracksCount(); ++i) {
+            if (m_document->trackInfoAt(i).isLocked) lockedTracks << QString::number(m_document->tracksCount() - i - 1);
+        }
+
+        if (m_selectionGroup) {
+            m_selectionGroup->setProperty("locked_tracks", lockedTracks);
+            m_selectionGroup->setPos(framePos);
+            scene()->addItem(m_selectionGroup);
+        }
+        else if (m_dragItem) {
+            m_dragItem->setProperty("locked_tracks", lockedTracks);
+            m_dragItem->setPos(framePos);
+            scene()->addItem(m_dragItem);
+        }
         //m_selectionGroup->setZValue(10);
         m_thumbsTimer.start();
-        return true;
-
-    } else {
-        // the drag is not a clip (may be effect, ...)
-        m_clipDrag = false;
-        return false;
     }
+    return true;
 }
 
 
@@ -1721,24 +1827,25 @@ bool CustomTrackView::insertDropClips(const QMimeData *data, const QPoint &pos)
 void CustomTrackView::dragEnterEvent(QDragEnterEvent * event)
 {
     if (insertDropClips(event->mimeData(), event->pos())) {
-      if (event->source() == this) {
-             event->setDropAction(Qt::MoveAction);
-             event->accept();
-         } else {
-            event->setDropAction(Qt::MoveAction);
-             event->acceptProposedAction();
-        }
+        if (event->source() == this) {
+            event->setDropAction(Qt::MoveAction);
+            event->accept();
+        } else {
+            event->setDropAction(Qt::MoveAction);
+            event->acceptProposedAction();
+        }
     } else QGraphicsView::dragEnterEvent(event);
 }
 
-bool CustomTrackView::itemCollision(AbstractClipItem *item, ItemInfo newPos)
+bool CustomTrackView::itemCollision(AbstractClipItem *item, const ItemInfo &newPos)
 {
     QRectF shape = QRectF(newPos.startPos.frames(m_document->fps()), newPos.track * m_tracksHeight + 1, (newPos.endPos - newPos.startPos).frames(m_document->fps()) - 0.02, m_tracksHeight - 1);
     QList<QGraphicsItem*> collindingItems = scene()->items(shape, Qt::IntersectsItemShape);
     collindingItems.removeAll(item);
-    if (collindingItems.isEmpty()) return false;
-    else {
-        for (int i = 0; i < collindingItems.count(); i++) {
+    if (collindingItems.isEmpty()) {
+        return false;
+    } else {
+        for (int i = 0; i < collindingItems.count(); ++i) {
             QGraphicsItem *collision = collindingItems.at(i);
             if (collision->type() == item->type()) {
                 // Collision
@@ -1759,7 +1866,7 @@ void CustomTrackView::slotRefreshEffects(ClipItem *clip)
         return;
     }
     bool success = true;
-    for (int i = 0; i < clip->effectsCount(); i++) {
+    for (int i = 0; i < clip->effectsCount(); ++i) {
         if (!m_document->renderer()->mltAddEffect(track, pos, getEffectArgs(clip->effect(i)), false)) success = false;
     }
     if (!success) emit displayMessage(i18n("Problem adding effect to clip"), ErrorMessage);
@@ -1785,7 +1892,7 @@ void CustomTrackView::addEffect(int track, GenTime pos, QDomElement effect)
     if (clip) {
         // Special case: speed effect
         if (effect.attribute("id") == "speed") {
-            if (clip->clipType() != VIDEO && clip->clipType() != AV && clip->clipType() != PLAYLIST) {
+            if (clip->clipType() != Video && clip->clipType() != AV && clip->clipType() != Playlist) {
                 emit displayMessage(i18n("Problem adding effect to clip"), ErrorMessage);
                 return;
             }
@@ -1800,22 +1907,24 @@ void CustomTrackView::addEffect(int track, GenTime pos, QDomElement effect)
             return;
         }
         EffectsParameterList params = clip->addEffect(effect);
-        if (!m_document->renderer()->mltAddEffect(track, pos, params))
+        if (!m_document->renderer()->mltAddEffect(track, pos, params)) {
             emit displayMessage(i18n("Problem adding effect to clip"), ErrorMessage);
-       clip->setSelectedEffect(params.paramValue("kdenlive_ix").toInt());
-        if (clip->isSelected()) emit clipItemSelected(clip);
+            clip->deleteEffect(params.paramValue("kdenlive_ix"));
+        }
+        else clip->setSelectedEffect(params.paramValue("kdenlive_ix").toInt());
+        if (clip->isMainSelectedClip()) emit clipItemSelected(clip);
     } else emit displayMessage(i18n("Cannot find clip to add effect"), ErrorMessage);
 }
 
-void CustomTrackView::deleteEffect(int track, GenTime pos, QDomElement effect)
+void CustomTrackView::deleteEffect(int track, const GenTime &pos, const QDomElement &effect)
 {
     QString index = effect.attribute("kdenlive_ix");
     if (pos < GenTime()) {
         // Delete track effect
         if (m_document->renderer()->mltRemoveTrackEffect(track, index.toInt(), true)) {
-           m_document->removeTrackEffect(track - 1, effect);
-       }
-       else emit displayMessage(i18n("Problem deleting effect"), ErrorMessage);
+            m_document->removeTrackEffect(track - 1, effect);
+        }
+        else emit displayMessage(i18n("Problem deleting effect"), ErrorMessage);
         emit updateTrackEffectState(track - 1);
         emit showTrackEffects(track, m_document->trackInfoAt(track - 1));
         return;
@@ -1839,7 +1948,7 @@ void CustomTrackView::deleteEffect(int track, GenTime pos, QDomElement effect)
     ClipItem *clip = getClipItemAt((int)pos.frames(m_document->fps()), m_document->tracksCount() - track);
     if (clip) {
         clip->deleteEffect(index);
-        emit clipItemSelected(clip);
+        if (clip->isMainSelectedClip()) emit clipItemSelected(clip);
     }
 }
 
@@ -1853,29 +1962,32 @@ void CustomTrackView::slotAddGroupEffect(QDomElement effect, AbstractGroupItem *
     if (!namenode.isNull()) effectName = i18n(namenode.text().toUtf8().data());
     else effectName = i18n("effect");
     effectCommand->setText(i18n("Add %1", effectName));
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == GroupWidget) {
+            itemList << itemList.at(i)->childItems();
+        }
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *item = static_cast <ClipItem *>(itemList.at(i));
             if (effect.tagName() == "effectgroup") {
-               QDomNodeList effectlist = effect.elementsByTagName("effect");
-               for (int j = 0; j < effectlist.count(); j++) {
-                   QDomElement subeffect = effectlist.at(j).toElement();
-                   if (subeffect.hasAttribute("kdenlive_info")) {
-                       // effect is in a group
-                       EffectInfo effectInfo;
-                       effectInfo.fromString(subeffect.attribute("kdenlive_info"));
-                       if (effectInfo.groupIndex < 0) {
-                           // group needs to be appended
-                           effectInfo.groupIndex = item->nextFreeEffectGroupIndex();
-                           subeffect.setAttribute("kdenlive_info", effectInfo.toString());
-                       }
-                   }
-                   processEffect(item, subeffect, offset, effectCommand);
-               }
-           }
+                QDomNodeList effectlist = effect.elementsByTagName("effect");
+                for (int j = 0; j < effectlist.count(); j++) {
+                    QDomElement subeffect = effectlist.at(j).toElement();
+                    if (subeffect.hasAttribute("kdenlive_info")) {
+                        // effect is in a group
+                        EffectInfo effectInfo;
+                        effectInfo.fromString(subeffect.attribute("kdenlive_info"));
+                        if (effectInfo.groupIndex < 0) {
+                            // group needs to be appended
+                            effectInfo.groupIndex = item->nextFreeEffectGroupIndex();
+                            subeffect.setAttribute("kdenlive_info", effectInfo.toString());
+                        }
+                    }
+                    processEffect(item, subeffect, offset, effectCommand);
+                }
+            }
             else {
-               processEffect(item, effect, offset, effectCommand);
-           }
+                processEffect(item, effect, offset, effectCommand);
+            }
         }
     }
     if (effectCommand->childCount() > 0) {
@@ -1883,31 +1995,46 @@ void CustomTrackView::slotAddGroupEffect(QDomElement effect, AbstractGroupItem *
         setDocumentModified();
     } else delete effectCommand;
     if (dropTarget) {
-       clearSelection(false);
-       m_dragItem = dropTarget;
-       m_dragItem->setSelected(true);
-       emit clipItemSelected(static_cast<ClipItem *>(dropTarget));
+        clearSelection(false);
+        m_dragItem = dropTarget;
+        m_dragItem->setSelected(true);
+        emit clipItemSelected(static_cast<ClipItem *>(dropTarget));
     }
 }
 
-void CustomTrackView::slotAddEffect(ClipItem *clip, QDomElement effect)
+void CustomTrackView::slotAddEffect(ClipItem *clip, const QDomElement &effect)
 {
     if (clip) slotAddEffect(effect, clip->startPos(), clip->track());
 }
 
-void CustomTrackView::slotAddEffect(QDomElement effect, GenTime pos, int track)
+void CustomTrackView::slotDropEffect(ClipItem *clip, QDomElement effect, GenTime pos, int track)
+{
+    if (clip == NULL) return;
+    slotAddEffect(effect, pos, track);
+    if (clip->parentItem()) {
+        // Clip is in a group, should not happen
+        kDebug()<<"/// DROPPED ON ITEM IN GRP";
+    }
+    else if (clip != m_dragItem) {
+        clearSelection(false);
+        m_dragItem = clip;
+        clip->setSelected(true);
+        emit clipItemSelected(clip);
+    }
+}
+
+void CustomTrackView::slotAddEffect(QDomElement effect, const GenTime &pos, int track)
 {
     QList<QGraphicsItem *> itemList;
     QUndoCommand *effectCommand = new QUndoCommand();
     QString effectName;
-    
     int offset = effect.attribute("clipstart").toInt();
     if (effect.tagName() == "effectgroup") {
-       effectName = effect.attribute("name");
+        effectName = effect.attribute("name");
     } else {
-       QDomElement namenode = effect.firstChildElement("name");
-       if (!namenode.isNull()) effectName = i18n(namenode.text().toUtf8().data());
-       else effectName = i18n("effect");
+        QDomElement namenode = effect.firstChildElement("name");
+        if (!namenode.isNull()) effectName = i18n(namenode.text().toUtf8().data());
+        else effectName = i18n("effect");
     }
     effectCommand->setText(i18n("Add %1", effectName));
 
@@ -1919,8 +2046,8 @@ void CustomTrackView::slotAddEffect(QDomElement effect, GenTime pos, int track)
     }
 
     //expand groups
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == GROUPWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == GroupWidget) {
             QList<QGraphicsItem *> subitems = itemList.at(i)->childItems();
             for (int j = 0; j < subitems.count(); j++) {
                 if (!itemList.contains(subitems.at(j))) itemList.append(subitems.at(j));
@@ -1928,95 +2055,96 @@ void CustomTrackView::slotAddEffect(QDomElement effect, GenTime pos, int track)
         }
     }
 
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *item = static_cast <ClipItem *>(itemList.at(i));
-           if (effect.tagName() == "effectgroup") {
-               QDomNodeList effectlist = effect.elementsByTagName("effect");
-               for (int j = 0; j < effectlist.count(); j++) {
-                   QDomElement subeffect = effectlist.at(j).toElement();
-                   if (subeffect.hasAttribute("kdenlive_info")) {
-                       // effect is in a group
-                       EffectInfo effectInfo;
-                       effectInfo.fromString(subeffect.attribute("kdenlive_info"));
-                       if (effectInfo.groupIndex < 0) {
-                           // group needs to be appended
-                           effectInfo.groupIndex = item->nextFreeEffectGroupIndex();
-                           subeffect.setAttribute("kdenlive_info", effectInfo.toString());
-                       }
-                   }
-                   processEffect(item, subeffect, offset, effectCommand);
-               }
-           }
+            if (effect.tagName() == "effectgroup") {
+                QDomNodeList effectlist = effect.elementsByTagName("effect");
+                for (int j = 0; j < effectlist.count(); j++) {
+                    QDomElement subeffect = effectlist.at(j).toElement();
+                    if (subeffect.hasAttribute("kdenlive_info")) {
+                        // effect is in a group
+                        EffectInfo effectInfo;
+                        effectInfo.fromString(subeffect.attribute("kdenlive_info"));
+                        if (effectInfo.groupIndex < 0) {
+                            // group needs to be appended
+                            effectInfo.groupIndex = item->nextFreeEffectGroupIndex();
+                            subeffect.setAttribute("kdenlive_info", effectInfo.toString());
+                        }
+                    }
+                    processEffect(item, subeffect, offset, effectCommand);
+                }
+            }
             else processEffect(item, effect, offset, effectCommand);
         }
     }
     if (effectCommand->childCount() > 0) {
         m_commandStack->push(effectCommand);
         setDocumentModified();
-       if (effectCommand->childCount() == 1) {
-           // Display newly added clip effect
-           for (int i = 0; i < itemList.count(); i++) {
-               if (itemList.at(i)->type() == AVWIDGET) {
-                   ClipItem *clip = static_cast<ClipItem *>(itemList.at(i));
-                   clip->setSelectedEffect(clip->effectsCount());
-                   if (!clip->isSelected()) {
-                       clearSelection(false);
-                       clip->setSelected(true);
-                       m_dragItem = clip;
-                   }
-                   emit clipItemSelected(clip);
-                   break;
-               }
-           }
-       }
-       else {
-           for (int i = 0; i < itemList.count(); i++) {
-               if (itemList.at(i)->type() == AVWIDGET) {
-                   ClipItem *clip = static_cast<ClipItem *>(itemList.at(i));
-                   if (clip->isMainSelectedClip()) {
-                       emit clipItemSelected(clip);
-                       break;
-                   }
-               }
-           }
-       }
+        /*if (effectCommand->childCount() == 1) {
+        // Display newly added clip effect
+        for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
+            ClipItem *clip = static_cast<ClipItem *>(itemList.at(i));
+            clip->setSelectedEffect(clip->effectsCount());
+            if (!clip->isSelected() && (!m_dragItem || !itemList.contains(m_dragItem))) {
+            kDebug()<<"// CLIP WAS NO SELECTED, DRG: "<<(m_dragItem == NULL);
+            clearSelection(false);
+            clip->setSelected(true);
+            m_dragItem = clip;
+            emit clipItemSelected(clip);
+            break;
+            }
+        }
+        }
+    }
+    else {
+        for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
+            ClipItem *clip = static_cast<ClipItem *>(itemList.at(i));
+            if (clip->isMainSelectedClip()) {
+            emit clipItemSelected(clip);
+            break;
+            }
+        }
+        }
+    }*/
     } else delete effectCommand;
 }
 
 void CustomTrackView::processEffect(ClipItem *item, QDomElement effect, int offset, QUndoCommand *effectCommand)
 {
     if (effect.attribute("type") == "audio") {
-       // Don't add audio effects on video clips
-        if (item->isVideoOnly() || (item->clipType() != AUDIO && item->clipType() != AV && item->clipType() != PLAYLIST)) {
-           /* do not show error message when item is part of a group as the user probably knows what he does then
+        // Don't add audio effects on video clips
+        if (item->isVideoOnly() || (item->clipType() != Audio && item->clipType() != AV && item->clipType() != Playlist)) {
+            /* do not show error message when item is part of a group as the user probably knows what he does then
             * and the message is annoying when working with the split audio feature */
             if (!item->parentItem() || item->parentItem() == m_selectionGroup)
-               emit displayMessage(i18n("Cannot add an audio effect to this clip"), ErrorMessage);
+                emit displayMessage(i18n("Cannot add an audio effect to this clip"), ErrorMessage);
             return;
         }
     } else if (effect.attribute("type") == "video" || !effect.hasAttribute("type")) {
-       // Don't add video effect on audio clips
-        if (item->isAudioOnly() || item->clipType() == AUDIO) {
-           /* do not show error message when item is part of a group as the user probably knows what he does then
+        // Don't add video effect on audio clips
+        if (item->isAudioOnly() || item->clipType() == Audio) {
+            /* do not show error message when item is part of a group as the user probably knows what he does then
             * and the message is annoying when working with the split audio feature */
             if (!item->parentItem() || item->parentItem() == m_selectionGroup)
-               emit displayMessage(i18n("Cannot add a video effect to this clip"), ErrorMessage);
+                emit displayMessage(i18n("Cannot add a video effect to this clip"), ErrorMessage);
             return;
         }
     }
     if (effect.attribute("unique", "0") != "0" && item->hasEffect(effect.attribute("tag"), effect.attribute("id")) != -1) {
-       emit displayMessage(i18n("Effect already present in clip"), ErrorMessage);
+        emit displayMessage(i18n("Effect already present in clip"), ErrorMessage);
         return;
     }
     if (item->isItemLocked()) {
-       return;
+        return;
     }
 
     if (effect.attribute("id") == "freeze" && m_cursorPos > item->startPos().frames(m_document->fps()) && m_cursorPos < item->endPos().frames(m_document->fps())) {
-       item->initEffect(effect, m_cursorPos - item->startPos().frames(m_document->fps()), offset);
+        item->initEffect(effect, m_cursorPos - item->startPos().frames(m_document->fps()), offset);
     } else {
-       item->initEffect(effect, 0, offset);
+        item->initEffect(effect, 0, offset);
     }
     new AddEffectCommand(this, m_document->tracksCount() - item->track(), item->startPos(), effect, true, effectCommand);
 }
@@ -2041,8 +2169,8 @@ void CustomTrackView::slotDeleteEffect(ClipItem *clip, int track, QDomElement ef
         delCommand->setText(i18n("Delete %1", effectName));
 
         //expand groups
-        for (int i = 0; i < items.count(); i++) {
-            if (items.at(i)->type() == GROUPWIDGET) {
+        for (int i = 0; i < items.count(); ++i) {
+            if (items.at(i)->type() == GroupWidget) {
                 QList<QGraphicsItem *> subitems = items.at(i)->childItems();
                 for (int j = 0; j < subitems.count(); j++) {
                     if (!items.contains(subitems.at(j))) items.append(subitems.at(j));
@@ -2050,8 +2178,8 @@ void CustomTrackView::slotDeleteEffect(ClipItem *clip, int track, QDomElement ef
             }
         }
 
-        for (int i = 0; i < items.count(); i++) {
-            if (items.at(i)->type() == AVWIDGET) {
+        for (int i = 0; i < items.count(); ++i) {
+            if (items.at(i)->type() == AVWidget) {
                 ClipItem *item = static_cast <ClipItem *>(items.at(i));
                 int ix = item->hasEffect(effect.attribute("tag"), effect.attribute("id"));
                 if (ix != -1) {
@@ -2075,7 +2203,7 @@ void CustomTrackView::slotDeleteEffect(ClipItem *clip, int track, QDomElement ef
 void CustomTrackView::updateEffect(int track, GenTime pos, QDomElement insertedEffect, bool updateEffectStack)
 {
     if (insertedEffect.isNull()) {
-       kDebug()<<"// Trying to add null effect";
+        kDebug()<<"// Trying to add null effect";
         emit displayMessage(i18n("Problem editing effect"), ErrorMessage);
         return;
     }
@@ -2092,7 +2220,7 @@ void CustomTrackView::updateEffect(int track, GenTime pos, QDomElement insertedE
         }*/
         if (!m_document->renderer()->mltEditEffect(m_document->tracksCount() - track, pos, effectParams)) {
             emit displayMessage(i18n("Problem editing effect"), ErrorMessage);
-       }
+        }
         m_document->setTrackEffect(m_document->tracksCount() - track - 1, ix, effect);
         emit updateTrackEffectState(track);
         setDocumentModified();
@@ -2113,12 +2241,12 @@ void CustomTrackView::updateEffect(int track, GenTime pos, QDomElement insertedE
                 doChangeClipSpeed(clip->info(), clip->speedIndependantInfo(), speed, clip->speed(), strobe, clip->baseClip()->getId());
             }
             clip->updateEffect(effect);
-           if (updateEffectStack && clip->isSelected())
-               emit clipItemSelected(clip);
-           if (ix == clip->selectedEffectIndex()) {
-               // make sure to update display of clip keyframes
-               clip->setSelectedEffect(ix);
-           }
+            if (updateEffectStack && clip->isSelected())
+                emit clipItemSelected(clip);
+            if (ix == clip->selectedEffectIndex()) {
+                // make sure to update display of clip keyframes
+                clip->setSelectedEffect(ix);
+            }
             return;
         }
 
@@ -2140,19 +2268,19 @@ void CustomTrackView::updateEffect(int track, GenTime pos, QDomElement insertedE
                 clip->setFadeOut(pos);
             }
         }
-       bool success = m_document->renderer()->mltEditEffect(m_document->tracksCount() - clip->track(), clip->startPos(), effectParams);
+        bool success = m_document->renderer()->mltEditEffect(m_document->tracksCount() - clip->track(), clip->startPos(), effectParams);
 
         if (success) {
-           clip->updateEffect(effect);
-           if (updateEffectStack && clip->isSelected()) {
-               emit clipItemSelected(clip);
-           }
-           if (ix == clip->selectedEffectIndex()) {
-               // make sure to update display of clip keyframes
-               clip->setSelectedEffect(ix);
-           }
-       }
-       else emit displayMessage(i18n("Problem editing effect"), ErrorMessage);
+            clip->updateEffect(effect);
+            if (updateEffectStack && clip->isSelected()) {
+                emit clipItemSelected(clip);
+            }
+            if (ix == clip->selectedEffectIndex()) {
+                // make sure to update display of clip keyframes
+                clip->setSelectedEffect(ix);
+            }
+        }
+        else emit displayMessage(i18n("Problem editing effect"), ErrorMessage);
     }
     else emit displayMessage(i18n("Cannot find clip to update effect"), ErrorMessage);
     setDocumentModified();
@@ -2164,8 +2292,8 @@ void CustomTrackView::updateEffectState(int track, GenTime pos, QList <int> effe
         // editing a track effect
         if (!m_document->renderer()->mltEnableEffects(m_document->tracksCount() - track, pos, effectIndexes, disable)) {
             emit displayMessage(i18n("Problem editing effect"), ErrorMessage);
-           return;
-       }
+            return;
+        }
         m_document->enableTrackEffects(m_document->tracksCount() - track - 1, effectIndexes, disable);
         emit updateTrackEffectState(track);
         setDocumentModified();
@@ -2174,78 +2302,78 @@ void CustomTrackView::updateEffectState(int track, GenTime pos, QList <int> effe
     // editing a clip effect
     ClipItem *clip = getClipItemAt((int)pos.frames(m_document->fps()), m_document->tracksCount() - track);
     if (clip) {
-       bool success = m_document->renderer()->mltEnableEffects(m_document->tracksCount() - clip->track(), clip->startPos(), effectIndexes, disable);
-       if (success) {
-           clip->enableEffects(effectIndexes, disable);
-           if (updateEffectStack && clip->isSelected()) {
-               emit clipItemSelected(clip);
-           }
-           if (effectIndexes.contains(clip->selectedEffectIndex())) {
-               // make sure to update display of clip keyframes
-               clip->setSelectedEffect(clip->selectedEffectIndex());
-           }
-       }
-       else emit displayMessage(i18n("Problem editing effect"), ErrorMessage);
+        bool success = m_document->renderer()->mltEnableEffects(m_document->tracksCount() - clip->track(), clip->startPos(), effectIndexes, disable);
+        if (success) {
+            clip->enableEffects(effectIndexes, disable);
+            if (updateEffectStack && clip->isSelected()) {
+                emit clipItemSelected(clip);
+            }
+            if (effectIndexes.contains(clip->selectedEffectIndex())) {
+                // make sure to update display of clip keyframes
+                clip->setSelectedEffect(clip->selectedEffectIndex());
+            }
+        }
+        else emit displayMessage(i18n("Problem editing effect"), ErrorMessage);
     }
     else emit displayMessage(i18n("Cannot find clip to update effect"), ErrorMessage);
 }
 
-void CustomTrackView::moveEffect(int track, GenTime pos, QList <int> oldPos, QList <int> newPos)
+void CustomTrackView::moveEffect(int track, const GenTime &pos, const QList <int> &oldPos, const QList <int> &newPos)
 {
     if (pos < GenTime()) {
         // Moving track effect
         int documentTrack = m_document->tracksCount() - track - 1;
         int max = m_document->getTrackEffects(documentTrack).count();
-       int new_position = newPos.at(0);
-       if (new_position > max) {
-           new_position = max;
-       }
-       int old_position = oldPos.at(0);
-       for (int i = 0; i < newPos.count(); i++) {
-           QDomElement act = m_document->getTrackEffect(documentTrack, new_position);
-           if (old_position > new_position) {
-               // Moving up, we need to adjust index
-               old_position = oldPos.at(i);
-               new_position = newPos.at(i);
-           }
-           QDomElement before = m_document->getTrackEffect(documentTrack, old_position);
-           if (!act.isNull() && !before.isNull()) {
-               m_document->setTrackEffect(documentTrack, new_position, before);
-               m_document->renderer()->mltMoveEffect(m_document->tracksCount() - track, pos, old_position, new_position);
-           } else emit displayMessage(i18n("Cannot move effect"), ErrorMessage);
-       }
-       emit showTrackEffects(m_document->tracksCount() - track, m_document->trackInfoAt(documentTrack));
+        int new_position = newPos.at(0);
+        if (new_position > max) {
+            new_position = max;
+        }
+        int old_position = oldPos.at(0);
+        for (int i = 0; i < newPos.count(); ++i) {
+            QDomElement act = m_document->getTrackEffect(documentTrack, new_position);
+            if (old_position > new_position) {
+                // Moving up, we need to adjust index
+                old_position = oldPos.at(i);
+                new_position = newPos.at(i);
+            }
+            QDomElement before = m_document->getTrackEffect(documentTrack, old_position);
+            if (!act.isNull() && !before.isNull()) {
+                m_document->setTrackEffect(documentTrack, new_position, before);
+                m_document->renderer()->mltMoveEffect(m_document->tracksCount() - track, pos, old_position, new_position);
+            } else emit displayMessage(i18n("Cannot move effect"), ErrorMessage);
+        }
+        emit showTrackEffects(m_document->tracksCount() - track, m_document->trackInfoAt(documentTrack));
         return;
     }
     ClipItem *clip = getClipItemAt((int)pos.frames(m_document->fps()), m_document->tracksCount() - track);
     if (clip) {
-       int new_position = newPos.at(0);
-       if (new_position > clip->effectsCount()) {
-           new_position = clip->effectsCount();
-       }
-       int old_position = oldPos.at(0);
-       for (int i = 0; i < newPos.count(); i++) {
-           QDomElement act = clip->effectAtIndex(new_position);
-           if (old_position > new_position) {
-               // Moving up, we need to adjust index
-               old_position = oldPos.at(i);
-               new_position = newPos.at(i);
-           }
-           QDomElement before = clip->effectAtIndex(old_position);
-           if (act.isNull() || before.isNull()) {
-               emit displayMessage(i18n("Cannot move effect"), ErrorMessage);
-               return;
-           }
-           clip->moveEffect(before, new_position);
-           // special case: speed effect, which is a pseudo-effect, not appearing in MLT's effects
-           if (act.attribute("id") == "speed") {
-               m_document->renderer()->mltUpdateEffectPosition(track, pos, old_position, new_position);
-           } else if (before.attribute("id") == "speed") {
-               m_document->renderer()->mltUpdateEffectPosition(track, pos, new_position, old_position);
-           } else m_document->renderer()->mltMoveEffect(track, pos, old_position, new_position);
-       }
-       clip->setSelectedEffect(newPos.at(0));
-       emit clipItemSelected(clip);
+        int new_position = newPos.at(0);
+        if (new_position > clip->effectsCount()) {
+            new_position = clip->effectsCount();
+        }
+        int old_position = oldPos.at(0);
+        for (int i = 0; i < newPos.count(); ++i) {
+            QDomElement act = clip->effectAtIndex(new_position);
+            if (old_position > new_position) {
+                // Moving up, we need to adjust index
+                old_position = oldPos.at(i);
+                new_position = newPos.at(i);
+            }
+            QDomElement before = clip->effectAtIndex(old_position);
+            if (act.isNull() || before.isNull()) {
+                emit displayMessage(i18n("Cannot move effect"), ErrorMessage);
+                return;
+            }
+            clip->moveEffect(before, new_position);
+            // special case: speed effect, which is a pseudo-effect, not appearing in MLT's effects
+            if (act.attribute("id") == "speed") {
+                m_document->renderer()->mltUpdateEffectPosition(track, pos, old_position, new_position);
+            } else if (before.attribute("id") == "speed") {
+                m_document->renderer()->mltUpdateEffectPosition(track, pos, new_position, old_position);
+            } else m_document->renderer()->mltMoveEffect(track, pos, old_position, new_position);
+        }
+        clip->setSelectedEffect(newPos.at(0));
+        emit clipItemSelected(clip);
         setDocumentModified();
     } else emit displayMessage(i18n("Cannot move effect"), ErrorMessage);
 }
@@ -2257,22 +2385,22 @@ void CustomTrackView::slotChangeEffectState(ClipItem *clip, int track, QList <in
         // editing track effect
         command = new ChangeEffectStateCommand(this, m_document->tracksCount() - track, GenTime(-1), effectIndexes, disable, false, true);
     } else {
-       // Check if we have a speed effect, disabling / enabling it needs a special procedure since it is a pseudoo effect
-       QList <int> speedEffectIndexes;
-       for (int i = 0; i < effectIndexes.count(); i++) {
-           QDomElement effect = clip->effectAtIndex(effectIndexes.at(i));
-           if (effect.attribute("id") == "speed") {
-               // speed effect
-               speedEffectIndexes << effectIndexes.at(i);
-               QDomElement newEffect = effect.cloneNode().toElement();
-               newEffect.setAttribute("disable", (int) disable);
-               EditEffectCommand *editcommand = new EditEffectCommand(this, m_document->tracksCount() - clip->track(), clip->startPos(), effect, newEffect, effectIndexes.at(i), false, true);
-               m_commandStack->push(editcommand);
-           }
-       }
-       for (int j = 0; j < speedEffectIndexes.count(); j++) {
-           effectIndexes.removeAll(speedEffectIndexes.at(j));
-       }
+        // Check if we have a speed effect, disabling / enabling it needs a special procedure since it is a pseudoo effect
+        QList <int> speedEffectIndexes;
+        for (int i = 0; i < effectIndexes.count(); ++i) {
+            QDomElement effect = clip->effectAtIndex(effectIndexes.at(i));
+            if (effect.attribute("id") == "speed") {
+                // speed effect
+                speedEffectIndexes << effectIndexes.at(i);
+                QDomElement newEffect = effect.cloneNode().toElement();
+                newEffect.setAttribute("disable", (int) disable);
+                EditEffectCommand *editcommand = new EditEffectCommand(this, m_document->tracksCount() - clip->track(), clip->startPos(), effect, newEffect, effectIndexes.at(i), false, true);
+                m_commandStack->push(editcommand);
+            }
+        }
+        for (int j = 0; j < speedEffectIndexes.count(); j++) {
+            effectIndexes.removeAll(speedEffectIndexes.at(j));
+        }
         command = new ChangeEffectStateCommand(this, m_document->tracksCount() - clip->track(), clip->startPos(), effectIndexes, disable, false, true);
     }
     m_commandStack->push(command);
@@ -2307,7 +2435,7 @@ void CustomTrackView::slotUpdateClipRegion(ClipItem *clip, int ix, QString regio
     m_commandStack->push(command);
 }
 
-ClipItem *CustomTrackView::cutClip(ItemInfo info, GenTime cutTime, bool cut, bool execute)
+ClipItem *CustomTrackView::cutClip(const ItemInfo &info, const GenTime &cutTime, bool cut, const EffectsList &oldStack, bool execute)
 {
     if (cut) {
         // cut clip
@@ -2323,12 +2451,12 @@ ClipItem *CustomTrackView::cutClip(ItemInfo info, GenTime cutTime, bool cut, boo
         }
 
         if (execute) {
-           if (!m_document->renderer()->mltCutClip(m_document->tracksCount() - info.track, cutTime)) {
-               // Error cuting clip in playlist
-               m_blockRefresh = false;
-               return NULL;
-           }
-       }
+            if (!m_document->renderer()->mltCutClip(m_document->tracksCount() - info.track, cutTime)) {
+                // Error cuting clip in playlist
+                m_blockRefresh = false;
+                return NULL;
+            }
+        }
         int cutPos = (int) cutTime.frames(m_document->fps());
         ItemInfo newPos;
         newPos.startPos = cutTime;
@@ -2368,17 +2496,23 @@ ClipItem *CustomTrackView::cutClip(ItemInfo info, GenTime cutTime, bool cut, boo
 
         item->resizeEnd(cutPos);
         scene()->addItem(dup);
-        if (item->checkKeyFrames())
+
+        if (item->checkKeyFrames(m_document->width(), m_document->height(), info.cropDuration.frames(m_document->fps())))
             slotRefreshEffects(item);
-        if (dup->checkKeyFrames())
+
+        if (dup->checkKeyFrames(m_document->width(), m_document->height(), info.cropDuration.frames(m_document->fps()), cutTime.frames(m_document->fps())))
             slotRefreshEffects(dup);
 
         item->baseClip()->addReference();
         m_document->updateClip(item->baseClip()->getId());
         setDocumentModified();
         KdenliveSettings::setSnaptopoints(snap);
-        if (execute && item->isSelected())
-            emit clipItemSelected(item);
+        if (execute && item->isSelected()) {
+            m_scene->clearSelection();
+            dup->setSelected(true);
+            m_dragItem = dup;
+            emit clipItemSelected(dup);
+        }
         return dup;
     } else {
         // uncut clip
@@ -2399,18 +2533,6 @@ ClipItem *CustomTrackView::cutClip(ItemInfo info, GenTime cutTime, bool cut, boo
         bool snap = KdenliveSettings::snaptopoints();
         KdenliveSettings::setSnaptopoints(false);
 
-        // join fade effects again
-        int ix = dup->hasEffect(QString(), "fadeout");
-        if (ix != -1) {
-            QDomElement effect = dup->effectAtIndex(ix);
-            item->addEffect(effect);
-        }
-        ix = dup->hasEffect(QString(), "fade_to_black");
-        if (ix != -1) {
-            QDomElement effect = dup->effectAtIndex(ix);
-            item->addEffect(effect);
-        }
-
         m_waitingThumbs.removeAll(dup);
         bool selected = item->isSelected();
         if (dup->isSelected()) {
@@ -2426,9 +2548,10 @@ ClipItem *CustomTrackView::cutClip(ItemInfo info, GenTime cutTime, bool cut, boo
 
         ItemInfo clipinfo = item->info();
         clipinfo.track = m_document->tracksCount() - clipinfo.track;
-        bool success = m_document->renderer()->mltResizeClipEnd(clipinfo, info.endPos - info.startPos);
+        bool success = m_document->renderer()->mltResizeClipEnd(clipinfo, info.endPos - info.startPos, false);
         if (success) {
             item->resizeEnd((int) info.endPos.frames(m_document->fps()));
+            item->setEffectList(oldStack);
             setDocumentModified();
         } else {
             emit displayMessage(i18n("Error when resizing clip"), ErrorMessage);
@@ -2450,7 +2573,7 @@ void CustomTrackView::slotAddTransitionToSelectedClips(QDomElement transition)
 {
     QList<QGraphicsItem *> itemList = scene()->selectedItems();
     if (itemList.count() == 1) {
-        if (itemList.at(0)->type() == AVWIDGET) {
+        if (itemList.at(0)->type() == AVWidget) {
             ClipItem *item = (ClipItem *) itemList.at(0);
             ItemInfo info;
             info.track = item->track();
@@ -2469,8 +2592,8 @@ void CustomTrackView::slotAddTransitionToSelectedClips(QDomElement transition)
                 QRectF r(info.startPos.frames(m_document->fps()), startY, (info.endPos - info.startPos).frames(m_document->fps()), m_tracksHeight / 2);
                 QList<QGraphicsItem *> selection = m_scene->items(r);
                 bool transitionAccepted = true;
-                for (int i = 0; i < selection.count(); i++) {
-                    if (selection.at(i)->type() == TRANSITIONWIDGET) {
+                for (int i = 0; i < selection.count(); ++i) {
+                    if (selection.at(i)->type() == TransitionWidget) {
                         Transition *tr = static_cast <Transition *>(selection.at(i));
                         if (tr->startPos() - info.startPos > GenTime(5, m_document->fps())) {
                             if (tr->startPos() < info.endPos) info.endPos = tr->startPos();
@@ -2495,8 +2618,8 @@ void CustomTrackView::slotAddTransitionToSelectedClips(QDomElement transition)
                 QRectF r(info.startPos.frames(m_document->fps()), startY, (info.endPos - info.startPos).frames(m_document->fps()), m_tracksHeight / 2);
                 QList<QGraphicsItem *> selection = m_scene->items(r);
                 bool transitionAccepted = true;
-                for (int i = 0; i < selection.count(); i++) {
-                    if (selection.at(i)->type() == TRANSITIONWIDGET) {
+                for (int i = 0; i < selection.count(); ++i) {
+                    if (selection.at(i)->type() == TransitionWidget) {
                         Transition *tr = static_cast <Transition *>(selection.at(i));
                         if (info.endPos - tr->endPos() > GenTime(5, m_document->fps())) {
                             if (tr->endPos() > info.startPos) info.startPos = tr->endPos();
@@ -2507,32 +2630,32 @@ void CustomTrackView::slotAddTransitionToSelectedClips(QDomElement transition)
                 else emit displayMessage(i18n("Cannot add transition"), ErrorMessage);
             }
         }
-    } else for (int i = 0; i < itemList.count(); i++) {
-            if (itemList.at(i)->type() == AVWIDGET) {
-                ClipItem *item = (ClipItem *) itemList.at(i);
-                ItemInfo info;
-                info.startPos = item->startPos();
-                info.endPos = info.startPos + GenTime(65, m_document->fps());
-                info.track = item->track();
+    } else for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
+            ClipItem *item = (ClipItem *) itemList.at(i);
+            ItemInfo info;
+            info.startPos = item->startPos();
+            info.endPos = info.startPos + GenTime(65, m_document->fps());
+            info.track = item->track();
 
-                // Check there is no other transition at that place
-                double startY = info.track * m_tracksHeight + 1 + m_tracksHeight / 2;
-                QRectF r(info.startPos.frames(m_document->fps()), startY, (info.endPos - info.startPos).frames(m_document->fps()), m_tracksHeight / 2);
-                QList<QGraphicsItem *> selection = m_scene->items(r);
-                bool transitionAccepted = true;
-                for (int i = 0; i < selection.count(); i++) {
-                    if (selection.at(i)->type() == TRANSITIONWIDGET) {
-                        Transition *tr = static_cast <Transition *>(selection.at(i));
-                        if (tr->startPos() - info.startPos > GenTime(5, m_document->fps())) {
-                            if (tr->startPos() < info.endPos) info.endPos = tr->startPos();
-                        } else transitionAccepted = false;
-                    }
+            // Check there is no other transition at that place
+            double startY = info.track * m_tracksHeight + 1 + m_tracksHeight / 2;
+            QRectF r(info.startPos.frames(m_document->fps()), startY, (info.endPos - info.startPos).frames(m_document->fps()), m_tracksHeight / 2);
+            QList<QGraphicsItem *> selection = m_scene->items(r);
+            bool transitionAccepted = true;
+            for (int i = 0; i < selection.count(); ++i) {
+                if (selection.at(i)->type() == TransitionWidget) {
+                    Transition *tr = static_cast <Transition *>(selection.at(i));
+                    if (tr->startPos() - info.startPos > GenTime(5, m_document->fps())) {
+                        if (tr->startPos() < info.endPos) info.endPos = tr->startPos();
+                    } else transitionAccepted = false;
                 }
-                int transitiontrack = getPreviousVideoTrack(info.track);
-                if (transitionAccepted) slotAddTransition(item, info, transitiontrack, transition);
-                else emit displayMessage(i18n("Cannot add transition"), ErrorMessage);
             }
+            int transitiontrack = getPreviousVideoTrack(info.track);
+            if (transitionAccepted) slotAddTransition(item, info, transitiontrack, transition);
+            else emit displayMessage(i18n("Cannot add transition"), ErrorMessage);
         }
+    }
 }
 
 void CustomTrackView::slotAddTransition(ClipItem* /*clip*/, ItemInfo transitionInfo, int endTrack, QDomElement transition)
@@ -2546,7 +2669,7 @@ void CustomTrackView::slotAddTransition(ClipItem* /*clip*/, ItemInfo transitionI
     setDocumentModified();
 }
 
-void CustomTrackView::addTransition(ItemInfo transitionInfo, int endTrack, QDomElement params, bool refresh)
+void CustomTrackView::addTransition(const ItemInfo &transitionInfo, int endTrack, const QDomElement &params, bool refresh)
 {
     Transition *tr = new Transition(transitionInfo, endTrack, m_document->fps(), params, true);
     //kDebug() << "---- ADDING transition " << params.attribute("value");
@@ -2559,7 +2682,7 @@ void CustomTrackView::addTransition(ItemInfo transitionInfo, int endTrack, QDomE
     }
 }
 
-void CustomTrackView::deleteTransition(ItemInfo transitionInfo, int endTrack, QDomElement /*params*/, bool refresh)
+void CustomTrackView::deleteTransition(const ItemInfo &transitionInfo, int endTrack, QDomElement /*params*/, bool refresh)
 {
     Transition *item = getTransitionItemAt(transitionInfo.startPos, transitionInfo.track);
     if (!item) {
@@ -2593,7 +2716,7 @@ void CustomTrackView::slotTransitionUpdated(Transition *tr, QDomElement old)
     setDocumentModified();
 }
 
-void CustomTrackView::updateTransition(int track, GenTime pos, QDomElement oldTransition, QDomElement transition, bool updateTransitionWidget)
+void CustomTrackView::updateTransition(int track, const GenTime &pos, const QDomElement &oldTransition, const QDomElement &transition, bool updateTransitionWidget)
 {
     Transition *item = getTransitionItemAt(pos, track);
     if (!item) {
@@ -2631,6 +2754,10 @@ void CustomTrackView::dragMoveEvent(QDragMoveEvent * event)
             m_selectionGroup->setPos(pos);
             emit mousePosition((int)(m_selectionGroup->scenePos().x() + 0.5));
             event->acceptProposedAction();
+        } else if (m_dragItem) {
+            m_dragItem->setPos(pos);
+            emit mousePosition((int)(m_dragItem->scenePos().x() + 0.5));
+            event->acceptProposedAction();
         } else {
             // Drag enter was not possible, try again at mouse position
             insertDropClips(event->mimeData(), event->pos());
@@ -2643,31 +2770,41 @@ void CustomTrackView::dragMoveEvent(QDragMoveEvent * event)
 
 void CustomTrackView::dragLeaveEvent(QDragLeaveEvent * event)
 {
-    if (m_selectionGroup && m_clipDrag) {
+    if ((m_selectionGroup || m_dragItem) && m_clipDrag) {
         m_thumbsTimer.stop();
         m_waitingThumbs.clear();
-        QList<QGraphicsItem *> items = m_selectionGroup->childItems();
+        QList<QGraphicsItem *> items;
+        QMutexLocker lock(&m_selectionMutex);
+        if (m_selectionGroup) items = m_selectionGroup->childItems();
+        else if (m_dragItem) items.append(m_dragItem);
         qDeleteAll(items);
-        scene()->destroyItemGroup(m_selectionGroup);
+        if (m_selectionGroup) scene()->destroyItemGroup(m_selectionGroup);
         m_selectionGroup = NULL;
+        m_dragItem = NULL;
+        event->accept();
     } else QGraphicsView::dragLeaveEvent(event);
 }
 
 void CustomTrackView::dropEvent(QDropEvent * event)
 {
-    if (m_selectionGroup && m_clipDrag) {
-        QList<QGraphicsItem *> items = m_selectionGroup->childItems();
+    if ((m_selectionGroup || m_dragItem) && m_clipDrag) {
+        QList<QGraphicsItem *> items;
+        if (m_selectionGroup) items = m_selectionGroup->childItems();
+        else if (m_dragItem) items.append(m_dragItem);
         resetSelectionGroup();
-       m_dragItem = NULL;
+        m_dragItem = NULL;
         m_scene->clearSelection();
         bool hasVideoClip = false;
         QUndoCommand *addCommand = new QUndoCommand();
         addCommand->setText(i18n("Add timeline clip"));
         QList <ClipItem *> brokenClips;
 
-        for (int i = 0; i < items.count(); i++) {
+        // Add refresh command for undo
+        new RefreshMonitorCommand(this, false, true, addCommand);
+
+        for (int i = 0; i < items.count(); ++i) {
             ClipItem *item = static_cast <ClipItem *>(items.at(i));
-            if (!hasVideoClip && (item->clipType() == AV || item->clipType() == VIDEO)) hasVideoClip = true;
+            if (!hasVideoClip && (item->clipType() == AV || item->clipType() == Video)) hasVideoClip = true;
             if (items.count() == 1) {
                 updateClipTypeActions(item);
             } else {
@@ -2698,12 +2835,17 @@ void CustomTrackView::dropEvent(QDropEvent * event)
             updateTrackDuration(info.track, addCommand);
 
             if (item->baseClip()->isTransparent() && getTransitionItemAtStart(info.startPos, info.track) == NULL) {
-                // add transparency transition
-                QDomElement trans = MainWindow::transitions.getEffectByTag("composite", "composite").cloneNode().toElement();
-                new AddTransitionCommand(this, info, getPreviousVideoTrack(info.track), trans, false, true, addCommand);
+                // add transparency transition if space is available
+                if (canBePastedTo(info, TransitionWidget)) {
+                    QDomElement trans = MainWindow::transitions.getEffectByTag("affine", QString()).cloneNode().toElement();
+                    new AddTransitionCommand(this, info, getPreviousVideoTrack(info.track), trans, false, true, addCommand);
+                }
             }
             item->setSelected(true);
         }
+        // Add refresh command for redo
+        new RefreshMonitorCommand(this, false, false, addCommand);
+
         qDeleteAll(brokenClips);
         brokenClips.clear();
         if (addCommand->childCount() > 0) m_commandStack->push(addCommand);
@@ -2721,8 +2863,8 @@ void CustomTrackView::dropEvent(QDropEvent * event)
         QStringList timelineList;
 
         kDebug()<<"// ITEMS on TRACK: "<<selection.count();
-        for (int i = 0; i < selection.count(); i++) {
-               if (selection.at(i)->type() == AVWIDGET) {
+        for (int i = 0; i < selection.count(); ++i) {
+               if (selection.at(i)->type() == AVWidget) {
                    ClipItem *clip = static_cast <ClipItem *>(selection.at(i));
                    int start = clip->startPos().frames(m_document->fps());
                    int end = clip->endPos().frames(m_document->fps());
@@ -2734,7 +2876,7 @@ void CustomTrackView::dropEvent(QDropEvent * event)
 
         m_pasteEffectsAction->setEnabled(m_copiedItems.count() == 1);
         if (items.count() > 1) {
-            groupSelectedItems(items, true);
+            groupSelectedItems(items);
         } else if (items.count() == 1) {
             m_dragItem = static_cast <AbstractClipItem *>(items.at(0));
             emit clipItemSelected((ClipItem*) m_dragItem, false);
@@ -2744,7 +2886,7 @@ void CustomTrackView::dropEvent(QDropEvent * event)
         event->accept();
 
         /// \todo enable when really working
-//        alignAudio();
+        //        alignAudio();
 
     } else QGraphicsView::dropEvent(event);
     setFocus();
@@ -2762,9 +2904,9 @@ void CustomTrackView::adjustTimelineClips(EDITMODE mode, ClipItem *item, ItemInf
         QRectF rect(info.startPos.frames(m_document->fps()), info.track * m_tracksHeight + m_tracksHeight / 2, (info.endPos - info.startPos).frames(m_document->fps()) - 1, 5);
         QList<QGraphicsItem *> selection = m_scene->items(rect);
         if (item) selection.removeAll(item);
-        for (int i = 0; i < selection.count(); i++) {
+        for (int i = 0; i < selection.count(); ++i) {
             if (!selection.at(i)->isEnabled()) continue;
-            if (selection.at(i)->type() == AVWIDGET) {
+            if (selection.at(i)->type() == AVWidget) {
                 ClipItem *clip = static_cast<ClipItem *>(selection.at(i));
                 if (clip->startPos() < info.startPos) {
                     if (clip->endPos() > info.endPos) {
@@ -2779,12 +2921,12 @@ void CustomTrackView::adjustTimelineClips(EDITMODE mode, ClipItem *item, ItemInf
                         newdupInfo.startPos = info.endPos;
                         newdupInfo.cropStart += diff2;
                         newdupInfo.cropDuration = clipInfo.endPos - info.endPos;
-                        new RazorClipCommand(this, clipInfo, info.startPos, false, command);
+                        new RazorClipCommand(this, clipInfo, clip->effectList(), info.startPos, false, command);
                         new ResizeClipCommand(this, dupInfo, newdupInfo, false, false, command);
-                        ClipItem *dup = cutClip(clipInfo, info.startPos, true, false);
+                        ClipItem *dup = cutClip(clipInfo, info.startPos, true, EffectsList(), false);
                         if (dup) {
-                           dup->resizeStart(info.endPos.frames(m_document->fps()));
-                       }
+                            dup->resizeStart(info.endPos.frames(m_document->fps()));
+                        }
                     } else {
                         ItemInfo newclipInfo = clip->info();
                         newclipInfo.endPos = info.startPos;
@@ -2813,8 +2955,8 @@ void CustomTrackView::adjustTimelineClips(EDITMODE mode, ClipItem *item, ItemInf
         QRectF rect(info.startPos.frames(m_document->fps()), info.track * m_tracksHeight + m_tracksHeight / 2, (info.endPos - info.startPos).frames(m_document->fps()) - 1, 5);
         QList<QGraphicsItem *> selection = m_scene->items(rect);
         if (item) selection.removeAll(item);
-        for (int i = 0; i < selection.count(); i++) {
-            if (selection.at(i)->type() == AVWIDGET) {
+        for (int i = 0; i < selection.count(); ++i) {
+            if (selection.at(i)->type() == AVWidget) {
                 ClipItem *clip = static_cast<ClipItem *>(selection.at(i));
                 if (clip->startPos() < info.startPos) {
                     if (clip->endPos() > info.startPos) {
@@ -2824,10 +2966,10 @@ void CustomTrackView::adjustTimelineClips(EDITMODE mode, ClipItem *item, ItemInf
                         dupInfo.startPos = info.startPos;
                         dupInfo.cropStart += diff;
                         dupInfo.cropDuration = clipInfo.endPos - info.startPos;
-                        new RazorClipCommand(this, clipInfo, info.startPos, false, command);
+                        new RazorClipCommand(this, clipInfo, clip->effectList(), info.startPos, true, command);
                         // Commented out; variable dup unused. --granjow
                         //ClipItem *dup = cutClip(clipInfo, info.startPos, true, false);
-                        cutClip(clipInfo, info.startPos, true, false);
+                        //cutClip(clipInfo, info.startPos, true, false);
                     }
                 }
                 // TODO: add insertspacecommand
@@ -2849,9 +2991,9 @@ void CustomTrackView::adjustTimelineTransitions(EDITMODE mode, Transition *item,
         QRectF rect(info.startPos.frames(m_document->fps()), info.track * m_tracksHeight + m_tracksHeight, (info.endPos - info.startPos).frames(m_document->fps()) - 1, 5);
         QList<QGraphicsItem *> selection = m_scene->items(rect);
         selection.removeAll(item);
-        for (int i = 0; i < selection.count(); i++) {
+        for (int i = 0; i < selection.count(); ++i) {
             if (!selection.at(i)->isEnabled()) continue;
-            if (selection.at(i)->type() == TRANSITIONWIDGET) {
+            if (selection.at(i)->type() == TransitionWidget) {
                 Transition *tr = static_cast<Transition *>(selection.at(i));
                 if (tr->startPos() < info.startPos) {
                     ItemInfo firstPos = tr->info();
@@ -2896,6 +3038,7 @@ Qt::DropActions CustomTrackView::supportedDropActions() const
 
 void CustomTrackView::setDuration(int duration)
 {
+    if (m_projectDuration == duration) return;
     int diff = qAbs(duration - sceneRect().width());
     if (diff * matrix().m11() > -50) {
         if (matrix().m11() < 0.4) setSceneRect(0, 0, (duration + 100 / matrix().m11()), sceneRect().height());
@@ -2909,26 +3052,26 @@ int CustomTrackView::duration() const
     return m_projectDuration;
 }
 
-void CustomTrackView::addTrack(TrackInfo type, int ix)
+void CustomTrackView::addTrack(const TrackInfo &type, int ix)
 {
     QList <TransitionInfo> transitionInfos;
     if (ix == -1 || ix == m_document->tracksCount()) {
         m_document->insertTrack(0, type);
-        transitionInfos = m_document->renderer()->mltInsertTrack(1, type.type == VIDEOTRACK);
+        transitionInfos = m_document->renderer()->mltInsertTrack(1, type.type == VideoTrack);
     } else {
         m_document->insertTrack(m_document->tracksCount() - ix, type);
         // insert track in MLT playlist
-        transitionInfos = m_document->renderer()->mltInsertTrack(m_document->tracksCount() - ix, type.type == VIDEOTRACK);
+        transitionInfos = m_document->renderer()->mltInsertTrack(m_document->tracksCount() - ix, type.type == VideoTrack);
 
         double startY = ix * m_tracksHeight + 1 + m_tracksHeight / 2;
         QRectF r(0, startY, sceneRect().width(), sceneRect().height() - startY);
         QList<QGraphicsItem *> selection = m_scene->items(r);
         resetSelectionGroup();
-
+        m_selectionMutex.lock();
         m_selectionGroup = new AbstractGroupItem(m_document->fps());
         scene()->addItem(m_selectionGroup);
-        for (int i = 0; i < selection.count(); i++) {
-            if ((!selection.at(i)->parentItem()) && (selection.at(i)->type() == AVWIDGET || selection.at(i)->type() == TRANSITIONWIDGET || selection.at(i)->type() == GROUPWIDGET)) {
+        for (int i = 0; i < selection.count(); ++i) {
+            if ((!selection.at(i)->parentItem()) && (selection.at(i)->type() == AVWidget || selection.at(i)->type() == TransitionWidget || selection.at(i)->type() == GroupWidget)) {
                 m_selectionGroup->addItem(selection.at(i));
             }
         }
@@ -2938,8 +3081,8 @@ void CustomTrackView::addTrack(TrackInfo type, int ix)
         // adjust track number
         Mlt::Tractor *tractor = m_document->renderer()->lockService();
         QList<QGraphicsItem *> children = m_selectionGroup->childItems();
-        for (int i = 0; i < children.count(); i++) {
-            if (children.at(i)->type() == GROUPWIDGET) {
+        for (int i = 0; i < children.count(); ++i) {
+            if (children.at(i)->type() == GroupWidget) {
                 AbstractGroupItem *grp = static_cast<AbstractGroupItem*>(children.at(i));
                 children << grp->childItems();
                 continue;
@@ -2947,12 +3090,12 @@ void CustomTrackView::addTrack(TrackInfo type, int ix)
             AbstractClipItem *item = static_cast <AbstractClipItem *>(children.at(i));
             item->updateItem();
             ItemInfo clipinfo = item->info();
-            if (item->type() == AVWIDGET) {
+            if (item->type() == AVWidget) {
                 ClipItem *clip = static_cast <ClipItem *>(item);
                 // slowmotion clips are not track dependant, so no need to update them
                 if (clip->speed() != 1.0) continue;
                 // We add a move clip command so that we get the correct producer for new track number
-                if (clip->clipType() == AV || clip->clipType() == AUDIO) {
+                if (clip->clipType() == AV || clip->clipType() == Audio) {
                     Mlt::Producer *prod = clip->getProducer(clipinfo.track);
                     if (m_document->renderer()->mltUpdateClipProducer(tractor, (int)(m_document->tracksCount() - clipinfo.track), clipinfo.startPos.frames(m_document->fps()), prod) == false) {
                         // problem updating clip
@@ -2968,27 +3111,25 @@ void CustomTrackView::addTrack(TrackInfo type, int ix)
             }*/
         }
         // Sync transition tracks with MLT playlist
-        Transition *tr;        
-       TransitionInfo info;
-       for (int i = 0; i < transitionInfos.count(); i++) {
-           info = transitionInfos.at(i);
-           tr = getTransitionItem(info);
-           if (tr) tr->setForcedTrack(info.forceTrack, info.a_track);
-           else kDebug()<<"// Cannot update TRANSITION AT: "<<info.b_track<<" / "<<info.startPos.frames(m_document->fps()); 
-       }
-       
+        Transition *tr;
+        TransitionInfo info;
+        for (int i = 0; i < transitionInfos.count(); ++i) {
+            info = transitionInfos.at(i);
+            tr = getTransitionItem(info);
+            if (tr) tr->setForcedTrack(info.forceTrack, info.a_track);
+            else kDebug()<<"// Cannot update TRANSITION AT: "<<info.b_track<<" / "<<info.startPos.frames(m_document->fps());
+        }
+        m_selectionMutex.unlock();
         resetSelectionGroup(false);
         m_document->renderer()->unlockService(tractor);
     }
 
     int maxHeight = m_tracksHeight * m_document->tracksCount() * matrix().m22();
-    for (int i = 0; i < m_guides.count(); i++) {
-        QLineF l = m_guides.at(i)->line();
-        l.setP2(QPointF(l.x2(), maxHeight));
-        m_guides.at(i)->setLine(l);
+    for (int i = 0; i < m_guides.count(); ++i) {
+        m_guides.at(i)->setLine(0, 0, 0, maxHeight - 1);
     }
 
-    m_cursorLine->setLine(m_cursorLine->line().x1(), 0, m_cursorLine->line().x1(), maxHeight - 1);
+    m_cursorLine->setLine(0, 0, 0, maxHeight - 1);
     setSceneRect(0, 0, sceneRect().width(), m_tracksHeight * m_document->tracksCount());
     viewport()->update();
     //QTimer::singleShot(500, this, SIGNAL(trackHeightChanged()));
@@ -2999,6 +3140,10 @@ void CustomTrackView::addTrack(TrackInfo type, int ix)
 
 void CustomTrackView::removeTrack(int ix)
 {
+    // Clear effect stack
+    clearSelection();
+    emit transitionItemSelected(NULL);
+
     // Delete track in MLT playlist
     m_document->renderer()->mltDeleteTrack(m_document->tracksCount() - ix);
     m_document->deleteTrack(m_document->tracksCount() - ix - 1);
@@ -3006,13 +3151,11 @@ void CustomTrackView::removeTrack(int ix)
     double startY = ix * (m_tracksHeight + 1) + m_tracksHeight / 2;
     QRectF r(0, startY, sceneRect().width(), sceneRect().height() - startY);
     QList<QGraphicsItem *> selection = m_scene->items(r);
-
-    resetSelectionGroup();
-
+    m_selectionMutex.lock();
     m_selectionGroup = new AbstractGroupItem(m_document->fps());
     scene()->addItem(m_selectionGroup);
-    for (int i = 0; i < selection.count(); i++) {
-        if ((!selection.at(i)->parentItem()) && (selection.at(i)->type() == AVWIDGET || selection.at(i)->type() == TRANSITIONWIDGET || selection.at(i)->type() == GROUPWIDGET)) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if ((selection.at(i) && !selection.at(i)->parentItem() && selection.at(i)->isEnabled()) && (selection.at(i)->type() == AVWidget || selection.at(i)->type() == TransitionWidget || selection.at(i)->type() == GroupWidget)) {
             m_selectionGroup->addItem(selection.at(i));
         }
     }
@@ -3024,24 +3167,24 @@ void CustomTrackView::removeTrack(int ix)
     // adjust track number
     QList<QGraphicsItem *> children = m_selectionGroup->childItems();
     //kDebug() << "// FOUND CLIPS TO MOVE: " << children.count();
-    for (int i = 0; i < children.count(); i++) {
-        if (children.at(i)->type() == GROUPWIDGET) {
+    for (int i = 0; i < children.count(); ++i) {
+        if (children.at(i)->type() == GroupWidget) {
             AbstractGroupItem *grp = static_cast<AbstractGroupItem*>(children.at(i));
             children << grp->childItems();
             continue;
         }
-        if (children.at(i)->type() == AVWIDGET) {
+        if (children.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(children.at(i));
             clip->updateItem();
             ItemInfo clipinfo = clip->info();
             // We add a move clip command so that we get the correct producer for new track number
-            if (clip->clipType() == AV || clip->clipType() == AUDIO || clip->clipType() == PLAYLIST) {
+            if (clip->clipType() == AV || clip->clipType() == Audio || clip->clipType() == Playlist) {
                 Mlt::Producer *prod = clip->getProducer(clipinfo.track);
                 if (prod == NULL || !m_document->renderer()->mltUpdateClipProducer(tractor, (int)(m_document->tracksCount() - clipinfo.track), clipinfo.startPos.frames(m_document->fps()), prod)) {
                     emit displayMessage(i18n("Cannot update clip (time: %1, track: %2)", clipinfo.startPos.frames(m_document->fps()), clipinfo.track), ErrorMessage);
                 }
             }
-        } else if (children.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (children.at(i)->type() == TransitionWidget) {
             Transition *tr = static_cast <Transition *>(children.at(i));
             tr->updateItem();
             int track = tr->transitionEndTrack();
@@ -3051,16 +3194,15 @@ void CustomTrackView::removeTrack(int ix)
             }
         }
     }
+    m_selectionMutex.unlock();
     resetSelectionGroup(false);
     m_document->renderer()->unlockService(tractor);
 
     int maxHeight = m_tracksHeight * m_document->tracksCount() * matrix().m22();
-    for (int i = 0; i < m_guides.count(); i++) {
-        QLineF l = m_guides.at(i)->line();
-        l.setP2(QPointF(l.x2(), maxHeight));
-        m_guides.at(i)->setLine(l);
+    for (int i = 0; i < m_guides.count(); ++i) {
+        m_guides.at(i)->setLine(0, 0, 0, maxHeight - 1);
     }
-    m_cursorLine->setLine(m_cursorLine->line().x1(), 0, m_cursorLine->line().x1(), maxHeight - 1);
+    m_cursorLine->setLine(0, 0, 0, maxHeight - 1);
     setSceneRect(0, 0, sceneRect().width(), m_tracksHeight * m_document->tracksCount());
 
     m_selectedTrack = qMin(m_selectedTrack, m_document->tracksCount() - 1);
@@ -3070,7 +3212,7 @@ void CustomTrackView::removeTrack(int ix)
     //QTimer::singleShot(500, this, SIGNAL(trackHeightChanged()));
 }
 
-void CustomTrackView::configTracks(QList < TrackInfo > trackInfos)
+void CustomTrackView::configTracks(const QList < TrackInfo > &trackInfos)
 {
     for (int i = 0; i < trackInfos.count(); ++i) {
         m_document->setTrackType(i, trackInfos.at(i));
@@ -3084,7 +3226,7 @@ void CustomTrackView::configTracks(QList < TrackInfo > trackInfos)
 
 void CustomTrackView::slotSwitchTrackAudio(int ix)
 {
-    /*for (int i = 0; i < m_document->tracksCount(); i++)
+    /*for (int i = 0; i < m_document->tracksCount(); ++i)
         kDebug() << "TRK " << i << " STATE: " << m_document->trackInfoAt(i).isMute << m_document->trackInfoAt(i).isBlind;*/
     int tracknumber = m_document->tracksCount() - ix - 1;
     m_document->switchTrackAudio(tracknumber, !m_document->trackInfoAt(tracknumber).isMute);
@@ -3110,8 +3252,8 @@ void CustomTrackView::lockTrack(int ix, bool lock, bool requestUpdate)
     AbstractClipItem *clip = NULL;
     QList<QGraphicsItem *> selection = m_scene->items(0, ix * m_tracksHeight + m_tracksHeight / 2, sceneRect().width(), m_tracksHeight / 2 - 2);
 
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == GROUPWIDGET && (AbstractGroupItem *)selection.at(i) != m_selectionGroup) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == GroupWidget && (AbstractGroupItem *)selection.at(i) != m_selectionGroup) {
             if (selection.at(i)->parentItem() && m_selectionGroup) {
                 selection.removeAll((QGraphicsItem*)m_selectionGroup);
                 resetSelectionGroup();
@@ -3122,9 +3264,9 @@ void CustomTrackView::lockTrack(int ix, bool lock, bool requestUpdate)
             QList <QGraphicsItem *> children =  selection.at(i)->childItems();
             for (int j = 0; j < children.count(); ++j) {
                 if (children.at(j)->isSelected()) {
-                    if (children.at(j)->type() == AVWIDGET)
+                    if (children.at(j)->type() == AVWidget)
                         emit clipItemSelected(NULL);
-                    else if (children.at(j)->type() == TRANSITIONWIDGET)
+                    else if (children.at(j)->type() == TransitionWidget)
                         emit transitionItemSelected(NULL);
                     else
                         continue;
@@ -3144,7 +3286,7 @@ void CustomTrackView::lockTrack(int ix, bool lock, bool requestUpdate)
             }
             if (changeGroupLock && hasClipOnTrack)
                 ((AbstractGroupItem*)selection.at(i))->setItemLocked(lock);
-        } else if((selection.at(i)->type() == AVWIDGET || selection.at(i)->type() == TRANSITIONWIDGET)) {
+        } else if((selection.at(i)->type() == AVWidget || selection.at(i)->type() == TransitionWidget)) {
             if (selection.at(i)->parentItem()) {
                 if (selection.at(i)->parentItem() == m_selectionGroup) {
                     selection.removeAll((QGraphicsItem*)m_selectionGroup);
@@ -3156,7 +3298,7 @@ void CustomTrackView::lockTrack(int ix, bool lock, bool requestUpdate)
             }
 
             if (selection.at(i)->isSelected()) {
-                if (selection.at(i)->type() == AVWIDGET)
+                if (selection.at(i)->type() == AVWidget)
                     emit clipItemSelected(NULL);
                 else
                     emit transitionItemSelected(NULL);
@@ -3186,9 +3328,9 @@ QList<QGraphicsItem *> CustomTrackView::checkForGroups(const QRectF &rect, bool
     QList<QGraphicsItem *> selection = scene()->items(rect);
     *ok = true;
     int maxHeight = m_tracksHeight * 1.5;
-    for (int i = 0; i < selection.count(); i++) {
+    for (int i = 0; i < selection.count(); ++i) {
         // Check that we don't try to move a group with clips on other tracks
-        if (selection.at(i)->type() == GROUPWIDGET && (selection.at(i)->boundingRect().height() >= maxHeight)) {
+        if (selection.at(i)->type() == GroupWidget && (selection.at(i)->boundingRect().height() >= maxHeight)) {
             *ok = false;
             break;
         } else if (selection.at(i)->parentItem() && (selection.at(i)->parentItem()->boundingRect().height() >= maxHeight)) {
@@ -3254,13 +3396,13 @@ void CustomTrackView::slotRemoveSpace()
     QList<ItemInfo> clipsToMove;
     QList<ItemInfo> transitionsToMove;
 
-    for (int i = 0; i < items.count(); i++) {
-        if (items.at(i)->type() == AVWIDGET || items.at(i)->type() == TRANSITIONWIDGET) {
+    for (int i = 0; i < items.count(); ++i) {
+        if (items.at(i)->type() == AVWidget || items.at(i)->type() == TransitionWidget) {
             AbstractClipItem *item = static_cast <AbstractClipItem *>(items.at(i));
             ItemInfo info = item->info();
-            if (item->type() == AVWIDGET) {
+            if (item->type() == AVWidget) {
                 clipsToMove.append(info);
-            } else if (item->type() == TRANSITIONWIDGET) {
+            } else if (item->type() == TransitionWidget) {
                 transitionsToMove.append(info);
             }
         }
@@ -3270,15 +3412,15 @@ void CustomTrackView::slotRemoveSpace()
         // Make sure that by moving the items, we don't get a transition collision
         // Find first transition
         ItemInfo info = transitionsToMove.at(0);
-        for (int i = 1; i < transitionsToMove.count(); i++)
+        for (int i = 1; i < transitionsToMove.count(); ++i)
             if (transitionsToMove.at(i).startPos < info.startPos) info = transitionsToMove.at(i);
 
         // make sure there are no transitions on the way
         QRectF rect(info.startPos.frames(m_document->fps()) - length, track * m_tracksHeight + m_tracksHeight / 2, length - 1, m_tracksHeight / 2 - 2);
         items = scene()->items(rect);
         int transitionCorrection = -1;
-        for (int i = 0; i < items.count(); i++) {
-            if (items.at(i)->type() == TRANSITIONWIDGET) {
+        for (int i = 0; i < items.count(); ++i) {
+            if (items.at(i)->type() == TransitionWidget) {
                 // There is a transition on the way
                 AbstractClipItem *item = static_cast <AbstractClipItem *>(items.at(i));
                 int transitionEnd = item->endPos().frames(m_document->fps());
@@ -3290,12 +3432,12 @@ void CustomTrackView::slotRemoveSpace()
             // We need to fix the move length
             length = info.startPos.frames(m_document->fps()) - transitionCorrection;
         }
-            
+
         // Make sure we don't send transition before 0
         if (info.startPos.frames(m_document->fps()) < length) {
             // reduce length to maximum possible
             length = info.startPos.frames(m_document->fps());
-        }           
+        }
     }
 
     InsertSpaceCommand *command = new InsertSpaceCommand(this, clipsToMove, transitionsToMove, track, GenTime(-length, m_document->fps()), true);
@@ -3314,7 +3456,7 @@ void CustomTrackView::slotInsertSpace()
         track = (int)(mapToScene(m_menuPosition).y() / m_tracksHeight) + 1;
     }
     QPointer<SpacerDialog> d = new SpacerDialog(GenTime(65, m_document->fps()),
-                m_document->timecode(), track, m_document->tracksList(), this);
+                                                m_document->timecode(), track, m_document->tracksList(), this);
     if (d->exec() != QDialog::Accepted) {
         delete d;
         return;
@@ -3350,13 +3492,13 @@ void CustomTrackView::slotInsertSpace()
     QList<ItemInfo> clipsToMove;
     QList<ItemInfo> transitionsToMove;
 
-    for (int i = 0; i < items.count(); i++) {
-        if (items.at(i)->type() == AVWIDGET || items.at(i)->type() == TRANSITIONWIDGET) {
+    for (int i = 0; i < items.count(); ++i) {
+        if (items.at(i)->type() == AVWidget || items.at(i)->type() == TransitionWidget) {
             AbstractClipItem *item = static_cast <AbstractClipItem *>(items.at(i));
             ItemInfo info = item->info();
-            if (item->type() == AVWIDGET)
+            if (item->type() == AVWidget)
                 clipsToMove.append(info);
-            else if (item->type() == TRANSITIONWIDGET)
+            else if (item->type() == TransitionWidget)
                 transitionsToMove.append(info);
         }
     }
@@ -3372,6 +3514,7 @@ void CustomTrackView::insertSpace(QList<ItemInfo> clipsToMove, QList<ItemInfo> t
 {
     int diff = duration.frames(m_document->fps());
     resetSelectionGroup();
+    m_selectionMutex.lock();
     m_selectionGroup = new AbstractGroupItem(m_document->fps());
     scene()->addItem(m_selectionGroup);
     ClipItem *clip;
@@ -3381,46 +3524,46 @@ void CustomTrackView::insertSpace(QList<ItemInfo> clipsToMove, QList<ItemInfo> t
     QMap <int, int> trackClipStartList;
     QMap <int, int> trackTransitionStartList;
 
-    for (int i = 1; i < m_document->tracksCount() + 1; i++) {
+    for (int i = 1; i < m_document->tracksCount() + 1; ++i) {
         trackClipStartList[i] = -1;
         trackTransitionStartList[i] = -1;
     }
 
-    if (!clipsToMove.isEmpty()) for (int i = 0; i < clipsToMove.count(); i++) {
-            clip = getClipItemAtStart(clipsToMove.at(i).startPos + offset, clipsToMove.at(i).track);
-            if (clip) {
-                if (clip->parentItem()) {
-                    m_selectionGroup->addItem(clip->parentItem());
-                } else {
-                    m_selectionGroup->addItem(clip);
-                }
-                if (trackClipStartList.value(m_document->tracksCount() - clipsToMove.at(i).track) == -1 || clipsToMove.at(i).startPos.frames(m_document->fps()) < trackClipStartList.value(m_document->tracksCount() - clipsToMove.at(i).track))
-                    trackClipStartList[m_document->tracksCount() - clipsToMove.at(i).track] = clipsToMove.at(i).startPos.frames(m_document->fps());
-            } else emit {
-                    displayMessage(i18n("Cannot move clip at position %1, track %2", m_document->timecode().getTimecodeFromFrames((clipsToMove.at(i).startPos + offset).frames(m_document->fps())), clipsToMove.at(i).track), ErrorMessage);
-                }
+    if (!clipsToMove.isEmpty()) for (int i = 0; i < clipsToMove.count(); ++i) {
+        clip = getClipItemAtStart(clipsToMove.at(i).startPos + offset, clipsToMove.at(i).track);
+        if (clip) {
+            if (clip->parentItem()) {
+                m_selectionGroup->addItem(clip->parentItem());
+            } else {
+                m_selectionGroup->addItem(clip);
             }
-    if (!transToMove.isEmpty()) for (int i = 0; i < transToMove.count(); i++) {
-            transition = getTransitionItemAtStart(transToMove.at(i).startPos + offset, transToMove.at(i).track);
-            if (transition) {
-                if (transition->parentItem()) {
-                    m_selectionGroup->addItem(transition->parentItem());
-                } else {
-                    m_selectionGroup->addItem(transition);
-                }
-                if (trackTransitionStartList.value(m_document->tracksCount() - transToMove.at(i).track) == -1 || transToMove.at(i).startPos.frames(m_document->fps()) < trackTransitionStartList.value(m_document->tracksCount() - transToMove.at(i).track))
-                    trackTransitionStartList[m_document->tracksCount() - transToMove.at(i).track] = transToMove.at(i).startPos.frames(m_document->fps());
-            } else emit displayMessage(i18n("Cannot move transition at position %1, track %2", m_document->timecode().getTimecodeFromFrames(transToMove.at(i).startPos.frames(m_document->fps())), transToMove.at(i).track), ErrorMessage);
+            if (trackClipStartList.value(m_document->tracksCount() - clipsToMove.at(i).track) == -1 || clipsToMove.at(i).startPos.frames(m_document->fps()) < trackClipStartList.value(m_document->tracksCount() - clipsToMove.at(i).track))
+                trackClipStartList[m_document->tracksCount() - clipsToMove.at(i).track] = clipsToMove.at(i).startPos.frames(m_document->fps());
+        } else {
+            emit displayMessage(i18n("Cannot move clip at position %1, track %2", m_document->timecode().getTimecodeFromFrames((clipsToMove.at(i).startPos + offset).frames(m_document->fps())), clipsToMove.at(i).track), ErrorMessage);
         }
+    }
+    if (!transToMove.isEmpty()) for (int i = 0; i < transToMove.count(); ++i) {
+        transition = getTransitionItemAtStart(transToMove.at(i).startPos + offset, transToMove.at(i).track);
+        if (transition) {
+            if (transition->parentItem()) {
+                m_selectionGroup->addItem(transition->parentItem());
+            } else {
+                m_selectionGroup->addItem(transition);
+            }
+            if (trackTransitionStartList.value(m_document->tracksCount() - transToMove.at(i).track) == -1 || transToMove.at(i).startPos.frames(m_document->fps()) < trackTransitionStartList.value(m_document->tracksCount() - transToMove.at(i).track))
+                trackTransitionStartList[m_document->tracksCount() - transToMove.at(i).track] = transToMove.at(i).startPos.frames(m_document->fps());
+        } else emit displayMessage(i18n("Cannot move transition at position %1, track %2", m_document->timecode().getTimecodeFromFrames(transToMove.at(i).startPos.frames(m_document->fps())), transToMove.at(i).track), ErrorMessage);
+    }
     m_selectionGroup->translate(diff, 0);
 
     // update items coordinates
     QList<QGraphicsItem *> itemList = m_selectionGroup->childItems();
 
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET || itemList.at(i)->type() == TRANSITIONWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget || itemList.at(i)->type() == TransitionWidget) {
             static_cast < AbstractClipItem *>(itemList.at(i))->updateItem();
-        } else if (itemList.at(i)->type() == GROUPWIDGET) {
+        } else if (itemList.at(i)->type() == GroupWidget) {
             QList<QGraphicsItem *> children = itemList.at(i)->childItems();
             for (int j = 0; j < children.count(); j++) {
                 AbstractClipItem * clp = static_cast < AbstractClipItem *>(children.at(j));
@@ -3428,6 +3571,7 @@ void CustomTrackView::insertSpace(QList<ItemInfo> clipsToMove, QList<ItemInfo> t
             }
         }
     }
+    m_selectionMutex.unlock();
     resetSelectionGroup(false);
     if (track != -1)
         track = m_document->tracksCount() - track;
@@ -3439,9 +3583,10 @@ void CustomTrackView::deleteClip(const QString &clipId)
     resetSelectionGroup();
     QList<QGraphicsItem *> itemList = items();
     QUndoCommand *deleteCommand = new QUndoCommand();
+    new RefreshMonitorCommand(this, false, true, deleteCommand);
     int count = 0;
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *item = (ClipItem *)itemList.at(i);
             if (item->clipProducer() == clipId) {
                 count++;
@@ -3453,15 +3598,14 @@ void CustomTrackView::deleteClip(const QString &clipId)
             }
         }
     }
-    deleteCommand->setText(i18np("Delete timeline clip", "Delete timeline clips", count));
     if (count == 0) {
         delete deleteCommand;
     } else {
+        deleteCommand->setText(i18np("Delete timeline clip", "Delete timeline clips", count));
+        new RefreshMonitorCommand(this, true, false, deleteCommand);
         updateTrackDuration(-1, deleteCommand);
-       new RefreshMonitorCommand(this, false, deleteCommand);
         m_commandStack->push(deleteCommand);
     }
-    m_document->renderer()->doRefresh();    
 }
 
 void CustomTrackView::seekCursorPos(int pos)
@@ -3479,10 +3623,10 @@ int CustomTrackView::seekPosition() const
 void CustomTrackView::setCursorPos(int pos)
 {
     if (pos != m_cursorPos) {
-       emit cursorMoved((int)(m_cursorPos), (int)(pos));
-       m_cursorPos = pos;
-       m_cursorLine->setPos(m_cursorPos, 0);
-       if (m_autoScroll) checkScrolling();
+        emit cursorMoved((int)(m_cursorPos), (int)(pos));
+        m_cursorPos = pos;
+        m_cursorLine->setPos(m_cursorPos, 0);
+        if (m_autoScroll) checkScrolling();
     }
     else emit updateRuler();
 }
@@ -3492,19 +3636,19 @@ void CustomTrackView::updateCursorPos()
     m_cursorLine->setPos(m_cursorPos, 0);
 }
 
-int CustomTrackView::cursorPos()
+int CustomTrackView::cursorPos() const
 {
-    return (int)(m_cursorPos);
+    return m_cursorPos;
 }
 
 void CustomTrackView::moveCursorPos(int delta)
 {
     int currentPos = m_document->renderer()->requestedSeekPosition;
     if (currentPos == SEEK_INACTIVE) {
-       currentPos = m_document->renderer()->seekPosition().frames(m_document->fps()) + delta;
+        currentPos = m_document->renderer()->seekPosition().frames(m_document->fps()) + delta;
     }
     else {
-       currentPos += delta;
+        currentPos += delta;
     }
     m_document->renderer()->seek(qMax(0, currentPos));
     emit updateRuler();
@@ -3512,7 +3656,7 @@ void CustomTrackView::moveCursorPos(int delta)
 
 void CustomTrackView::initCursorPos(int pos)
 {
-    emit cursorMoved((int)(m_cursorPos), (int)(pos));
+    emit cursorMoved(m_cursorPos, pos);
     m_cursorPos = pos;
     m_cursorLine->setPos(pos, 0);
     checkScrolling();
@@ -3525,23 +3669,23 @@ void CustomTrackView::checkScrolling()
 
 void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
 {
-    if (m_moveOpMode == SEEK) m_moveOpMode = NONE;
-    if (m_operationMode == SCROLLTIMELINE) {
-       m_operationMode = NONE;
-       setDragMode(QGraphicsView::NoDrag);
-       QGraphicsView::mouseReleaseEvent(event);
-       return;
+    if (m_moveOpMode == Seek) m_moveOpMode = None;
+    if (m_operationMode == ScrollTimeline) {
+        m_operationMode = None;
+        setDragMode(QGraphicsView::NoDrag);
+        QGraphicsView::mouseReleaseEvent(event);
+        return;
     }
-    if (!m_controlModifier && m_operationMode != RUBBERSELECTION) {
-       //event->accept();
-       setDragMode(QGraphicsView::NoDrag);
-       if (m_clipDrag) QGraphicsView::mouseReleaseEvent(event);
+    if (!m_controlModifier && m_operationMode != RubberSelection) {
+        //event->accept();
+        setDragMode(QGraphicsView::NoDrag);
+        if (m_clipDrag) QGraphicsView::mouseReleaseEvent(event);
     }
     m_clipDrag = false;
     //setViewportUpdateMode(QGraphicsView::MinimalViewportUpdate);
 #if QT_VERSION >= 0x040800
     if (m_dragItem) {
-       m_dragItem->setGraphicsEffect(NULL);
+        m_dragItem->setGraphicsEffect(NULL);
     }
 #endif
     if (m_scrollTimer.isActive()) m_scrollTimer.stop();
@@ -3549,9 +3693,9 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
         return;
     }
 
-    if (m_operationMode == MOVEGUIDE) {
+    if (m_operationMode == MoveGuide) {
         setCursor(Qt::ArrowCursor);
-        m_operationMode = NONE;
+        m_operationMode = None;
         m_dragGuide->setFlag(QGraphicsItem::ItemIsMovable, false);
         GenTime newPos = GenTime(m_dragGuide->pos().x(), m_document->fps());
         if (newPos != m_dragGuide->position()) {
@@ -3563,15 +3707,16 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
         }
         m_dragGuide = NULL;
         m_dragItem = NULL;
+        QGraphicsView::mouseReleaseEvent(event);
         return;
-    } else if (m_operationMode == SPACER && m_selectionGroup) {
+    } else if (m_operationMode == Spacer && m_selectionGroup) {
         int track;
         if (event->modifiers() != Qt::ControlModifier) {
             // We are moving all tracks
             track = -1;
         } else track = (int)(mapToScene(m_clickEvent).y() / m_tracksHeight);
         GenTime timeOffset = GenTime((int)(m_selectionGroup->scenePos().x()), m_document->fps()) - m_selectionGroupInfo.startPos;
-       QList <AbstractGroupItem*> groups;
+        QList <AbstractGroupItem*> groups;
 
         if (timeOffset != GenTime()) {
             QList<QGraphicsItem *> items = m_selectionGroup->childItems();
@@ -3583,28 +3728,28 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
             QMap <int, int> trackClipStartList;
             QMap <int, int> trackTransitionStartList;
 
-            for (int i = 1; i < m_document->tracksCount() + 1; i++) {
+            for (int i = 1; i < m_document->tracksCount() + 1; ++i) {
                 trackClipStartList[i] = -1;
                 trackTransitionStartList[i] = -1;
             }
 
-            for (int i = 0; i < items.count(); i++) {
-                if (items.at(i)->type() == GROUPWIDGET) {
-                   AbstractGroupItem* group = (AbstractGroupItem*)items.at(i);
-                   if (!groups.contains(group)) groups.append(group);
-                   items += items.at(i)->childItems();
-               }
+            for (int i = 0; i < items.count(); ++i) {
+                if (items.at(i)->type() == GroupWidget) {
+                    AbstractGroupItem* group = (AbstractGroupItem*)items.at(i);
+                    if (!groups.contains(group)) groups.append(group);
+                    items += items.at(i)->childItems();
+                }
             }
 
-            for (int i = 0; i < items.count(); i++) {
-                if (items.at(i)->type() == AVWIDGET) {
+            for (int i = 0; i < items.count(); ++i) {
+                if (items.at(i)->type() == AVWidget) {
                     AbstractClipItem *item = static_cast <AbstractClipItem *>(items.at(i));
                     ItemInfo info = item->info();
                     clipsToMove.append(info);
                     item->updateItem();
                     if (trackClipStartList.value(m_document->tracksCount() - info.track) == -1 || info.startPos.frames(m_document->fps()) < trackClipStartList.value(m_document->tracksCount() - info.track))
                         trackClipStartList[m_document->tracksCount() - info.track] = info.startPos.frames(m_document->fps());
-                } else if (items.at(i)->type() == TRANSITIONWIDGET) {
+                } else if (items.at(i)->type() == TransitionWidget) {
                     AbstractClipItem *item = static_cast <AbstractClipItem *>(items.at(i));
                     ItemInfo info = item->info();
                     transitionsToMove.append(info);
@@ -3623,28 +3768,28 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
             }
         }
         resetSelectionGroup();
-        for (int i = 0; i < groups.count(); i++) {
-           rebuildGroup(groups.at(i));
-       }
+        for (int i = 0; i < groups.count(); ++i) {
+            rebuildGroup(groups.at(i));
+        }
+
 
-       
         clearSelection();
-       
-        m_operationMode = NONE;
-    } else if (m_operationMode == RUBBERSELECTION) {
-       //event->accept();
-       QGraphicsView::mouseReleaseEvent(event);
-       setDragMode(QGraphicsView::NoDrag);
-       setViewportUpdateMode(QGraphicsView::MinimalViewportUpdate);
-       if (event->modifiers() != Qt::ControlModifier) m_dragItem = NULL;
+
+        m_operationMode = None;
+    } else if (m_operationMode == RubberSelection) {
+        //event->accept();
+        QGraphicsView::mouseReleaseEvent(event);
+        setDragMode(QGraphicsView::NoDrag);
+        setViewportUpdateMode(QGraphicsView::MinimalViewportUpdate);
+        if (event->modifiers() != Qt::ControlModifier) m_dragItem = NULL;
         resetSelectionGroup();
         groupSelectedItems();
-        m_operationMode = NONE;
-       if (m_selectionGroup == NULL && m_dragItem) {
-           // Only 1 item selected
-           if (m_dragItem->type() == AVWIDGET)
-               emit clipItemSelected(static_cast<ClipItem *>(m_dragItem));
-       }
+        m_operationMode = None;
+        if (m_selectionGroup == NULL && m_dragItem) {
+            // Only 1 item selected
+            if (m_dragItem->type() == AVWidget)
+                emit clipItemSelected(static_cast<ClipItem *>(m_dragItem));
+        }
     }
 
     if (m_dragItem == NULL && m_selectionGroup == NULL) {
@@ -3654,11 +3799,11 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
     ItemInfo info;
     if (m_dragItem) info = m_dragItem->info();
 
-    if (m_operationMode == MOVE) {
+    if (m_operationMode == MoveOperation) {
         setCursor(Qt::OpenHandCursor);
         if (m_dragItem->parentItem() == 0) {
             // we are moving one clip, easy
-            if (m_dragItem->type() == AVWIDGET && (m_dragItemInfo.startPos != info.startPos || m_dragItemInfo.track != info.track)) {
+            if (m_dragItem->type() == AVWidget && (m_dragItemInfo.startPos != info.startPos || m_dragItemInfo.track != info.track)) {
                 ClipItem *item = static_cast <ClipItem *>(m_dragItem);
                 Mlt::Producer *prod = item->getProducer(info.track);
                 bool success = m_document->renderer()->mltMoveClip((int)(m_document->tracksCount() - m_dragItemInfo.track), (int)(m_document->tracksCount() - info.track), (int) m_dragItemInfo.startPos.frames(m_document->fps()), (int)(info.startPos.frames(m_document->fps())), prod, m_scene->editMode() == OVERWRITEEDIT, m_scene->editMode() == INSERTEDIT);
@@ -3785,7 +3930,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                     emit displayMessage(i18n("Cannot move clip to position %1", m_document->timecode().getTimecodeFromFrames(info.startPos.frames(m_document->fps()))), ErrorMessage);
                 }
                 setDocumentModified();
-            } else if (m_dragItem->type() == TRANSITIONWIDGET && (m_dragItemInfo.startPos != info.startPos || m_dragItemInfo.track != info.track)) {
+            } else if (m_dragItem->type() == TransitionWidget && (m_dragItemInfo.startPos != info.startPos || m_dragItemInfo.track != info.track)) {
                 Transition *transition = static_cast <Transition *>(m_dragItem);
                 transition->updateTransitionEndTrack(getPreviousVideoTrack(m_dragItem->track()));
                 if (!m_document->renderer()->mltMoveTransition(transition->transitionTag(), (int)(m_document->tracksCount() - m_dragItemInfo.track), (int)(m_document->tracksCount() - m_dragItem->track()), transition->transitionEndTrack(), m_dragItemInfo.startPos, m_dragItemInfo.endPos, info.startPos, info.endPos)) {
@@ -3810,10 +3955,10 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
             QGraphicsItemGroup *group;
             if (m_selectionGroup) {
                 group = static_cast <QGraphicsItemGroup *>(m_selectionGroup);
-           }
+            }
             else {
                 group = static_cast <QGraphicsItemGroup *>(m_dragItem->parentItem());
-           }
+            }
             QList<QGraphicsItem *> items = group->childItems();
             QList<ItemInfo> clipsToMove;
             QList<ItemInfo> transitionsToMove;
@@ -3828,17 +3973,17 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
 
                 // Expand groups
                 int max = items.count();
-                for (int i = 0; i < max; i++) {
-                    if (items.at(i)->type() == GROUPWIDGET) {
+                for (int i = 0; i < max; ++i) {
+                    if (items.at(i)->type() == GroupWidget) {
                         items += items.at(i)->childItems();
                     }
                 }
                 m_document->renderer()->blockSignals(true);
-                for (int i = 0; i < items.count(); i++) {
-                    if (items.at(i)->type() != AVWIDGET && items.at(i)->type() != TRANSITIONWIDGET) continue;
+                for (int i = 0; i < items.count(); ++i) {
+                    if (items.at(i)->type() != AVWidget && items.at(i)->type() != TransitionWidget) continue;
                     AbstractClipItem *item = static_cast <AbstractClipItem *>(items.at(i));
                     ItemInfo info = item->info();
-                    if (item->type() == AVWIDGET) {
+                    if (item->type() == AVWidget) {
                         if (m_document->renderer()->mltRemoveClip(m_document->tracksCount() - info.track, info.startPos) == false) {
                             // error, clip cannot be removed from playlist
                             emit displayMessage(i18n("Error removing clip at %1 on track %2", m_document->timecode().getTimecodeFromFrames(info.startPos.frames(m_document->fps())), info.track), ErrorMessage);
@@ -3852,9 +3997,9 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                     }
                 }
                 m_document->renderer()->blockSignals(false);
-                for (int i = 0; i < items.count(); i++) {
+                for (int i = 0; i < items.count(); ++i) {
                     // re-add items in correct place
-                    if (items.at(i)->type() != AVWIDGET && items.at(i)->type() != TRANSITIONWIDGET) continue;
+                    if (items.at(i)->type() != AVWidget && items.at(i)->type() != TransitionWidget) continue;
                     AbstractClipItem *item = static_cast <AbstractClipItem *>(items.at(i));
                     item->updateItem();
                     ItemInfo info = item->info();
@@ -3865,13 +4010,13 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                         item->setItemLocked(true);
                     }
 
-                    if (item->type() == AVWIDGET) {
+                    if (item->type() == AVWidget) {
                         ClipItem *clip = static_cast <ClipItem*>(item);
                         int trackProducer = info.track;
                         info.track = m_document->tracksCount() - info.track;
                         adjustTimelineClips(m_scene->editMode(), clip, ItemInfo(), moveGroup);
                         m_document->renderer()->mltInsertClip(info, clip->xml(), clip->getProducer(trackProducer), m_scene->editMode() == OVERWRITEEDIT, m_scene->editMode() == INSERTEDIT);
-                        for (int i = 0; i < clip->effectsCount(); i++) {
+                        for (int i = 0; i < clip->effectsCount(); ++i) {
                             m_document->renderer()->mltAddEffect(info.track, info.startPos, getEffectArgs(clip->effect(i)), false);
                         }
                     } else {
@@ -3897,51 +4042,51 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                 if (m_selectionGroup) {
                     m_selectionGroupInfo.startPos = GenTime(m_selectionGroup->scenePos().x(), m_document->fps());
                     m_selectionGroupInfo.track = m_selectionGroup->track();
-                   items = m_selectionGroup->childItems();
-                   resetSelectionGroup(false);
-
-                   QSet <QGraphicsItem*> groupList;
-                   QSet <QGraphicsItem*> itemList;
-                   while (!items.isEmpty()) {
-                       QGraphicsItem *first = items.takeFirst();
-                       if (first->type() == GROUPWIDGET) {
-                           if (first != m_selectionGroup) {
-                               groupList.insert(first);
-                           }
+                    items = m_selectionGroup->childItems();
+                    resetSelectionGroup(false);
+
+                    QSet <QGraphicsItem*> groupList;
+                    QSet <QGraphicsItem*> itemList;
+                    while (!items.isEmpty()) {
+                        QGraphicsItem *first = items.takeFirst();
+                        if (first->type() == GroupWidget) {
+                            if (first != m_selectionGroup) {
+                                groupList.insert(first);
+                            }
+                        }
+                        else if (first->type() == AVWidget || first->type() == TransitionWidget) {
+                            if (first->parentItem() && first->parentItem()->type() == GroupWidget) {
+                                if (first->parentItem() != m_selectionGroup) {
+                                    groupList.insert(first->parentItem());
+                                }
+                                else itemList.insert(first);
+                            }
+                            else itemList.insert(first);
                         }
-                        else if (first->type() == AVWIDGET || first->type() == TRANSITIONWIDGET) {
-                           if (first->parentItem() && first->parentItem()->type() == GROUPWIDGET) {
-                               if (first->parentItem() != m_selectionGroup) {
-                                   groupList.insert(first->parentItem());
-                               }
-                               else itemList.insert(first);
-                           }
-                           else itemList.insert(first);
-                       }
-                   }
-                   foreach(QGraphicsItem *item, groupList) {
-                       itemList.unite(item->childItems().toSet());
-                       rebuildGroup(static_cast <AbstractGroupItem*>(item));
-                   }
+                    }
+                    foreach(QGraphicsItem *item, groupList) {
+                        itemList.unite(item->childItems().toSet());
+                        rebuildGroup(static_cast <AbstractGroupItem*>(item));
+                    }
 
                     foreach(QGraphicsItem *item, itemList) {
-                       item->setSelected(true);
+                        item->setSelected(true);
                         if (item->parentItem())
                             item->parentItem()->setSelected(true);
                     }
                     resetSelectionGroup();
                     groupSelectedItems(itemList.toList());
                 } else {
-                   AbstractGroupItem *grp = static_cast <AbstractGroupItem *>(group);
+                    AbstractGroupItem *grp = static_cast <AbstractGroupItem *>(group);
                     rebuildGroup(grp);
                 }
                 setDocumentModified();
             }
         }
         m_document->renderer()->doRefresh();
-    } else if (m_operationMode == RESIZESTART && m_dragItem->startPos() != m_dragItemInfo.startPos) {
+    } else if (m_operationMode == ResizeStart && m_dragItem->startPos() != m_dragItemInfo.startPos) {
         // resize start
-        if (!m_controlModifier && m_dragItem->type() == AVWIDGET && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
+        if (!m_controlModifier && m_dragItem->type() == AVWidget && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
             AbstractGroupItem *parent = static_cast <AbstractGroupItem *>(m_dragItem->parentItem());
             if (parent) {
                 QUndoCommand *resizeCommand = new QUndoCommand();
@@ -3952,7 +4097,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                 int itemcount = 0;
                 for (int i = 0; i < items.count(); ++i) {
                     AbstractClipItem *item = static_cast<AbstractClipItem *>(items.at(i));
-                    if (item && item->type() == AVWIDGET) {
+                    if (item && item->type() == AVWidget) {
                         ItemInfo info = infos.at(itemcount);
                         prepareResizeClipStart(item, info, item->startPos().frames(m_document->fps()), false, resizeCommand);
                         ++itemcount;
@@ -3962,11 +4107,11 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
             }
         } else {
             prepareResizeClipStart(m_dragItem, m_dragItemInfo, m_dragItem->startPos().frames(m_document->fps()));
-           if (m_dragItem->type() == AVWIDGET) static_cast <ClipItem*>(m_dragItem)->slotUpdateRange();
+            if (m_dragItem->type() == AVWidget) static_cast <ClipItem*>(m_dragItem)->slotUpdateRange();
         }
-    } else if (m_operationMode == RESIZEEND && m_dragItem->endPos() != m_dragItemInfo.endPos) {
+    } else if (m_operationMode == ResizeEnd && m_dragItem->endPos() != m_dragItemInfo.endPos) {
         // resize end
-        if (!m_controlModifier && m_dragItem->type() == AVWIDGET && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
+        if (!m_controlModifier && m_dragItem->type() == AVWidget && m_dragItem->parentItem() && m_dragItem->parentItem() != m_selectionGroup) {
             AbstractGroupItem *parent = static_cast <AbstractGroupItem *>(m_dragItem->parentItem());
             if (parent) {
                 QUndoCommand *resizeCommand = new QUndoCommand();
@@ -3977,7 +4122,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                 int itemcount = 0;
                 for (int i = 0; i < items.count(); ++i) {
                     AbstractClipItem *item = static_cast<AbstractClipItem *>(items.at(i));
-                    if (item && item->type() == AVWIDGET) {
+                    if (item && item->type() == AVWidget) {
                         ItemInfo info = infos.at(itemcount);
                         prepareResizeClipEnd(item, info, item->endPos().frames(m_document->fps()), false, resizeCommand);
                         ++itemcount;
@@ -3988,9 +4133,9 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
             }
         } else {
             prepareResizeClipEnd(m_dragItem, m_dragItemInfo, m_dragItem->endPos().frames(m_document->fps()));
-           if (m_dragItem->type() == AVWIDGET) static_cast <ClipItem*>(m_dragItem)->slotUpdateRange();
+            if (m_dragItem->type() == AVWidget) static_cast <ClipItem*>(m_dragItem)->slotUpdateRange();
         }
-    } else if (m_operationMode == FADEIN) {
+    } else if (m_operationMode == FadeIn) {
         // resize fade in effect
         ClipItem * item = static_cast <ClipItem *>(m_dragItem);
         int ix = item->hasEffect("volume", "fadein");
@@ -4011,7 +4156,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
             }
         } else if (item->fadeIn() != 0 && ix2 == -1) {
             QDomElement effect;
-            if (item->isVideoOnly() || (item->clipType() != AUDIO && item->clipType() != AV && item->clipType() != PLAYLIST)) {
+            if (item->isVideoOnly() || (item->clipType() != Audio && item->clipType() != AV && item->clipType() != Playlist)) {
                 // add video fade
                 effect = MainWindow::videoEffects.getEffectByTag("", "fade_from_black").cloneNode().toElement();
             } else effect = MainWindow::audioEffects.getEffectByTag("volume", "fadein").cloneNode().toElement();
@@ -4033,7 +4178,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                 emit clipItemSelected(item);
             }
         }
-    } else if (m_operationMode == FADEOUT) {
+    } else if (m_operationMode == FadeOut) {
         // resize fade out effect
         ClipItem * item = static_cast <ClipItem *>(m_dragItem);
         int ix = item->hasEffect("volume", "fadeout");
@@ -4055,7 +4200,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
             }
         } else if (item->fadeOut() != 0 && ix2 == -1) {
             QDomElement effect;
-            if (item->isVideoOnly() || (item->clipType() != AUDIO && item->clipType() != AV && item->clipType() != PLAYLIST)) {
+            if (item->isVideoOnly() || (item->clipType() != Audio && item->clipType() != AV && item->clipType() != Playlist)) {
                 // add video fade
                 effect = MainWindow::videoEffects.getEffectByTag("", "fade_to_black").cloneNode().toElement();
             } else effect = MainWindow::audioEffects.getEffectByTag("volume", "fadeout").cloneNode().toElement();
@@ -4081,7 +4226,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
                 emit clipItemSelected(item);
             }
         }
-    } else if (m_operationMode == KEYFRAME) {
+    } else if (m_operationMode == KeyFrame) {
         // update the MLT effect
         ClipItem * item = static_cast <ClipItem *>(m_dragItem);
         QDomElement oldEffect = item->selectedEffect().cloneNode().toElement();
@@ -4111,8 +4256,8 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
         updateEffect(m_document->tracksCount() - item->track(), item->startPos(), item->selectedEffect());
         emit clipItemSelected(item);
     }
-    if (m_operationMode != NONE && m_operationMode != MOVE) setDocumentModified();
-    m_operationMode = NONE;
+    if (m_operationMode != None && m_operationMode != MoveOperation) setDocumentModified();
+    m_operationMode = None;
 }
 
 void CustomTrackView::deleteClip(ItemInfo info, bool refresh)
@@ -4175,22 +4320,23 @@ void CustomTrackView::deleteSelectedClips()
     }
     scene()->clearSelection();
     QUndoCommand *deleteSelected = new QUndoCommand();
+    new RefreshMonitorCommand(this, false, true, deleteSelected);
 
     int groupCount = 0;
     int clipCount = 0;
     int transitionCount = 0;
     // expand & destroy groups
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == GROUPWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == GroupWidget) {
             groupCount++;
             QList<QGraphicsItem *> children = itemList.at(i)->childItems();
             QList <ItemInfo> clipInfos;
             QList <ItemInfo> transitionInfos;
             for (int j = 0; j < children.count(); j++) {
-                if (children.at(j)->type() == AVWIDGET) {
+                if (children.at(j)->type() == AVWidget) {
                     AbstractClipItem *clip = static_cast <AbstractClipItem *>(children.at(j));
                     if (!clip->isItemLocked()) clipInfos.append(clip->info());
-                } else if (children.at(j)->type() == TRANSITIONWIDGET) {
+                } else if (children.at(j)->type() == TransitionWidget) {
                     AbstractClipItem *clip = static_cast <AbstractClipItem *>(children.at(j));
                     if (!clip->isItemLocked()) transitionInfos.append(clip->info());
                 }
@@ -4203,23 +4349,22 @@ void CustomTrackView::deleteSelectedClips()
             if (clipInfos.count() > 0)
                 new GroupClipsCommand(this, clipInfos, transitionInfos, false, deleteSelected);
 
-        } else if (itemList.at(i)->parentItem() && itemList.at(i)->parentItem()->type() == GROUPWIDGET)
+        } else if (itemList.at(i)->parentItem() && itemList.at(i)->parentItem()->type() == GroupWidget)
             itemList.insert(i + 1, itemList.at(i)->parentItem());
     }
-
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    emit clipItemSelected(NULL);
+    emit transitionItemSelected(NULL);
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             clipCount++;
             ClipItem *item = static_cast <ClipItem *>(itemList.at(i));
             //kDebug()<<"// DELETE CLP AT: "<<item->info().startPos.frames(25);
             new AddTimelineClipCommand(this, item->xml(), item->clipProducer(), item->info(), item->effectList(), false, false, true, true, deleteSelected);
-            emit clipItemSelected(NULL);
-        } else if (itemList.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (itemList.at(i)->type() == TransitionWidget) {
             transitionCount++;
             Transition *item = static_cast <Transition *>(itemList.at(i));
             //kDebug()<<"// DELETE TRANS AT: "<<item->info().startPos.frames(25);
             new AddTransitionCommand(this, item->info(), item->transitionEndTrack(), item->toXML(), true, true, deleteSelected);
-            emit transitionItemSelected(NULL);
         }
     }
     if (groupCount > 0 && clipCount == 0 && transitionCount == 0)
@@ -4230,13 +4375,12 @@ void CustomTrackView::deleteSelectedClips()
         deleteSelected->setText(i18np("Delete selected transition", "Delete selected transitions", transitionCount));
     else deleteSelected->setText(i18n("Delete selected items"));
     updateTrackDuration(-1, deleteSelected);
-    new RefreshMonitorCommand(this, false, deleteSelected);
+    new RefreshMonitorCommand(this, true, false, deleteSelected);
     m_commandStack->push(deleteSelected);
-    m_document->renderer()->doRefresh();
 }
 
 
-void CustomTrackView::doChangeClipSpeed(ItemInfo info, ItemInfo speedIndependantInfo, const double speed, const double oldspeed, int strobe, const QString &id)
+void CustomTrackView::doChangeClipSpeed(ItemInfo info, const ItemInfo &speedIndependantInfo, const double speed, const double oldspeed, int strobe, const QString &id)
 {
     Q_UNUSED(id)
     //DocClipBase *baseclip = m_document->clipManager()->getClipById(id);
@@ -4254,7 +4398,7 @@ void CustomTrackView::doChangeClipSpeed(ItemInfo info, ItemInfo speedIndependant
         item->updateRectGeometry();
         if (item->cropDuration().frames(m_document->fps()) != endPos)
             item->resizeEnd((int) info.startPos.frames(m_document->fps()) + endPos - 1);
-        updatePositionEffects(item, info);
+        updatePositionEffects(item, info, false);
         setDocumentModified();
     } else {
         emit displayMessage(i18n("Invalid clip"), ErrorMessage);
@@ -4269,17 +4413,17 @@ void CustomTrackView::cutSelectedClips()
     for (int i = 0; i < itemList.count(); ++i) {
         if (!itemList.at(i))
             continue;
-        if (itemList.at(i)->type() == AVWIDGET) {
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *item = static_cast <ClipItem *>(itemList.at(i));
             if (item->parentItem() && item->parentItem() != m_selectionGroup) {
                 AbstractGroupItem *group = static_cast <AbstractGroupItem *>(item->parentItem());
                 if (!groups.contains(group))
                     groups << group;
             } else if (currentPos > item->startPos() && currentPos < item->endPos()) {
-                RazorClipCommand *command = new RazorClipCommand(this, item->info(), currentPos);
+                RazorClipCommand *command = new RazorClipCommand(this, item->info(), item->effectList(), currentPos);
                 m_commandStack->push(command);
             }
-        } else if (itemList.at(i)->type() == GROUPWIDGET && itemList.at(i) != m_selectionGroup) {
+        } else if (itemList.at(i)->type() == GroupWidget && itemList.at(i) != m_selectionGroup) {
             AbstractGroupItem *group = static_cast<AbstractGroupItem *>(itemList.at(i));
             if (!groups.contains(group))
                 groups << group;
@@ -4294,105 +4438,86 @@ void CustomTrackView::razorGroup(AbstractGroupItem* group, GenTime cutPos)
 {
     if (group) {
         QList <QGraphicsItem *> children = group->childItems();
+        QUndoCommand *command = new QUndoCommand;
+        command->setText(i18n("Cut Group"));
+        groupClips(false, children, command);
         QList <ItemInfo> clips1, transitions1;
-        QList <ItemInfo> clipsCut, transitionsCut;
+        QList <ItemInfo> transitionsCut;
         QList <ItemInfo> clips2, transitions2;
+        QList <QGraphicsItem *> clipsToCut;
+
+        // Collect info
         for (int i = 0; i < children.count(); ++i) {
             children.at(i)->setSelected(false);
             AbstractClipItem *child = static_cast <AbstractClipItem *>(children.at(i));
-            if (child->type() == AVWIDGET) {
+            if (child->type() == AVWidget) {
                 if (cutPos > child->endPos())
                     clips1 << child->info();
                 else if (cutPos < child->startPos())
                     clips2 << child->info();
-                else
-                    clipsCut << child->info();
+                else {
+                    clipsToCut << child;
+                }
             } else {
                 if (cutPos > child->endPos())
                     transitions1 << child->info();
                 else if (cutPos < child->startPos())
                     transitions2 << child->info();
-                else
-                    transitionsCut << child->info();
+                else {
+                    //transitionsCut << child->info();
+                    // Transition cut not implemented, leave it in first group...
+                    transitions1 << child->info();
+                }
             }
         }
-        if (clipsCut.isEmpty() && transitionsCut.isEmpty() && ((clips1.isEmpty() && transitions1.isEmpty()) || (clips2.isEmpty() && transitions2.isEmpty())))
+        if (clipsToCut.isEmpty() && transitionsCut.isEmpty() && ((clips1.isEmpty() && transitions1.isEmpty()) || (clips2.isEmpty() && transitions2.isEmpty()))) {
+            delete command;
             return;
-        RazorGroupCommand *command = new RazorGroupCommand(this, clips1, transitions1, clipsCut, transitionsCut, clips2, transitions2, cutPos);
-        m_commandStack->push(command);
-    }
-}
-
-void CustomTrackView::slotRazorGroup(QList <ItemInfo> clips1, QList <ItemInfo> transitions1, QList <ItemInfo> clipsCut, QList <ItemInfo> transitionsCut, QList <ItemInfo> clips2, QList <ItemInfo> transitions2, GenTime cutPos, bool cut)
-{
-    if (cut) {
-        for (int i = 0; i < clipsCut.count(); ++i) {
-            ClipItem *clip = getClipItemAt(clipsCut.at(i).startPos.frames(m_document->fps()), clipsCut.at(i).track);
-            if (clip) {
-                ClipItem *clipBehind = cutClip(clipsCut.at(i), cutPos, true);
-                clips1 << clip->info();
-                if (clipBehind != NULL)
-                    clips2 << clipBehind->info();
-            }
-        }
-        /* TODO: cut transitionsCut
-         * For now just append them to group1 */
-        transitions1 << transitionsCut;
-        doGroupClips(clips1, transitions1, true);
-        doGroupClips(clips2, transitions2, true);
-    } else {
-        /* we might also just use clipsCut.at(0)->parentItem().
-         * Do this loop just in case something went wrong during cut */
-        for (int i = 0; i < clipsCut.count(); ++i) {
-            ClipItem *clip = getClipItemAt(cutPos.frames(m_document->fps()), clipsCut.at(i).track);
-            if (clip && clip->parentItem() && clip->parentItem()->type() == GROUPWIDGET) {
-                AbstractGroupItem *group = static_cast <AbstractGroupItem *>(clip->parentItem());
-                QList <QGraphicsItem *> children = group->childItems();
-                QList <ItemInfo> groupClips;
-                QList <ItemInfo> groupTrans;
-                for (int j = 0; j < children.count(); ++j) {
-                    if (children.at(j)->type() == AVWIDGET)
-                        groupClips << ((AbstractClipItem *)children.at(j))->info();
-                    else if (children.at(j)->type() == TRANSITIONWIDGET)
-                        groupTrans << ((AbstractClipItem *)children.at(j))->info();
-                }
-                doGroupClips(groupClips, groupTrans, false);
-                break;
-            }
         }
-        for (int i = 0; i < clipsCut.count(); ++i)
-            cutClip(clipsCut.at(i), cutPos, false);
-        // TODO: uncut transitonsCut
-        doGroupClips(QList <ItemInfo>() << clips1 << clipsCut << clips2, QList <ItemInfo>() << transitions1 << transitionsCut << transitions2, true);
+        // Process the cut
+        for (int i = 0; i < clipsToCut.count(); ++i) {
+            ClipItem *clip = static_cast<ClipItem *>(clipsToCut.at(i));
+            new RazorClipCommand(this, clip->info(), clip->effectList(), cutPos, false, command);
+            ClipItem *secondClip = cutClip(clip->info(), cutPos, true);
+            clips1 << clip->info();
+            clips2 << secondClip->info();
+        }
+        new GroupClipsCommand(this, clips1, transitions1, true, command);
+        new GroupClipsCommand(this, clips2, transitions2, true, command);
+        m_commandStack->push(command);
     }
 }
 
-void CustomTrackView::groupClips(bool group)
+void CustomTrackView::groupClips(bool group, QList<QGraphicsItem *> itemList, QUndoCommand *command)
 {
-    QList<QGraphicsItem *> itemList = scene()->selectedItems();
+    if (itemList.isEmpty()) itemList = scene()->selectedItems();
     QList <ItemInfo> clipInfos;
     QList <ItemInfo> transitionInfos;
 
     // Expand groups
     int max = itemList.count();
-    for (int i = 0; i < max; i++) {
-        if (itemList.at(i)->type() == GROUPWIDGET) {
+    for (int i = 0; i < max; ++i) {
+        if (itemList.at(i)->type() == GroupWidget) {
             itemList += itemList.at(i)->childItems();
         }
     }
 
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             AbstractClipItem *clip = static_cast <AbstractClipItem *>(itemList.at(i));
             if (!clip->isItemLocked()) clipInfos.append(clip->info());
-        } else if (itemList.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (itemList.at(i)->type() == TransitionWidget) {
             AbstractClipItem *clip = static_cast <AbstractClipItem *>(itemList.at(i));
             if (!clip->isItemLocked()) transitionInfos.append(clip->info());
         }
     }
     if (clipInfos.count() > 0) {
-        GroupClipsCommand *command = new GroupClipsCommand(this, clipInfos, transitionInfos, group);
-        m_commandStack->push(command);
+        if (command) {
+            new GroupClipsCommand(this, clipInfos, transitionInfos, group, command);
+        } else {
+            GroupClipsCommand *command = new GroupClipsCommand(this, clipInfos, transitionInfos, group);
+            m_commandStack->push(command);
+        }
     }
 }
 
@@ -4401,23 +4526,27 @@ void CustomTrackView::doGroupClips(QList <ItemInfo> clipInfos, QList <ItemInfo>
     resetSelectionGroup();
     m_scene->clearSelection();
     if (!group) {
-        for (int i = 0; i < clipInfos.count(); i++) {
+        // ungroup, find main group to destroy it...
+        for (int i = 0; i < clipInfos.count(); ++i) {
             ClipItem *clip = getClipItemAt(clipInfos.at(i).startPos, clipInfos.at(i).track);
             if (clip == NULL) continue;
-            if (clip->parentItem() && clip->parentItem()->type() == GROUPWIDGET) {
+            if (clip->parentItem() && clip->parentItem()->type() == GroupWidget) {
                 AbstractGroupItem *grp = static_cast <AbstractGroupItem *>(clip->parentItem());
                 m_document->clipManager()->removeGroup(grp);
+                if (grp == m_selectionGroup) m_selectionGroup = NULL;
                 scene()->destroyItemGroup(grp);
             }
             clip->setFlag(QGraphicsItem::ItemIsMovable, true);
         }
-        for (int i = 0; i < transitionInfos.count(); i++) {
+        for (int i = 0; i < transitionInfos.count(); ++i) {
             Transition *tr = getTransitionItemAt(transitionInfos.at(i).startPos, transitionInfos.at(i).track);
             if (tr == NULL) continue;
-            if (tr->parentItem() && tr->parentItem()->type() == GROUPWIDGET) {
+            if (tr->parentItem() && tr->parentItem()->type() == GroupWidget) {
                 AbstractGroupItem *grp = static_cast <AbstractGroupItem *>(tr->parentItem());
                 m_document->clipManager()->removeGroup(grp);
+                if (grp == m_selectionGroup) m_selectionGroup = NULL;
                 scene()->destroyItemGroup(grp);
+                grp = NULL;
             }
             tr->setFlag(QGraphicsItem::ItemIsMovable, true);
         }
@@ -4425,21 +4554,21 @@ void CustomTrackView::doGroupClips(QList <ItemInfo> clipInfos, QList <ItemInfo>
         return;
     }
     QList <QGraphicsItem *>list;
-    for (int i = 0; i < clipInfos.count(); i++) {
+    for (int i = 0; i < clipInfos.count(); ++i) {
         ClipItem *clip = getClipItemAt(clipInfos.at(i).startPos, clipInfos.at(i).track);
         if (clip) {
-           list.append(clip);
+            list.append(clip);
             //clip->setSelected(true);
         }
     }
-    for (int i = 0; i < transitionInfos.count(); i++) {
+    for (int i = 0; i < transitionInfos.count(); ++i) {
         Transition *clip = getTransitionItemAt(transitionInfos.at(i).startPos, transitionInfos.at(i).track);
         if (clip) {
-           list.append(clip);
+            list.append(clip);
             //clip->setSelected(true);
         }
     }
-    groupSelectedItems(list, false, true, true);
+    groupSelectedItems(list, true, true);
     setDocumentModified();
 }
 
@@ -4457,7 +4586,7 @@ void CustomTrackView::addClip(QDomElement xml, const QString &clipId, ItemInfo i
         emit displayMessage(i18n("Waiting for clip..."), InformationMessage);
         emit forceClipProcessing(clipId);
         qApp->processEvents();
-        for (int i = 0; i < 10; i++) {
+        for (int i = 0; i < 10; ++i) {
             if (baseclip->getProducer() == NULL) {
                 qApp->processEvents();
                 m_producerNotReady.wait(&m_mutex, 200);
@@ -4487,12 +4616,13 @@ void CustomTrackView::addClip(QDomElement xml, const QString &clipId, ItemInfo i
     m_document->updateClip(baseclip->getId());
     info.track = m_document->tracksCount() - info.track;
     m_document->renderer()->mltInsertClip(info, xml, item->getProducer(producerTrack), overwrite, push);
-    for (int i = 0; i < item->effectsCount(); i++) {
+    for (int i = 0; i < item->effectsCount(); ++i) {
         m_document->renderer()->mltAddEffect(info.track, info.startPos, getEffectArgs(item->effect(i)), false);
     }
     setDocumentModified();
-    if (refresh)
+    if (refresh) {
         m_document->renderer()->doRefresh();
+    }
     if (!baseclip->isPlaceHolder())
         m_waitingThumbs.append(item);
     m_thumbsTimer.start();
@@ -4507,7 +4637,7 @@ void CustomTrackView::slotUpdateClip(const QString &clipId, bool reload)
     DocClipBase *baseClip = NULL;
     Mlt::Tractor *tractor = m_document->renderer()->lockService();
     for (int i = 0; i < list.size(); ++i) {
-        if (list.at(i)->type() == AVWIDGET) {
+        if (list.at(i)->type() == AVWidget) {
             clip = static_cast <ClipItem *>(list.at(i));
             if (clip->clipProducer() == clipId) {
                 if (baseClip == NULL) {
@@ -4525,7 +4655,7 @@ void CustomTrackView::slotUpdateClip(const QString &clipId, bool reload)
             }
         }
     }
-    for (int i = 0; i < clipList.count(); i++)
+    for (int i = 0; i < clipList.count(); ++i)
         clipList.at(i)->refreshClip(true, true);
     m_document->renderer()->unlockService(tractor);
 }
@@ -4535,9 +4665,9 @@ ClipItem *CustomTrackView::getClipItemAtEnd(GenTime pos, int track)
     int framepos = (int)(pos.frames(m_document->fps()));
     QList<QGraphicsItem *> list = scene()->items(QPointF(framepos - 1, track * m_tracksHeight + m_tracksHeight / 2));
     ClipItem *clip = NULL;
-    for (int i = 0; i < list.size(); i++) {
+    for (int i = 0; i < list.size(); ++i) {
         if (!list.at(i)->isEnabled()) continue;
-        if (list.at(i)->type() == AVWIDGET) {
+        if (list.at(i)->type() == AVWidget) {
             ClipItem *test = static_cast <ClipItem *>(list.at(i));
             if (test->endPos() == pos) clip = test;
             break;
@@ -4550,9 +4680,9 @@ ClipItem *CustomTrackView::getClipItemAtStart(GenTime pos, int track)
 {
     QList<QGraphicsItem *> list = scene()->items(QPointF(pos.frames(m_document->fps()), track * m_tracksHeight + m_tracksHeight / 2));
     ClipItem *clip = NULL;
-    for (int i = 0; i < list.size(); i++) {
+    for (int i = 0; i < list.size(); ++i) {
         if (!list.at(i)->isEnabled()) continue;
-        if (list.at(i)->type() == AVWIDGET) {
+        if (list.at(i)->type() == AVWidget) {
             ClipItem *test = static_cast <ClipItem *>(list.at(i));
             if (test->startPos() == pos) clip = test;
             break;
@@ -4566,9 +4696,9 @@ ClipItem *CustomTrackView::getClipItemAt(int pos, int track)
     const QPointF p(pos, track * m_tracksHeight + m_tracksHeight / 2);
     QList<QGraphicsItem *> list = scene()->items(p);
     ClipItem *clip = NULL;
-    for (int i = 0; i < list.size(); i++) {
+    for (int i = 0; i < list.size(); ++i) {
         if (!list.at(i)->isEnabled()) continue;
-        if (list.at(i)->type() == AVWIDGET) {
+        if (list.at(i)->type() == AVWidget) {
             clip = static_cast <ClipItem *>(list.at(i));
             break;
         }
@@ -4594,9 +4724,9 @@ Transition *CustomTrackView::getTransitionItemAt(int pos, int track)
     const QPointF p(pos, track * m_tracksHeight + Transition::itemOffset() + 1);
     QList<QGraphicsItem *> list = scene()->items(p);
     Transition *clip = NULL;
-    for (int i = 0; i < list.size(); i++) {
+    for (int i = 0; i < list.size(); ++i) {
         if (!list.at(i)->isEnabled()) continue;
-        if (list.at(i)->type() == TRANSITIONWIDGET) {
+        if (list.at(i)->type() == TransitionWidget) {
             clip = static_cast <Transition *>(list.at(i));
             break;
         }
@@ -4615,9 +4745,9 @@ Transition *CustomTrackView::getTransitionItemAtEnd(GenTime pos, int track)
     const QPointF p(framepos - 1, track * m_tracksHeight + Transition::itemOffset() + 1);
     QList<QGraphicsItem *> list = scene()->items(p);
     Transition *clip = NULL;
-    for (int i = 0; i < list.size(); i++) {
+    for (int i = 0; i < list.size(); ++i) {
         if (!list.at(i)->isEnabled()) continue;
-        if (list.at(i)->type() == TRANSITIONWIDGET) {
+        if (list.at(i)->type() == TransitionWidget) {
             Transition *test = static_cast <Transition *>(list.at(i));
             if (test->endPos() == pos) clip = test;
             break;
@@ -4633,7 +4763,7 @@ Transition *CustomTrackView::getTransitionItemAtStart(GenTime pos, int track)
     Transition *clip = NULL;
     for (int i = 0; i < list.size(); ++i) {
         if (!list.at(i)->isEnabled()) continue;
-        if (list.at(i)->type() == TRANSITIONWIDGET) {
+        if (list.at(i)->type() == TransitionWidget) {
             Transition *test = static_cast <Transition *>(list.at(i));
             if (test->startPos() == pos) clip = test;
             break;
@@ -4703,17 +4833,17 @@ bool CustomTrackView::moveClip(const ItemInfo &start, const ItemInfo &end, bool
     return success;
 }
 
-void CustomTrackView::moveGroup(QList <ItemInfo> startClip, QList <ItemInfo> startTransition, const GenTime &offset, const int trackOffset, bool reverseMove)
+void CustomTrackView::moveGroup(QList<ItemInfo> startClip, QList<ItemInfo> startTransition, const GenTime &offset, const int trackOffset, bool reverseMove)
 {
     // Group Items
     resetSelectionGroup();
     m_scene->clearSelection();
-
+    m_selectionMutex.lock();
     m_selectionGroup = new AbstractGroupItem(m_document->fps());
     scene()->addItem(m_selectionGroup);
 
     m_document->renderer()->blockSignals(true);
-    for (int i = 0; i < startClip.count(); i++) {
+    for (int i = 0; i < startClip.count(); ++i) {
         if (reverseMove) {
             startClip[i].startPos = startClip.at(i).startPos - offset;
             startClip[i].track = startClip.at(i).track - trackOffset;
@@ -4729,7 +4859,7 @@ void CustomTrackView::moveGroup(QList <ItemInfo> startClip, QList <ItemInfo> sta
             m_document->renderer()->mltRemoveClip(m_document->tracksCount() - startClip.at(i).track, startClip.at(i).startPos);
         } else kDebug() << "//MISSING CLIP AT: " << startClip.at(i).startPos.frames(25);
     }
-    for (int i = 0; i < startTransition.count(); i++) {
+    for (int i = 0; i < startTransition.count(); ++i) {
         if (reverseMove) {
             startTransition[i].startPos = startTransition.at(i).startPos - offset;
             startTransition[i].track = startTransition.at(i).track - trackOffset;
@@ -4755,26 +4885,26 @@ void CustomTrackView::moveGroup(QList <ItemInfo> startClip, QList <ItemInfo> sta
         //m_selectionGroup->moveBy(offset.frames(m_document->fps()), trackOffset *(qreal) m_tracksHeight);
 
         QList<QGraphicsItem *> children = m_selectionGroup->childItems();
-       QList <AbstractGroupItem*> groupList;
+        QList <AbstractGroupItem*> groupList;
         // Expand groups
         int max = children.count();
-        for (int i = 0; i < max; i++) {
-            if (children.at(i)->type() == GROUPWIDGET) {
+        for (int i = 0; i < max; ++i) {
+            if (children.at(i)->type() == GroupWidget) {
                 children += children.at(i)->childItems();
                 //AbstractGroupItem *grp = static_cast<AbstractGroupItem *>(children.at(i));
                 //grp->moveBy(offset.frames(m_document->fps()), trackOffset *(qreal) m_tracksHeight);
                 /*m_document->clipManager()->removeGroup(grp);
                 m_scene->destroyItemGroup(grp);*/
-               AbstractGroupItem *group = (AbstractGroupItem*) children.at(i);
-               if (!groupList.contains(group)) groupList.append(group);
+                AbstractGroupItem *group = (AbstractGroupItem*) children.at(i);
+                if (!groupList.contains(group)) groupList.append(group);
                 children.removeAll(children.at(i));
-                i--;
+                --i;
             }
         }
 
-        for (int i = 0; i < children.count(); i++) {
+        for (int i = 0; i < children.count(); ++i) {
             // re-add items in correct place
-            if (children.at(i)->type() != AVWIDGET && children.at(i)->type() != TRANSITIONWIDGET) continue;
+            if (children.at(i)->type() != AVWidget && children.at(i)->type() != TransitionWidget) continue;
             AbstractClipItem *item = static_cast <AbstractClipItem *>(children.at(i));
             item->updateItem();
             ItemInfo info = item->info();
@@ -4785,15 +4915,15 @@ void CustomTrackView::moveGroup(QList <ItemInfo> startClip, QList <ItemInfo> sta
             else if (item->isItemLocked())
                 item->setItemLocked(false);
 
-            if (item->type() == AVWIDGET) {
+            if (item->type() == AVWidget) {
                 ClipItem *clip = static_cast <ClipItem*>(item);
                 int trackProducer = info.track;
                 info.track = m_document->tracksCount() - info.track;
                 m_document->renderer()->mltInsertClip(info, clip->xml(), clip->getProducer(trackProducer));
-                for (int i = 0; i < clip->effectsCount(); i++) {
+                for (int i = 0; i < clip->effectsCount(); ++i) {
                     m_document->renderer()->mltAddEffect(info.track, info.startPos, getEffectArgs(clip->effect(i)), false);
                 }
-            } else if (item->type() == TRANSITIONWIDGET) {
+            } else if (item->type() == TransitionWidget) {
                 Transition *tr = static_cast <Transition*>(item);
                 int newTrack;
                 if (!tr->forcedTrack()) {
@@ -4806,11 +4936,11 @@ void CustomTrackView::moveGroup(QList <ItemInfo> startClip, QList <ItemInfo> sta
                 m_document->renderer()->mltAddTransition(tr->transitionTag(), newTrack, m_document->tracksCount() - info.track, info.startPos, info.endPos, tr->toXML());
             }
         }
-
+        m_selectionMutex.unlock();
         resetSelectionGroup(false);
-       for (int i = 0; i < groupList.count(); i++) {
-           rebuildGroup(groupList.at(i));
-       }
+        for (int i = 0; i < groupList.count(); ++i) {
+            rebuildGroup(groupList.at(i));
+        }
 
         clearSelection();
 
@@ -4891,7 +5021,7 @@ void CustomTrackView::resizeClip(const ItemInfo &start, const ItemInfo &end, boo
         bool success = m_document->renderer()->mltResizeClipStart(clipinfo, end.startPos - clipinfo.startPos);
         if (success) {
             item->resizeStart((int) end.startPos.frames(m_document->fps()));
-       }
+        }
         else
             emit displayMessage(i18n("Error when resizing clip"), ErrorMessage);
     } else {
@@ -4948,7 +5078,7 @@ void CustomTrackView::prepareResizeClipStart(AbstractClipItem* item, ItemInfo ol
         new RebuildGroupCommand(this, item->info().track, item->endPos() - GenTime(1, m_document->fps()), command);
 
     ItemInfo info = item->info();
-    if (item->type() == AVWIDGET) {
+    if (item->type() == AVWidget) {
         ItemInfo resizeinfo = oldInfo;
         resizeinfo.track = m_document->tracksCount() - resizeinfo.track;
         bool success = m_document->renderer()->mltResizeClipStart(resizeinfo, item->startPos() - oldInfo.startPos);
@@ -4988,7 +5118,7 @@ void CustomTrackView::prepareResizeClipStart(AbstractClipItem* item, ItemInfo ol
             KdenliveSettings::setSnaptopoints(snap);
             emit displayMessage(i18n("Error when resizing clip"), ErrorMessage);
         }
-    } else if (item->type() == TRANSITIONWIDGET) {
+    } else if (item->type() == TransitionWidget) {
         Transition *transition = static_cast <Transition *>(item);
         if (!m_document->renderer()->mltMoveTransition(transition->transitionTag(), (int)(m_document->tracksCount() - oldInfo.track), (int)(m_document->tracksCount() - oldInfo.track), transition->transitionEndTrack(), oldInfo.startPos, oldInfo.endPos, info.startPos, info.endPos)) {
             // Cannot resize transition
@@ -5039,7 +5169,7 @@ void CustomTrackView::prepareResizeClipEnd(AbstractClipItem* item, ItemInfo oldI
         new RebuildGroupCommand(this, item->info().track, item->startPos(), command);
 
     ItemInfo info = item->info();
-    if (item->type() == AVWIDGET) {
+    if (item->type() == AVWidget) {
         if (!hasParentCommand) command->setText(i18n("Resize clip end"));
         ItemInfo resizeinfo = info;
         resizeinfo.track = m_document->tracksCount() - resizeinfo.track;
@@ -5082,7 +5212,7 @@ void CustomTrackView::prepareResizeClipEnd(AbstractClipItem* item, ItemInfo oldI
             KdenliveSettings::setSnaptopoints(true);
             emit displayMessage(i18n("Error when resizing clip"), ErrorMessage);
         }
-    } else if (item->type() == TRANSITIONWIDGET) {
+    } else if (item->type() == TransitionWidget) {
         if (!hasParentCommand) command->setText(i18n("Resize transition end"));
         Transition *transition = static_cast <Transition *>(item);
         if (!m_document->renderer()->mltMoveTransition(transition->transitionTag(), (int)(m_document->tracksCount() - oldInfo.track), (int)(m_document->tracksCount() - oldInfo.track), transition->transitionEndTrack(), oldInfo.startPos, oldInfo.endPos, info.startPos, info.endPos)) {
@@ -5094,11 +5224,22 @@ void CustomTrackView::prepareResizeClipEnd(AbstractClipItem* item, ItemInfo oldI
         } else {
             // Check transition keyframes
             QDomElement old = transition->toXML();
-            if (transition->updateKeyframes()) {
+            if (transition->updateKeyframes(oldInfo.cropDuration.frames(m_document->fps()) - 1)) {
                 QDomElement xml = transition->toXML();
                 m_document->renderer()->mltUpdateTransition(xml.attribute("tag"), xml.attribute("tag"), xml.attribute("transition_btrack").toInt(), m_document->tracksCount() - xml.attribute("transition_atrack").toInt(), transition->startPos(), transition->endPos(), xml);
                 new EditTransitionCommand(this, transition->track(), transition->startPos(), old, xml, false, command);
             }
+            ItemInfo info = transition->info();
+            QPoint p;
+            ClipItem *transitionClip = getClipItemAt(info.startPos, info.track);
+            if (transitionClip && transitionClip->baseClip()) {
+                QString size = transitionClip->baseClip()->getProperty("frame_size");
+                double factor = transitionClip->baseClip()->getProperty("aspect_ratio").toDouble();
+                if (factor == 0) factor = 1.0;
+                p.setX((int)(size.section('x', 0, 0).toInt() * factor + 0.5));
+                p.setY(size.section('x', 1, 1).toInt());
+            }
+            emit transitionItemSelected(transition, getPreviousVideoTrack(info.track), p, true);
             new MoveTransitionCommand(this, oldInfo, info, false, command);
         }
     }
@@ -5111,7 +5252,7 @@ void CustomTrackView::prepareResizeClipEnd(AbstractClipItem* item, ItemInfo oldI
     }
 }
 
-void CustomTrackView::updatePositionEffects(ClipItem* item, ItemInfo info, bool standalone)
+void CustomTrackView::updatePositionEffects(ClipItem* item, const ItemInfo &info, bool standalone)
 {
     int end = item->fadeIn();
     if (end != 0) {
@@ -5167,7 +5308,7 @@ void CustomTrackView::updatePositionEffects(ClipItem* item, ItemInfo info, bool
         if (effectPos != -1) {
             QDomElement effect = item->getEffectAtIndex(effectPos);
             int max = item->cropDuration().frames(m_document->fps());
-            int end = max + item->cropStart().frames(m_document->fps());
+            int end = max + item->cropStart().frames(m_document->fps()) - 1;
             if (start > max) {
                 // Make sure the fade effect is not longer than the clip
                 item->setFadeOut(max);
@@ -5188,7 +5329,7 @@ void CustomTrackView::updatePositionEffects(ClipItem* item, ItemInfo info, bool
         if (effectPos != -1) {
             QDomElement effect = item->getEffectAtIndex(effectPos);
             int max = item->cropDuration().frames(m_document->fps());
-            int end = max + item->cropStart().frames(m_document->fps());
+            int end = max + item->cropStart().frames(m_document->fps()) - 1;
             if (start > max) {
                 // Make sure the fade effect is not longer than the clip
                 item->setFadeOut(max);
@@ -5234,10 +5375,10 @@ void CustomTrackView::updateSnapPoints(AbstractClipItem *selected, QList <GenTim
     QList <GenTime> snaps;
     if (selected && offsetList.isEmpty()) offsetList.append(selected->cropDuration());
     QList<QGraphicsItem *> itemList = items();
-    for (int i = 0; i < itemList.count(); i++) {
+    for (int i = 0; i < itemList.count(); ++i) {
         if (itemList.at(i) == selected) continue;
         if (skipSelectedItems && itemList.at(i)->isSelected()) continue;
-        if (itemList.at(i)->type() == AVWIDGET) {
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *item = static_cast <ClipItem *>(itemList.at(i));
             GenTime start = item->startPos();
             GenTime end = item->endPos();
@@ -5260,7 +5401,7 @@ void CustomTrackView::updateSnapPoints(AbstractClipItem *selected, QList <GenTim
                     }
                 }
             }
-        } else if (itemList.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (itemList.at(i)->type() == TransitionWidget) {
             Transition *transition = static_cast <Transition*>(itemList.at(i));
             GenTime start = transition->startPos();
             GenTime end = transition->endPos();
@@ -5285,7 +5426,7 @@ void CustomTrackView::updateSnapPoints(AbstractClipItem *selected, QList <GenTim
     }
 
     // add guides
-    for (int i = 0; i < m_guides.count(); i++) {
+    for (int i = 0; i < m_guides.count(); ++i) {
         snaps.append(m_guides.at(i)->position());
         if (!offsetList.isEmpty()) {
             for (int j = 0; j < offsetList.size(); j++) {
@@ -5351,27 +5492,29 @@ void CustomTrackView::slotAddClipExtraData(const QString &id, const QString &key
 
 void CustomTrackView::slotAddClipMarker(const QString &id, QList <CommentedTime> newMarkers, QUndoCommand *groupCommand)
 {
+    DocClipBase *base = m_document->clipManager()->getClipById(id);
+    if (!base) return;
     QUndoCommand *subCommand = NULL;
     if (newMarkers.count() > 1 && groupCommand == NULL) {
-       subCommand = new QUndoCommand;
-       subCommand->setText("Add markers");
-    }
-    for (int i = 0; i < newMarkers.count(); i++) {
-       CommentedTime oldMarker = m_document->clipManager()->getClipById(id)->markerAt(newMarkers.at(i).time());
-       if (oldMarker == CommentedTime()) {
-           oldMarker = newMarkers.at(i);
-           oldMarker.setMarkerType(-1);
-       }
-       if (newMarkers.count() == 1 && !groupCommand) {
-           AddMarkerCommand *command = new AddMarkerCommand(this, oldMarker, newMarkers.at(i), id, groupCommand);
-           m_commandStack->push(command);
-       }
-       else if (groupCommand) {
-           (void) new AddMarkerCommand(this, oldMarker, newMarkers.at(i), id, groupCommand);
-       }
-       else {
-           (void) new AddMarkerCommand(this, oldMarker, newMarkers.at(i), id, subCommand);
-       }
+        subCommand = new QUndoCommand;
+        subCommand->setText("Add markers");
+    }
+    for (int i = 0; i < newMarkers.count(); ++i) {
+        CommentedTime oldMarker = base->markerAt(newMarkers.at(i).time());
+        if (oldMarker == CommentedTime()) {
+            oldMarker = newMarkers.at(i);
+            oldMarker.setMarkerType(-1);
+        }
+        if (newMarkers.count() == 1 && !groupCommand) {
+            AddMarkerCommand *command = new AddMarkerCommand(this, oldMarker, newMarkers.at(i), id, groupCommand);
+            m_commandStack->push(command);
+        }
+        else if (groupCommand) {
+            (void) new AddMarkerCommand(this, oldMarker, newMarkers.at(i), id, groupCommand);
+        }
+        else {
+            (void) new AddMarkerCommand(this, oldMarker, newMarkers.at(i), id, subCommand);
+        }
     }
     if (subCommand) m_commandStack->push(subCommand);
 }
@@ -5388,6 +5531,7 @@ void CustomTrackView::slotDeleteClipMarker(const QString &comment, const QString
 void CustomTrackView::slotDeleteAllClipMarkers(const QString &id)
 {
     DocClipBase *base = m_document->clipManager()->getClipById(id);
+    if (!base) return;
     QList <CommentedTime> markers = base->commentedSnapMarkers();
 
     if (markers.isEmpty()) {
@@ -5397,10 +5541,10 @@ void CustomTrackView::slotDeleteAllClipMarkers(const QString &id)
     QUndoCommand *deleteMarkers = new QUndoCommand();
     deleteMarkers->setText("Delete clip markers");
 
-    for (int i = 0; i < markers.size(); i++) {
-       CommentedTime oldMarker = markers.at(i);
-       CommentedTime marker = oldMarker;
-       marker.setMarkerType(-1);
+    for (int i = 0; i < markers.size(); ++i) {
+        CommentedTime oldMarker = markers.at(i);
+        CommentedTime marker = oldMarker;
+        marker.setMarkerType(-1);
         new AddMarkerCommand(this, oldMarker, marker, id, deleteMarkers);
     }
     m_commandStack->push(deleteMarkers);
@@ -5409,47 +5553,48 @@ void CustomTrackView::slotDeleteAllClipMarkers(const QString &id)
 void CustomTrackView::slotSaveClipMarkers(const QString &id)
 {
     DocClipBase *base = m_document->clipManager()->getClipById(id);
+    if (!base) return;
     QList < CommentedTime > markers = base->commentedSnapMarkers();
     if (!markers.isEmpty()) {
-       // Set  up categories
-       QComboBox *cbox = new QComboBox;
-       cbox->insertItem(0, i18n("All categories"));
-       for (int i = 0; i < 5; ++i) {
-           cbox->insertItem(i + 1, i18n("Category %1", i));
-           cbox->setItemData(i + 1, CommentedTime::markerColor(i), Qt::DecorationRole);
-       }
-       cbox->setCurrentIndex(0);
-       KFileDialog fd(KUrl("kfiledialog:///projectfolder"), "text/plain", this, cbox);
-       fd.setMode(KFile::File);
-       fd.setOperationMode(KFileDialog::Saving);
-       fd.exec();
-       QString url = fd.selectedFile();
-       //QString url = KFileDialog::getSaveFileName(KUrl("kfiledialog:///projectfolder"), "text/plain", this, i18n("Save markers"));
-       if (url.isEmpty()) return;
-
-       QString data;
-       int category = cbox->currentIndex() - 1;
-       for (int i = 0; i < markers.count(); i++) {
-           if (category >= 0) {
-               // Save only the markers in selected category
-               if (markers.at(i).markerType() != category) continue;
-           }
-           data.append(QString::number(markers.at(i).time().seconds()));
-           data.append("\t");
-           data.append(QString::number(markers.at(i).time().seconds()));
-           data.append("\t");
-           data.append(markers.at(i).comment());
-           data.append("\n");
-       }
-       delete cbox;
-       
-       QFile file(url);
-       if (!file.open(QIODevice::WriteOnly | QIODevice::Text)) {
-           emit displayMessage(i18n("Cannot open file %1", url), ErrorMessage);
-           return;
-       }
-       file.write(data.toUtf8());
-       file.close();
+        // Set  up categories
+        QComboBox *cbox = new QComboBox;
+        cbox->insertItem(0, i18n("All categories"));
+        for (int i = 0; i < 5; ++i) {
+            cbox->insertItem(i + 1, i18n("Category %1", i));
+            cbox->setItemData(i + 1, CommentedTime::markerColor(i), Qt::DecorationRole);
+        }
+        cbox->setCurrentIndex(0);
+        KFileDialog fd(KUrl("kfiledialog:///projectfolder"), "text/plain", this, cbox);
+        fd.setMode(KFile::File);
+        fd.setOperationMode(KFileDialog::Saving);
+        if (fd.exec() != QDialog::Accepted) return;
+        QString url = fd.selectedFile();
+        //QString url = KFileDialog::getSaveFileName(KUrl("kfiledialog:///projectfolder"), "text/plain", this, i18n("Save markers"));
+        if (url.isEmpty()) return;
+
+        QString data;
+        int category = cbox->currentIndex() - 1;
+        for (int i = 0; i < markers.count(); ++i) {
+            if (category >= 0) {
+                // Save only the markers in selected category
+                if (markers.at(i).markerType() != category) continue;
+            }
+            data.append(QString::number(markers.at(i).time().seconds()));
+            data.append("\t");
+            data.append(QString::number(markers.at(i).time().seconds()));
+            data.append("\t");
+            data.append(markers.at(i).comment());
+            data.append("\n");
+        }
+        delete cbox;
+
+        QFile file(url);
+        if (!file.open(QIODevice::WriteOnly | QIODevice::Text)) {
+            emit displayMessage(i18n("Cannot open file %1", url), ErrorMessage);
+            return;
+        }
+        file.write(data.toUtf8());
+        file.close();
     }
 }
 
@@ -5457,24 +5602,24 @@ void CustomTrackView::slotLoadClipMarkers(const QString &id)
 {
     QComboBox *cbox = new QComboBox;
     for (int i = 0; i < 5; ++i) {
-       cbox->insertItem(i, i18n("Category %1", i));
-       cbox->setItemData(i, CommentedTime::markerColor(i), Qt::DecorationRole);
+        cbox->insertItem(i, i18n("Category %1", i));
+        cbox->setItemData(i, CommentedTime::markerColor(i), Qt::DecorationRole);
     }
     cbox->setCurrentIndex(KdenliveSettings::default_marker_type());
     KFileDialog fd(KUrl("kfiledialog:///projectfolder"), "text/plain", this, cbox);
     fd.setMode(KFile::File);
     fd.setOperationMode(KFileDialog::Opening);
-    fd.exec();
+    if (fd.exec() != QDialog::Accepted) return;
     QString url = fd.selectedFile();
-       
+
     //KUrl url = KFileDialog::getOpenUrl(KUrl("kfiledialog:///projectfolder"), "text/plain", this, i18n("Load marker file"));
     if (url.isEmpty()) return;
     int category = cbox->currentIndex();
     delete cbox;
     QFile file(url);
     if (!file.open(QIODevice::ReadOnly | QIODevice::Text)) {
-       emit displayMessage(i18n("Cannot open file %1", KUrl(url).fileName()), ErrorMessage);
-       return;
+        emit displayMessage(i18n("Cannot open file %1", KUrl(url).fileName()), ErrorMessage);
+        return;
     }
     QString data = QString::fromUtf8(file.readAll());
     file.close();
@@ -5486,48 +5631,48 @@ void CustomTrackView::slotLoadClipMarkers(const QString &id)
     QString markerText;
     QList <CommentedTime> markersList;
     foreach(QString line, lines) {
-       markerText.clear();
-       values = line.split("\t", QString::SkipEmptyParts);
-       double time1 = values.at(0).toDouble(&ok);
-       double time2 = -1;
-       if (!ok) continue;
-       if (values.count() >1) {
-           time2 = values.at(1).toDouble(&ok);
-           if (values.count() == 2) {
-               // Check if second value is a number or text
-               if (!ok) {
-                   time2 = -1;
-                   markerText = values.at(1);
-               }
-               else markerText = i18n("Marker");
-           }
-           else {
-               // We assume 3 values per line: in out name
-               if (!ok) {
-                   // 2nd value is not a number, drop
-               }
-               else {
-                   markerText = values.at(2);
-               }
-           }
-       }
-       if (!markerText.isEmpty()) {
-           // Marker found, add it
-           //TODO: allow user to set a marker category
-           CommentedTime marker1(GenTime(time1), markerText, category);
-           markersList << marker1;
-           if (time2 > 0 && time2 != time1) {
-               CommentedTime marker2(GenTime(time2), markerText, category);
-               markersList << marker2;
-           }
-       }
+        markerText.clear();
+        values = line.split("\t", QString::SkipEmptyParts);
+        double time1 = values.at(0).toDouble(&ok);
+        double time2 = -1;
+        if (!ok) continue;
+        if (values.count() >1) {
+            time2 = values.at(1).toDouble(&ok);
+            if (values.count() == 2) {
+                // Check if second value is a number or text
+                if (!ok) {
+                    time2 = -1;
+                    markerText = values.at(1);
+                }
+                else markerText = i18n("Marker");
+            }
+            else {
+                // We assume 3 values per line: in out name
+                if (!ok) {
+                    // 2nd value is not a number, drop
+                }
+                else {
+                    markerText = values.at(2);
+                }
+            }
+        }
+        if (!markerText.isEmpty()) {
+            // Marker found, add it
+            //TODO: allow user to set a marker category
+            CommentedTime marker1(GenTime(time1), markerText, category);
+            markersList << marker1;
+            if (time2 > 0 && time2 != time1) {
+                CommentedTime marker2(GenTime(time2), markerText, category);
+                markersList << marker2;
+            }
+        }
     }
     if (!markersList.isEmpty()) slotAddClipMarker(id, markersList, command);
     if (command->childCount() > 0) m_commandStack->push(command);
     else delete command;
 }
 
-void CustomTrackView::addMarker(const QString &id, const CommentedTime marker)
+void CustomTrackView::addMarker(const QString &id, const CommentedTime &marker)
 {
     DocClipBase *base = m_document->clipManager()->getClipById(id);
     if (base == NULL) return;
@@ -5550,7 +5695,7 @@ void CustomTrackView::addData(const QString &id, const QString &key, const QStri
 
 int CustomTrackView::hasGuide(int pos, int offset)
 {
-    for (int i = 0; i < m_guides.count(); i++) {
+    for (int i = 0; i < m_guides.count(); ++i) {
         int guidePos = m_guides.at(i)->position().frames(m_document->fps());
         if (qAbs(guidePos - pos) <= offset) return guidePos;
         else if (guidePos > pos) return -1;
@@ -5563,7 +5708,7 @@ void CustomTrackView::buildGuidesMenu(QMenu *goMenu) const
     QAction *act;
     goMenu->clear();
     double fps = m_document->fps();
-    for (int i = 0; i < m_guides.count(); i++) {
+    for (int i = 0; i < m_guides.count(); ++i) {
         act = goMenu->addAction(m_guides.at(i)->label() + '/' + Timecode::getStringTimecode(m_guides.at(i)->position().frames(fps), fps));
         act->setData(m_guides.at(i)->position().frames(m_document->fps()));
     }
@@ -5572,20 +5717,10 @@ void CustomTrackView::buildGuidesMenu(QMenu *goMenu) const
 
 void CustomTrackView::editGuide(const GenTime &oldPos, const GenTime &pos, const QString &comment)
 {
-    if (oldPos > GenTime() && pos > GenTime()) {
-        // move guide
-        for (int i = 0; i < m_guides.count(); i++) {
-            if (m_guides.at(i)->position() == oldPos) {
-                Guide *item = m_guides.at(i);
-                item->updateGuide(pos, comment);
-                break;
-            }
-        }
-    } else if (pos > GenTime()) addGuide(pos, comment);
-    else {
-        // remove guide
+    if (comment.isEmpty() && pos < GenTime()) {
+        // Delete guide
         bool found = false;
-        for (int i = 0; i < m_guides.count(); i++) {
+        for (int i = 0; i < m_guides.count(); ++i) {
             if (m_guides.at(i)->position() == oldPos) {
                 delete m_guides.takeAt(i);
                 found = true;
@@ -5594,13 +5729,24 @@ void CustomTrackView::editGuide(const GenTime &oldPos, const GenTime &pos, const
         }
         if (!found) emit displayMessage(i18n("No guide at cursor time"), ErrorMessage);
     }
+    
+    else if (oldPos >= GenTime()) {
+        // move guide
+        for (int i = 0; i < m_guides.count(); ++i) {
+            if (m_guides.at(i)->position() == oldPos) {
+                Guide *item = m_guides.at(i);
+                item->updateGuide(pos, comment);
+                break;
+            }
+        }
+    } else addGuide(pos, comment);
     qSort(m_guides.begin(), m_guides.end(), sortGuidesList);
     m_document->syncGuides(m_guides);
 }
 
 bool CustomTrackView::addGuide(const GenTime &pos, const QString &comment)
 {
-    for (int i = 0; i < m_guides.count(); i++) {
+    for (int i = 0; i < m_guides.count(); ++i) {
         if (m_guides.at(i)->position() == pos) {
             emit displayMessage(i18n("A guide already exists at position %1", m_document->timecode().getTimecodeFromFrames(pos.frames(m_document->fps()))), ErrorMessage);
             return false;
@@ -5619,7 +5765,7 @@ void CustomTrackView::slotAddGuide(bool dialog)
     CommentedTime marker(GenTime(m_cursorPos, m_document->fps()), i18n("Guide"));
     if (dialog) {
         QPointer<MarkerDialog> d = new MarkerDialog(NULL, marker,
-                             m_document->timecode(), i18n("Add Guide"), this);
+                                                    m_document->timecode(), i18n("Add Guide"), this);
         if (d->exec() != QDialog::Accepted) {
             delete d;
             return;
@@ -5641,7 +5787,7 @@ void CustomTrackView::slotEditGuide(int guidePos)
     if (guidePos == -1) pos = GenTime(m_cursorPos, m_document->fps());
     else pos = GenTime(guidePos, m_document->fps());
     bool found = false;
-    for (int i = 0; i < m_guides.count(); i++) {
+    for (int i = 0; i < m_guides.count(); ++i) {
         if (m_guides.at(i)->position() == pos) {
             slotEditGuide(m_guides.at(i)->info());
             found = true;
@@ -5651,7 +5797,7 @@ void CustomTrackView::slotEditGuide(int guidePos)
     if (!found) emit displayMessage(i18n("No guide at cursor time"), ErrorMessage);
 }
 
-void CustomTrackView::slotEditGuide(CommentedTime guide)
+void CustomTrackView::slotEditGuide(const CommentedTime &guide)
 {
     QPointer<MarkerDialog> d = new MarkerDialog(NULL, guide, m_document->timecode(), i18n("Edit Guide"), this);
     if (d->exec() == QDialog::Accepted) {
@@ -5667,7 +5813,7 @@ void CustomTrackView::slotEditTimeLineGuide()
     if (m_dragGuide == NULL) return;
     CommentedTime guide = m_dragGuide->info();
     QPointer<MarkerDialog> d = new MarkerDialog(NULL, guide,
-                     m_document->timecode(), i18n("Edit Guide"), this);
+                                                m_document->timecode(), i18n("Edit Guide"), this);
     if (d->exec() == QDialog::Accepted) {
         EditGuideCommand *command = new EditGuideCommand(this, guide.time(), guide.comment(), d->newMarker().time(), d->newMarker().comment(), true);
         m_commandStack->push(command);
@@ -5681,9 +5827,9 @@ void CustomTrackView::slotDeleteGuide(int guidePos)
     if (guidePos == -1) pos = GenTime(m_cursorPos, m_document->fps());
     else pos = GenTime(guidePos, m_document->fps());
     bool found = false;
-    for (int i = 0; i < m_guides.count(); i++) {
+    for (int i = 0; i < m_guides.count(); ++i) {
         if (m_guides.at(i)->position() == pos) {
-            EditGuideCommand *command = new EditGuideCommand(this, m_guides.at(i)->position(), m_guides.at(i)->label(), GenTime(), QString(), true);
+            EditGuideCommand *command = new EditGuideCommand(this, m_guides.at(i)->position(), m_guides.at(i)->label(), GenTime(-1), QString(), true);
             m_commandStack->push(command);
             found = true;
             break;
@@ -5696,7 +5842,7 @@ void CustomTrackView::slotDeleteGuide(int guidePos)
 void CustomTrackView::slotDeleteTimeLineGuide()
 {
     if (m_dragGuide == NULL) return;
-    EditGuideCommand *command = new EditGuideCommand(this, m_dragGuide->position(), m_dragGuide->label(), GenTime(), QString(), true);
+    EditGuideCommand *command = new EditGuideCommand(this, m_dragGuide->position(), m_dragGuide->label(), GenTime(-1), QString(), true);
     m_commandStack->push(command);
 }
 
@@ -5705,24 +5851,24 @@ void CustomTrackView::slotDeleteAllGuides()
 {
     QUndoCommand *deleteAll = new QUndoCommand();
     deleteAll->setText("Delete all guides");
-    for (int i = 0; i < m_guides.count(); i++) {
-        new EditGuideCommand(this, m_guides.at(i)->position(), m_guides.at(i)->label(), GenTime(), QString(), true, deleteAll);
+    for (int i = 0; i < m_guides.count(); ++i) {
+        new EditGuideCommand(this, m_guides.at(i)->position(), m_guides.at(i)->label(), GenTime(-1), QString(), true, deleteAll);
     }
     m_commandStack->push(deleteAll);
 }
 
-void CustomTrackView::setTool(PROJECTTOOL tool)
+void CustomTrackView::setTool(ProjectTool tool)
 {
     m_tool = tool;
     switch (m_tool) {
-      case RAZORTOOL:
-         setCursor(m_razorCursor);
-         break;
-      case SPACERTOOL:
-         setCursor(m_spacerCursor);
-         break;
-      default:
-         unsetCursor();
+    case RazorTool:
+        setCursor(m_razorCursor);
+        break;
+    case SpacerTool:
+        setCursor(m_spacerCursor);
+        break;
+    default:
+        unsetCursor();
     }
 }
 
@@ -5737,11 +5883,9 @@ void CustomTrackView::setScale(double scaleFactor, double verticalScale)
     setMatrix(newmatrix);
     if (adjust) {
         double newHeight = m_tracksHeight * m_document->tracksCount() * matrix().m22();
-        m_cursorLine->setLine(m_cursorLine->line().x1(), 0, m_cursorLine->line().x1(), newHeight - 1);
-        for (int i = 0; i < m_guides.count(); i++) {
-            QLineF l = m_guides.at(i)->line();
-            l.setP2(QPointF(l.x2(), newHeight));
-            m_guides.at(i)->setLine(l);
+        m_cursorLine->setLine(0, 0, 0, newHeight - 1);
+        for (int i = 0; i < m_guides.count(); ++i) {
+            m_guides.at(i)->setLine(0, 0, 0, newHeight - 1);
         }
         setSceneRect(0, 0, sceneRect().width(), m_tracksHeight * m_document->tracksCount());
     }
@@ -5760,7 +5904,7 @@ void CustomTrackView::setScale(double scaleFactor, double verticalScale)
 void CustomTrackView::slotRefreshGuides()
 {
     if (KdenliveSettings::showmarkers()) {
-        for (int i = 0; i < m_guides.count(); i++)
+        for (int i = 0; i < m_guides.count(); ++i)
             m_guides.at(i)->update();
     }
 }
@@ -5779,9 +5923,9 @@ void CustomTrackView::drawBackground(QPainter * painter, const QRectF &rect)
     int maxTrack = m_document->tracksCount();
     QColor lockedColor = palette().button().color();
     QColor audioColor = palette().alternateBase().color();
-    for (int i = 0; i < maxTrack; i++) {
+    for (int i = 0; i < maxTrack; ++i) {
         TrackInfo info = m_document->trackInfoAt(maxTrack - i - 1);
-        if (info.isLocked || info.type == AUDIOTRACK || i == m_selectedTrack) {
+        if (info.isLocked || info.type == AudioTrack || i == m_selectedTrack) {
             const QRectF track(min, m_tracksHeight * i + 1, max - min, m_tracksHeight - 1);
             if (i == m_selectedTrack)
                 painter->fillRect(track, m_activeTrackBrush.brush(this));
@@ -5841,9 +5985,9 @@ void CustomTrackView::initSearchStrings()
 {
     m_searchPoints.clear();
     QList<QGraphicsItem *> itemList = items();
-    for (int i = 0; i < itemList.count(); i++) {
+    for (int i = 0; i < itemList.count(); ++i) {
         // parse all clip names
-        if (itemList.at(i)->type() == AVWIDGET) {
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *item = static_cast <ClipItem *>(itemList.at(i));
             GenTime start = item->startPos();
             CommentedTime t(start, item->clipName());
@@ -5855,7 +5999,7 @@ void CustomTrackView::initSearchStrings()
     }
 
     // add guides
-    for (int i = 0; i < m_guides.count(); i++)
+    for (int i = 0; i < m_guides.count(); ++i)
         m_searchPoints.append(m_guides.at(i)->info());
 
     qSort(m_searchPoints);
@@ -5871,8 +6015,8 @@ QList<ItemInfo> CustomTrackView::findId(const QString &clipId)
 {
     QList<ItemInfo> matchingInfo;
     QList<QGraphicsItem *> itemList = items();
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *item = (ClipItem *)itemList.at(i);
             if (item->clipProducer() == clipId)
                 matchingInfo << item->info();
@@ -5890,12 +6034,12 @@ void CustomTrackView::copyClip()
         emit displayMessage(i18n("Select a clip before copying"), ErrorMessage);
         return;
     }
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(itemList.at(i));
             ClipItem *clone = clip->clone(clip->info());
             m_copiedItems.append(clone);
-        } else if (itemList.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (itemList.at(i)->type() == TransitionWidget) {
             Transition *dup = static_cast <Transition *>(itemList.at(i));
             m_copiedItems.append(dup->clone());
         }
@@ -5910,17 +6054,17 @@ bool CustomTrackView::canBePastedTo(ItemInfo info, int type) const
     }
     int height = m_tracksHeight - 2;
     int offset = 0;
-    if (type == TRANSITIONWIDGET) {
+    if (type == TransitionWidget) {
         height = Transition::itemHeight();
         offset = Transition::itemOffset();
     }
-    else if (type == AVWIDGET) {
+    else if (type == AVWidget) {
         height = ClipItem::itemHeight();
         offset = ClipItem::itemOffset();
     }
     QRectF rect((double) info.startPos.frames(m_document->fps()), (double)(info.track * m_tracksHeight + 1 + offset), (double)(info.endPos - info.startPos).frames(m_document->fps()), (double) height);
     QList<QGraphicsItem *> collisions = scene()->items(rect, Qt::IntersectsItemBoundingRect);
-    for (int i = 0; i < collisions.count(); i++) {
+    for (int i = 0; i < collisions.count(); ++i) {
         if (collisions.at(i)->type() == type) {
             return false;
         }
@@ -5931,12 +6075,12 @@ bool CustomTrackView::canBePastedTo(ItemInfo info, int type) const
 bool CustomTrackView::canBePastedTo(QList <ItemInfo> infoList, int type) const
 {
     QPainterPath path;
-    for (int i = 0; i < infoList.count(); i++) {
+    for (int i = 0; i < infoList.count(); ++i) {
         const QRectF rect((double) infoList.at(i).startPos.frames(m_document->fps()), (double)(infoList.at(i).track * m_tracksHeight + 1), (double)(infoList.at(i).endPos - infoList.at(i).startPos).frames(m_document->fps()), (double)(m_tracksHeight - 1));
         path.addRect(rect);
     }
     QList<QGraphicsItem *> collisions = scene()->items(path);
-    for (int i = 0; i < collisions.count(); i++) {
+    for (int i = 0; i < collisions.count(); ++i) {
         if (collisions.at(i)->type() == type) return false;
     }
     return true;
@@ -5944,7 +6088,7 @@ bool CustomTrackView::canBePastedTo(QList <ItemInfo> infoList, int type) const
 
 bool CustomTrackView::canBePasted(QList<AbstractClipItem *> items, GenTime offset, int trackOffset) const
 {
-    for (int i = 0; i < items.count(); i++) {
+    for (int i = 0; i < items.count(); ++i) {
         ItemInfo info = items.at(i)->info();
         info.startPos += offset;
         info.endPos += offset;
@@ -5959,7 +6103,7 @@ bool CustomTrackView::canBeMoved(QList<AbstractClipItem *> items, GenTime offset
     QPainterPath movePath;
     movePath.moveTo(0, 0);
 
-    for (int i = 0; i < items.count(); i++) {
+    for (int i = 0; i < items.count(); ++i) {
         ItemInfo info = items.at(i)->info();
         info.startPos = info.startPos + offset;
         info.endPos = info.endPos + offset;
@@ -5972,8 +6116,8 @@ bool CustomTrackView::canBeMoved(QList<AbstractClipItem *> items, GenTime offset
         movePath.addRect(rect);
     }
     QList<QGraphicsItem *> collisions = scene()->items(movePath, Qt::IntersectsItemBoundingRect);
-    for (int i = 0; i < collisions.count(); i++) {
-        if ((collisions.at(i)->type() == AVWIDGET || collisions.at(i)->type() == TRANSITIONWIDGET) && !items.contains(static_cast <AbstractClipItem *>(collisions.at(i)))) {
+    for (int i = 0; i < collisions.count(); ++i) {
+        if ((collisions.at(i)->type() == AVWidget || collisions.at(i)->type() == TransitionWidget) && !items.contains(static_cast <AbstractClipItem *>(collisions.at(i)))) {
             kDebug() << "  ////////////   CLIP COLLISION, MOVE NOT ALLOWED";
             return false;
         }
@@ -6011,7 +6155,7 @@ void CustomTrackView::pasteClip()
     GenTime leftPos = m_copiedItems.at(0)->startPos();
     int lowerTrack = m_copiedItems.at(0)->track();
     int upperTrack = m_copiedItems.at(0)->track();
-    for (int i = 1; i < m_copiedItems.count(); i++) {
+    for (int i = 1; i < m_copiedItems.count(); ++i) {
         if (m_copiedItems.at(i)->startPos() < leftPos) leftPos = m_copiedItems.at(i)->startPos();
         if (m_copiedItems.at(i)->track() < lowerTrack) lowerTrack = m_copiedItems.at(i)->track();
         if (m_copiedItems.at(i)->track() > upperTrack) upperTrack = m_copiedItems.at(i)->track();
@@ -6028,19 +6172,20 @@ void CustomTrackView::pasteClip()
     }
     QUndoCommand *pasteClips = new QUndoCommand();
     pasteClips->setText("Paste clips");
+    new RefreshMonitorCommand(this, false, true, pasteClips);
 
-    for (int i = 0; i < m_copiedItems.count(); i++) {
+    for (int i = 0; i < m_copiedItems.count(); ++i) {
         // parse all clip names
-        if (m_copiedItems.at(i) && m_copiedItems.at(i)->type() == AVWIDGET) {
+        if (m_copiedItems.at(i) && m_copiedItems.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(m_copiedItems.at(i));
             ItemInfo info = clip->info();
             info.startPos += offset;
             info.endPos += offset;
             info.track += trackOffset;
-            if (canBePastedTo(info, AVWIDGET)) {
+            if (canBePastedTo(info, AVWidget)) {
                 new AddTimelineClipCommand(this, clip->xml(), clip->clipProducer(), info, clip->effectList(), m_scene->editMode() == OVERWRITEEDIT, m_scene->editMode() == INSERTEDIT, true, false, pasteClips);
             } else emit displayMessage(i18n("Cannot paste clip to selected place"), ErrorMessage);
-        } else if (m_copiedItems.at(i) && m_copiedItems.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (m_copiedItems.at(i) && m_copiedItems.at(i)->type() == TransitionWidget) {
             Transition *tr = static_cast <Transition *>(m_copiedItems.at(i));
             ItemInfo info;
             info.startPos = tr->startPos() + offset;
@@ -6049,7 +6194,7 @@ void CustomTrackView::pasteClip()
             int transitionEndTrack;
             if (!tr->forcedTrack()) transitionEndTrack = getPreviousVideoTrack(info.track);
             else transitionEndTrack = tr->transitionEndTrack();
-            if (canBePastedTo(info, TRANSITIONWIDGET)) {
+            if (canBePastedTo(info, TransitionWidget)) {
                 if (info.startPos >= info.endPos) {
                     emit displayMessage(i18n("Invalid transition"), ErrorMessage);
                 } else new AddTransitionCommand(this, info, transitionEndTrack, tr->toXML(), false, true, pasteClips);
@@ -6057,13 +6202,13 @@ void CustomTrackView::pasteClip()
         }
     }
     updateTrackDuration(-1, pasteClips);
-    new RefreshMonitorCommand(this, false, pasteClips);
+    new RefreshMonitorCommand(this, false, false, pasteClips);
     m_commandStack->push(pasteClips);
 }
 
 void CustomTrackView::pasteClipEffects()
 {
-    if (m_copiedItems.count() != 1 || m_copiedItems.at(0)->type() != AVWIDGET) {
+    if (m_copiedItems.count() != 1 || m_copiedItems.at(0)->type() != AVWidget) {
         emit displayMessage(i18n("You must copy exactly one clip before pasting effects"), ErrorMessage);
         return;
     }
@@ -6076,10 +6221,10 @@ void CustomTrackView::pasteClipEffects()
 
     // expand groups
     for (int i = 0; i < clips.count(); ++i) {
-        if (clips.at(i)->type() == GROUPWIDGET) {
+        if (clips.at(i)->type() == GroupWidget) {
             QList<QGraphicsItem *> children = clips.at(i)->childItems();
             for (int j = 0; j < children.count(); j++) {
-                if (children.at(j)->type() == AVWIDGET && !clips.contains(children.at(j))) {
+                if (children.at(j)->type() == AVWidget && !clips.contains(children.at(j))) {
                     clips.append(children.at(j));
                 }
             }
@@ -6087,7 +6232,7 @@ void CustomTrackView::pasteClipEffects()
     }
 
     for (int i = 0; i < clips.count(); ++i) {
-        if (clips.at(i)->type() == AVWIDGET) {
+        if (clips.at(i)->type() == AVWidget) {
             ClipItem *item = static_cast < ClipItem *>(clips.at(i));
             for (int j = 0; j < clip->effectsCount(); j++) {
                 QDomElement eff = clip->effect(j);
@@ -6103,7 +6248,7 @@ void CustomTrackView::pasteClipEffects()
 
     //adjust effects (fades, ...)
     for (int i = 0; i < clips.count(); ++i) {
-        if (clips.at(i)->type() == AVWIDGET) {
+        if (clips.at(i)->type() == AVWidget) {
             ClipItem *item = static_cast < ClipItem *>(clips.at(i));
             updatePositionEffects(item, item->info());
         }
@@ -6118,7 +6263,7 @@ void CustomTrackView::adjustKeyfames(GenTime oldstart, GenTime newstart, GenTime
     int max = (newstart + duration).frames(m_document->fps());
     QLocale locale;
     QDomNodeList params = xml.elementsByTagName("parameter");
-    for (int i = 0; i < params.count(); i++) {
+    for (int i = 0; i < params.count(); ++i) {
         QDomElement e = params.item(i).toElement();
         if (!e.isNull() && (e.attribute("type") == "keyframe" || e.attribute("type") == "simplekeyframe")) {
             QString def = e.attribute("default");
@@ -6144,10 +6289,10 @@ ClipItem *CustomTrackView::getClipUnderCursor() const
 {
     QRectF rect((double) m_cursorPos, 0.0, 1.0, (double)(m_tracksHeight * m_document->tracksCount()));
     QList<QGraphicsItem *> collisions = scene()->items(rect, Qt::IntersectsItemBoundingRect);
-    for (int i = 0; i < collisions.count(); i++) {
-        if (collisions.at(i)->type() == AVWIDGET) {
-           ClipItem *clip = static_cast < ClipItem *>(collisions.at(i));
-           if (!clip->isItemLocked()) return clip;
+    for (int i = 0; i < collisions.count(); ++i) {
+        if (collisions.at(i)->type() == AVWidget) {
+            ClipItem *clip = static_cast < ClipItem *>(collisions.at(i));
+            if (!clip->isItemLocked()) return clip;
         }
     }
     return NULL;
@@ -6161,7 +6306,7 @@ AbstractClipItem *CustomTrackView::getMainActiveClip() const
     } else {
         AbstractClipItem *item = NULL;
         for (int i = 0; i < clips.count(); ++i) {
-            if (clips.at(i)->type() == AVWIDGET) {
+            if (clips.at(i)->type() == AVWidget) {
                 item = static_cast < AbstractClipItem *>(clips.at(i));
                 if (clips.count() > 1 && item->startPos().frames(m_document->fps()) <= m_cursorPos && item->endPos().frames(m_document->fps()) >= m_cursorPos) break;
             }
@@ -6180,12 +6325,12 @@ ClipItem *CustomTrackView::getActiveClipUnderCursor(bool allowOutsideCursor) con
         ClipItem *item;
         // remove all items in the list that are not clips
         for (int i = 0; i < clips.count();) {
-            if (clips.at(i)->type() != AVWIDGET) clips.removeAt(i);
-            else i++;
+            if (clips.at(i)->type() != AVWidget) clips.removeAt(i);
+            else ++i;
         }
         if (clips.count() == 1 && allowOutsideCursor) return static_cast < ClipItem *>(clips.at(0));
         for (int i = 0; i < clips.count(); ++i) {
-            if (clips.at(i)->type() == AVWIDGET) {
+            if (clips.at(i)->type() == AVWidget) {
                 item = static_cast < ClipItem *>(clips.at(i));
                 if (item->startPos().frames(m_document->fps()) <= m_cursorPos && item->endPos().frames(m_document->fps()) >= m_cursorPos)
                     return item;
@@ -6199,7 +6344,7 @@ void CustomTrackView::setInPoint()
 {
     AbstractClipItem *clip = getActiveClipUnderCursor(true);
     if (clip == NULL) {
-        if (m_dragItem && m_dragItem->type() == TRANSITIONWIDGET) {
+        if (m_dragItem && m_dragItem->type() == TransitionWidget) {
             clip = m_dragItem;
         } else {
             emit displayMessage(i18n("You must select one clip for this action"), ErrorMessage);
@@ -6209,21 +6354,21 @@ void CustomTrackView::setInPoint()
 
     AbstractGroupItem *parent = static_cast <AbstractGroupItem *>(clip->parentItem());
     if (parent) {
-       // Resizing a group
-       QUndoCommand *resizeCommand = new QUndoCommand();
+        // Resizing a group
+        QUndoCommand *resizeCommand = new QUndoCommand();
         resizeCommand->setText(i18n("Resize group"));
         QList <QGraphicsItem *> items = parent->childItems();
         for (int i = 0; i < items.count(); ++i) {
-           AbstractClipItem *item = static_cast<AbstractClipItem *>(items.at(i));
-            if (item && item->type() == AVWIDGET) {
+            AbstractClipItem *item = static_cast<AbstractClipItem *>(items.at(i));
+            if (item && item->type() == AVWidget) {
                 prepareResizeClipStart(item, item->info(), m_cursorPos, true, resizeCommand);
             }
         }
         if (resizeCommand->childCount() > 0) m_commandStack->push(resizeCommand);
-       else {
-           //TODO warn user of failed resize
-           delete resizeCommand;
-       }
+        else {
+            //TODO warn user of failed resize
+            delete resizeCommand;
+        }
     }
     else prepareResizeClipStart(clip, clip->info(), m_cursorPos, true);
 }
@@ -6232,7 +6377,7 @@ void CustomTrackView::setOutPoint()
 {
     AbstractClipItem *clip = getActiveClipUnderCursor(true);
     if (clip == NULL) {
-        if (m_dragItem && m_dragItem->type() == TRANSITIONWIDGET) {
+        if (m_dragItem && m_dragItem->type() == TransitionWidget) {
             clip = m_dragItem;
         } else {
             emit displayMessage(i18n("You must select one clip for this action"), ErrorMessage);
@@ -6241,21 +6386,21 @@ void CustomTrackView::setOutPoint()
     }
     AbstractGroupItem *parent = static_cast <AbstractGroupItem *>(clip->parentItem());
     if (parent) {
-       // Resizing a group
-       QUndoCommand *resizeCommand = new QUndoCommand();
+        // Resizing a group
+        QUndoCommand *resizeCommand = new QUndoCommand();
         resizeCommand->setText(i18n("Resize group"));
         QList <QGraphicsItem *> items = parent->childItems();
         for (int i = 0; i < items.count(); ++i) {
-           AbstractClipItem *item = static_cast<AbstractClipItem *>(items.at(i));
-            if (item && item->type() == AVWIDGET) {
+            AbstractClipItem *item = static_cast<AbstractClipItem *>(items.at(i));
+            if (item && item->type() == AVWidget) {
                 prepareResizeClipEnd(item, item->info(), m_cursorPos, true, resizeCommand);
             }
         }
         if (resizeCommand->childCount() > 0) m_commandStack->push(resizeCommand);
-       else {
-           //TODO warn user of failed resize
-           delete resizeCommand;
-       }
+        else {
+            //TODO warn user of failed resize
+            delete resizeCommand;
+        }
     }
     else prepareResizeClipEnd(clip, clip->info(), m_cursorPos, true);
 }
@@ -6267,12 +6412,12 @@ void CustomTrackView::slotUpdateAllThumbs()
     //if (itemList.isEmpty()) return;
     ClipItem *item;
     const QString thumbBase = m_document->projectFolder().path() + "/thumbs/";
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             item = static_cast <ClipItem *>(itemList.at(i));
-            if (item && item->isEnabled() && item->clipType() != COLOR && item->clipType() != AUDIO) {
+            if (item && item->isEnabled() && item->clipType() != Color && item->clipType() != Audio) {
                 // Check if we have a cached thumbnail
-                if (item->clipType() == IMAGE || item->clipType() == TEXT) {
+                if (item->clipType() == Image || item->clipType() == Text) {
                     QString thumb = thumbBase + item->baseClip()->getClipHash() + "_0.png";
                     if (QFile::exists(thumb)) {
                         QPixmap pix(thumb);
@@ -6307,12 +6452,12 @@ void CustomTrackView::saveThumbnails()
     QList<QGraphicsItem *> itemList = items();
     ClipItem *item;
     QString thumbBase = m_document->projectFolder().path() + "/thumbs/";
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == AVWidget) {
             item = static_cast <ClipItem *>(itemList.at(i));
-            if (item->clipType() != COLOR) {
+            if (item->clipType() != Color) {
                 // Check if we have a cached thumbnail
-                if (item->clipType() == IMAGE || item->clipType() == TEXT || item->clipType() == AUDIO) {
+                if (item->clipType() == Image || item->clipType() == Text || item->clipType() == Audio) {
                     QString thumb = thumbBase + item->baseClip()->getClipHash() + "_0.png";
                     if (!QFile::exists(thumb)) {
                         QPixmap pix(item->startThumb());
@@ -6353,12 +6498,12 @@ void CustomTrackView::slotInsertTrack(int ix)
         info.duration = 0;
         info.isMute = false;
         info.isLocked = false;
-       info.effectsList = EffectsList(true);
+        info.effectsList = EffectsList(true);
         if (d->video_track->isChecked()) {
-            info.type = VIDEOTRACK;
+            info.type = VideoTrack;
             info.isBlind = false;
         } else {
-            info.type = AUDIOTRACK;
+            info.type = AudioTrack;
             info.isBlind = true;
         }
         AddTrackCommand *addTrack = new AddTrackCommand(this, ix, info, true);
@@ -6392,7 +6537,7 @@ void CustomTrackView::slotDeleteTrack(int ix)
 void CustomTrackView::slotConfigTracks(int ix)
 {
     QPointer<TracksConfigDialog> d = new TracksConfigDialog(m_document,
-                                                        ix, parentWidget());
+                                                            ix, parentWidget());
     if (d->exec() == QDialog::Accepted) {
         ConfigTracksCommand *configTracks = new ConfigTracksCommand(this, m_document->tracksList(), d->tracksList());
         m_commandStack->push(configTracks);
@@ -6409,22 +6554,27 @@ void CustomTrackView::slotConfigTracks(int ix)
 void CustomTrackView::deleteTimelineTrack(int ix, TrackInfo trackinfo)
 {
     if (m_document->tracksCount() < 2) return;
+    // Clear effect stack
+    clearSelection();
+    emit transitionItemSelected(NULL);
+    
     double startY = ix * m_tracksHeight + 1 + m_tracksHeight / 2;
     QRectF r(0, startY, sceneRect().width(), m_tracksHeight / 2 - 1);
     QList<QGraphicsItem *> selection = m_scene->items(r);
     QUndoCommand *deleteTrack = new QUndoCommand();
     deleteTrack->setText("Delete track");
+    new RefreshMonitorCommand(this, false, true, deleteTrack);
 
     // Delete all clips in selected track
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             ClipItem *item =  static_cast <ClipItem *>(selection.at(i));
             new AddTimelineClipCommand(this, item->xml(), item->clipProducer(), item->info(), item->effectList(), false, false, false, true, deleteTrack);
             m_waitingThumbs.removeAll(item);
             m_scene->removeItem(item);
             delete item;
             item = NULL;
-        } else if (selection.at(i)->type() == TRANSITIONWIDGET) {
+        } else if (selection.at(i)->type() == TransitionWidget) {
             Transition *item =  static_cast <Transition *>(selection.at(i));
             new AddTransitionCommand(this, item->info(), item->transitionEndTrack(), item->toXML(), true, false, deleteTrack);
             m_scene->removeItem(item);
@@ -6434,13 +6584,14 @@ void CustomTrackView::deleteTimelineTrack(int ix, TrackInfo trackinfo)
     }
 
     new AddTrackCommand(this, ix, trackinfo, false, deleteTrack);
+    new RefreshMonitorCommand(this, true, false, deleteTrack);
     m_commandStack->push(deleteTrack);
 }
 
 void CustomTrackView::autoTransition()
 {
     QList<QGraphicsItem *> itemList = scene()->selectedItems();
-    if (itemList.count() != 1 || itemList.at(0)->type() != TRANSITIONWIDGET) {
+    if (itemList.count() != 1 || itemList.at(0)->type() != TransitionWidget) {
         emit displayMessage(i18n("You must select one transition for this action"), ErrorMessage);
         return;
     }
@@ -6456,7 +6607,7 @@ void CustomTrackView::clipNameChanged(const QString &id, const QString &name)
     QList<QGraphicsItem *> list = scene()->items();
     ClipItem *clip = NULL;
     for (int i = 0; i < list.size(); ++i) {
-        if (list.at(i)->type() == AVWIDGET) {
+        if (list.at(i)->type() == AVWidget) {
             clip = static_cast <ClipItem *>(list.at(i));
             if (clip->clipProducer() == id) {
                 clip->setClipName(name);
@@ -6473,9 +6624,9 @@ void CustomTrackView::getClipAvailableSpace(AbstractClipItem *item, GenTime &min
     QList<QGraphicsItem *> selection;
     selection = m_scene->items(0, item->track() * m_tracksHeight + m_tracksHeight / 2, sceneRect().width(), 2);
     selection.removeAll(item);
-    for (int i = 0; i < selection.count(); i++) {
+    for (int i = 0; i < selection.count(); ++i) {
         AbstractClipItem *clip = static_cast <AbstractClipItem *>(selection.at(i));
-        if (clip && clip->type() == AVWIDGET) {
+        if (clip && clip->type() == AVWidget) {
             if (clip->endPos() <= item->startPos() && clip->endPos() > minimum) minimum = clip->endPos();
             if (clip->startPos() > item->startPos() && (clip->startPos() < maximum || maximum == GenTime())) maximum = clip->startPos();
         }
@@ -6489,9 +6640,9 @@ void CustomTrackView::getTransitionAvailableSpace(AbstractClipItem *item, GenTim
     QList<QGraphicsItem *> selection;
     selection = m_scene->items(0, (item->track() + 1) * m_tracksHeight, sceneRect().width(), 2);
     selection.removeAll(item);
-    for (int i = 0; i < selection.count(); i++) {
+    for (int i = 0; i < selection.count(); ++i) {
         AbstractClipItem *clip = static_cast <AbstractClipItem *>(selection.at(i));
-        if (clip && clip->type() == TRANSITIONWIDGET) {
+        if (clip && clip->type() == TransitionWidget) {
             if (clip->endPos() <= item->startPos() && clip->endPos() > minimum) minimum = clip->endPos();
             if (clip->startPos() > item->startPos() && (clip->startPos() < maximum || maximum == GenTime())) maximum = clip->startPos();
         }
@@ -6500,10 +6651,10 @@ void CustomTrackView::getTransitionAvailableSpace(AbstractClipItem *item, GenTim
 
 void CustomTrackView::loadGroups(const QDomNodeList &groups)
 {
-    for (int i = 0; i < groups.count(); i++) {
+    for (int i = 0; i < groups.count(); ++i) {
         QDomNodeList children = groups.at(i).childNodes();
         scene()->clearSelection();
-       QList <QGraphicsItem*>list;
+        QList <QGraphicsItem*>list;
         for (int nodeindex = 0; nodeindex < children.count(); nodeindex++) {
             QDomElement elem = children.item(nodeindex).toElement();
             int pos = elem.attribute("position").toInt();
@@ -6516,7 +6667,7 @@ void CustomTrackView::loadGroups(const QDomNodeList &groups)
                 if (clip) list.append(clip);//clip->setSelected(true);
             }
         }
-        groupSelectedItems(list, false, true);
+        groupSelectedItems(list, true);
     }
 }
 
@@ -6530,10 +6681,10 @@ void CustomTrackView::splitAudio()
     }
     QUndoCommand *splitCommand = new QUndoCommand();
     splitCommand->setText(i18n("Split audio"));
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(selection.at(i));
-            if (clip->clipType() == AV || clip->clipType() == PLAYLIST) {
+            if (clip->clipType() == AV || clip->clipType() == Playlist) {
                 if (clip->parentItem()) {
                     emit displayMessage(i18n("Cannot split audio of grouped clips"), ErrorMessage);
                 } else {
@@ -6560,9 +6711,9 @@ void CustomTrackView::setAudioAlignReference()
     if (m_audioCorrelator != NULL) {
         delete m_audioCorrelator;
     }
-    if (selection.at(0)->type() == AVWIDGET) {
+    if (selection.at(0)->type() == AVWidget) {
         ClipItem *clip = static_cast<ClipItem*>(selection.at(0));
-        if (clip->clipType() == AV || clip->clipType() == AUDIO) {
+        if (clip->clipType() == AV || clip->clipType() == Audio) {
             m_audioAlignmentReference = clip;
 
             AudioEnvelope *envelope = new AudioEnvelope(clip->baseClip()->fileURL().path(), clip->getProducer(clip->track()));
@@ -6602,14 +6753,14 @@ void CustomTrackView::alignAudio()
     int counter = 0;
     QList<QGraphicsItem *> selection = scene()->selectedItems();
     foreach (QGraphicsItem *item, selection) {
-        if (item->type() == AVWIDGET) {
+        if (item->type() == AVWidget) {
 
             ClipItem *clip = static_cast<ClipItem*>(item);
             if (clip == m_audioAlignmentReference) {
                 continue;
             }
 
-            if (clip->clipType() == AV || clip->clipType() == AUDIO) {
+            if (clip->clipType() == AV || clip->clipType() == Audio) {
                 AudioEnvelope *envelope = new AudioEnvelope(clip->baseClip()->fileURL().path(), clip->getProducer(clip->track()), clip->info().cropStart.frames(m_document->fps()), clip->info().cropDuration.frames(m_document->fps()));
 
                 // FFT only for larger vectors. We could use it all time, but for small vectors
@@ -6700,12 +6851,12 @@ void CustomTrackView::doSplitAudio(const GenTime &pos, int track, EffectsList ef
         int freetrack = m_document->tracksCount() - track - 1;
 
         // do not split audio when we are on an audio track
-        if (m_document->trackInfoAt(freetrack).type == AUDIOTRACK)
+        if (m_document->trackInfoAt(freetrack).type == AudioTrack)
             return;
 
         for (; freetrack > 0; freetrack--) {
             //kDebug() << "// CHK DOC TRK:" << freetrack << ", DUR:" << m_document->renderer()->mltTrackDuration(freetrack);
-            if (m_document->trackInfoAt(freetrack - 1).type == AUDIOTRACK && !m_document->trackInfoAt(freetrack - 1).isLocked) {
+            if (m_document->trackInfoAt(freetrack - 1).type == AudioTrack && !m_document->trackInfoAt(freetrack - 1).isLocked) {
                 //kDebug() << "// CHK DOC TRK:" << freetrack << ", DUR:" << m_document->renderer()->mltTrackDuration(freetrack);
                 if (m_document->renderer()->mltTrackDuration(freetrack) < start || m_document->renderer()->mltGetSpaceLength(pos, freetrack, false) >= clip->cropDuration().frames(m_document->fps())) {
                     //kDebug() << "FOUND SPACE ON TRK: " << freetrack;
@@ -6718,14 +6869,14 @@ void CustomTrackView::doSplitAudio(const GenTime &pos, int track, EffectsList ef
         } else {
             ItemInfo info = clip->info();
             info.track = m_document->tracksCount() - freetrack;
-           QDomElement xml = clip->xml();
-           xml.setAttribute("audio_only", 1);
-           scene()->clearSelection();
+            QDomElement xml = clip->xml();
+            xml.setAttribute("audio_only", 1);
+            scene()->clearSelection();
             addClip(xml, clip->clipProducer(), info, clip->effectList(), false, false, false);
             clip->setSelected(true);
             ClipItem *audioClip = getClipItemAt(start, info.track);
             if (audioClip) {
-               clip->setVideoOnly(true);
+                clip->setVideoOnly(true);
                 Mlt::Tractor *tractor = m_document->renderer()->lockService();
                 if (m_document->renderer()->mltUpdateClipProducer(tractor, m_document->tracksCount() - track, start, clip->baseClip()->videoProducer(info.track)) == false) {
                     emit displayMessage(i18n("Cannot update clip (time: %1, track: %2)", start, track), ErrorMessage);
@@ -6745,12 +6896,12 @@ void CustomTrackView::doSplitAudio(const GenTime &pos, int track, EffectsList ef
                         videoIx++;
                     }
                 }
-                groupSelectedItems(QList <QGraphicsItem*>()<<clip<<audioClip, false, true);
+                groupSelectedItems(QList <QGraphicsItem*>()<<clip<<audioClip, true);
             }
         }
     } else {
         // unsplit clip: remove audio part and change video part to normal clip
-        if (clip->parentItem() == NULL || clip->parentItem()->type() != GROUPWIDGET) {
+        if (clip->parentItem() == NULL || clip->parentItem()->type() != GroupWidget) {
             kDebug() << "//CANNOT FIND CLP GRP";
             return;
         }
@@ -6760,7 +6911,7 @@ void CustomTrackView::doSplitAudio(const GenTime &pos, int track, EffectsList ef
             kDebug() << "//SOMETHING IS WRONG WITH CLP GRP";
             return;
         }
-        for (int i = 0; i < children.count(); i++) {
+        for (int i = 0; i < children.count(); ++i) {
             if (children.at(i) != clip) {
                 ClipItem *clp = static_cast <ClipItem *>(children.at(i));
                 ItemInfo info = clip->info();
@@ -6791,6 +6942,7 @@ void CustomTrackView::doSplitAudio(const GenTime &pos, int track, EffectsList ef
         }
         clip->setFlag(QGraphicsItem::ItemIsMovable, true);
         m_document->clipManager()->removeGroup(grp);
+        if (grp == m_selectionGroup) m_selectionGroup = NULL;
         scene()->destroyItemGroup(grp);
     }
 }
@@ -6805,10 +6957,10 @@ void CustomTrackView::setVideoOnly()
     }
     QUndoCommand *videoCommand = new QUndoCommand();
     videoCommand->setText(i18n("Video only"));
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(selection.at(i));
-            if (clip->clipType() == AV || clip->clipType() == PLAYLIST) {
+            if (clip->clipType() == AV || clip->clipType() == Playlist) {
                 if (clip->parentItem()) {
                     emit displayMessage(i18n("Cannot change grouped clips"), ErrorMessage);
                 } else {
@@ -6830,10 +6982,10 @@ void CustomTrackView::setAudioOnly()
     }
     QUndoCommand *videoCommand = new QUndoCommand();
     videoCommand->setText(i18n("Audio only"));
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(selection.at(i));
-            if (clip->clipType() == AV || clip->clipType() == PLAYLIST) {
+            if (clip->clipType() == AV || clip->clipType() == Playlist) {
                 if (clip->parentItem()) {
                     emit displayMessage(i18n("Cannot change grouped clips"), ErrorMessage);
                 } else {
@@ -6855,10 +7007,10 @@ void CustomTrackView::setAudioAndVideo()
     }
     QUndoCommand *videoCommand = new QUndoCommand();
     videoCommand->setText(i18n("Audio and Video"));
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(selection.at(i));
-            if (clip->clipType() == AV || clip->clipType() == PLAYLIST) {
+            if (clip->clipType() == AV || clip->clipType() == Playlist) {
                 if (clip->parentItem()) {
                     emit displayMessage(i18n("Cannot change grouped clips"), ErrorMessage);
                 } else {
@@ -6912,22 +7064,47 @@ void CustomTrackView::doChangeClipType(const GenTime &pos, int track, bool video
 
 void CustomTrackView::updateClipTypeActions(ClipItem *clip)
 {
-    if (clip == NULL || (clip->clipType() != AV && clip->clipType() != PLAYLIST)) {
+    bool hasAudio;
+    bool hasAV;
+    if (clip == NULL || (clip->clipType() != AV && clip->clipType() != Playlist)) {
         m_clipTypeGroup->setEnabled(false);
+        hasAudio = clip != NULL && clip->clipType() == Audio;
+        hasAV = false;
     } else {
+        switch (clip->clipType()) {
+        case AV:
+        case Playlist:
+            hasAudio = true;
+            hasAV = true;
+            break;
+        case Audio:
+            hasAudio = true;
+            hasAV = false;
+            break;
+        default:
+            hasAudio = false;
+            hasAV = false;
+        }
         m_clipTypeGroup->setEnabled(true);
         QList <QAction *> actions = m_clipTypeGroup->actions();
         QString lookup;
         if (clip->isAudioOnly()) lookup = "clip_audio_only";
         else if (clip->isVideoOnly()) lookup = "clip_video_only";
         else  lookup = "clip_audio_and_video";
-        for (int i = 0; i < actions.count(); i++) {
+        for (int i = 0; i < actions.count(); ++i) {
             if (actions.at(i)->data().toString() == lookup) {
                 actions.at(i)->setChecked(true);
                 break;
             }
         }
     }
+    
+    for (int i = 0; i < m_audioActions.count(); ++i) {
+        m_audioActions.at(i)->setEnabled(hasAudio);
+    }
+    for (int i = 0; i < m_avActions.count(); ++i) {
+        m_avActions.at(i)->setEnabled(hasAV);
+    }
 }
 
 void CustomTrackView::slotGoToMarker(QAction *action)
@@ -6942,7 +7119,7 @@ void CustomTrackView::reloadTransitionLumas()
     QString lumaFiles;
     QDomElement lumaTransition = MainWindow::transitions.getEffectByTag("luma", "luma");
     QDomNodeList params = lumaTransition.elementsByTagName("parameter");
-    for (int i = 0; i < params.count(); i++) {
+    for (int i = 0; i < params.count(); ++i) {
         QDomElement e = params.item(i).toElement();
         if (e.attribute("tag") == "resource") {
             lumaNames = e.attribute("paramlistdisplay");
@@ -6954,13 +7131,13 @@ void CustomTrackView::reloadTransitionLumas()
     QList<QGraphicsItem *> itemList = items();
     Transition *transitionitem;
     QDomElement transitionXml;
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == TRANSITIONWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == TransitionWidget) {
             transitionitem = static_cast <Transition*>(itemList.at(i));
             transitionXml = transitionitem->toXML();
             if (transitionXml.attribute("id") == "luma" && transitionXml.attribute("tag") == "luma") {
                 QDomNodeList params = transitionXml.elementsByTagName("parameter");
-                for (int i = 0; i < params.count(); i++) {
+                for (int i = 0; i < params.count(); ++i) {
                     QDomElement e = params.item(i).toElement();
                     if (e.attribute("tag") == "resource") {
                         e.setAttribute("paramlistdisplay", lumaNames);
@@ -6971,7 +7148,7 @@ void CustomTrackView::reloadTransitionLumas()
             }
             if (transitionXml.attribute("id") == "composite" && transitionXml.attribute("tag") == "composite") {
                 QDomNodeList params = transitionXml.elementsByTagName("parameter");
-                for (int i = 0; i < params.count(); i++) {
+                for (int i = 0; i < params.count(); ++i) {
                     QDomElement e = params.item(i).toElement();
                     if (e.attribute("tag") == "luma") {
                         e.setAttribute("paramlistdisplay", lumaNames);
@@ -6997,20 +7174,20 @@ void CustomTrackView::updateProjectFps()
     scene()->clearSelection();
     m_dragItem = NULL;
     QList<QGraphicsItem *> itemList = items();
-    for (int i = 0; i < itemList.count(); i++) {
+    for (int i = 0; i < itemList.count(); ++i) {
         // remove all items and re-add them one by one
         if (itemList.at(i) != m_cursorLine && itemList.at(i)->parentItem() == NULL) m_scene->removeItem(itemList.at(i));
     }
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->parentItem() == 0 && (itemList.at(i)->type() == AVWIDGET || itemList.at(i)->type() == TRANSITIONWIDGET)) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->parentItem() == 0 && (itemList.at(i)->type() == AVWidget || itemList.at(i)->type() == TransitionWidget)) {
             AbstractClipItem *clip = static_cast <AbstractClipItem *>(itemList.at(i));
             clip->updateFps(m_document->fps());
             m_scene->addItem(clip);
-        } else if (itemList.at(i)->type() == GROUPWIDGET) {
+        } else if (itemList.at(i)->type() == GroupWidget) {
             AbstractGroupItem *grp = static_cast <AbstractGroupItem *>(itemList.at(i));
             QList<QGraphicsItem *> children = grp->childItems();
             for (int j = 0; j < children.count(); j++) {
-                if (children.at(j)->type() == AVWIDGET || children.at(j)->type() == TRANSITIONWIDGET) {
+                if (children.at(j)->type() == AVWidget || children.at(j)->type() == TransitionWidget) {
                     AbstractClipItem *clip = static_cast <AbstractClipItem *>(children.at(j));
                     clip->setFlag(QGraphicsItem::ItemIsMovable, true);
                     clip->updateFps(m_document->fps());
@@ -7018,15 +7195,16 @@ void CustomTrackView::updateProjectFps()
             }
             m_document->clipManager()->removeGroup(grp);
             m_scene->addItem(grp);
+            if (grp == m_selectionGroup) m_selectionGroup = NULL;
             scene()->destroyItemGroup(grp);
             scene()->clearSelection();
             /*for (int j = 0; j < children.count(); j++) {
-                if (children.at(j)->type() == AVWIDGET || children.at(j)->type() == TRANSITIONWIDGET) {
+                if (children.at(j)->type() == AVWidget || children.at(j)->type() == TRANSITIONWIDGET) {
                     //children.at(j)->setParentItem(0);
                     children.at(j)->setSelected(true);
                 }
             }*/
-            groupSelectedItems(children, true, true);
+            groupSelectedItems(children, true);
         } else if (itemList.at(i)->type() == GUIDEITEM) {
             Guide *g = static_cast<Guide *>(itemList.at(i));
             g->updatePos();
@@ -7065,8 +7243,8 @@ QStringList CustomTrackView::selectedClips() const
 {
     QStringList clipIds;
     QList<QGraphicsItem *> selection = m_scene->selectedItems();
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             ClipItem *item = (ClipItem *)selection.at(i);
             clipIds << item->clipProducer();
         }
@@ -7078,8 +7256,8 @@ QList<ClipItem *> CustomTrackView::selectedClipItems() const
 {
     QList<ClipItem *> clips;
     QList<QGraphicsItem *> selection = m_scene->selectedItems();
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             clips.append((ClipItem *)selection.at(i));
         }
     }
@@ -7103,19 +7281,19 @@ void CustomTrackView::slotSelectClipsInTrack()
     QList<QGraphicsItem *> selection = m_scene->items(rect);
     m_scene->clearSelection();
     QList<QGraphicsItem *> list;
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET || selection.at(i)->type() == TRANSITIONWIDGET || selection.at(i)->type() == GROUPWIDGET) {
-           list.append(selection.at(i));
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget || selection.at(i)->type() == TransitionWidget || selection.at(i)->type() == GroupWidget) {
+            list.append(selection.at(i));
         }
-    }    
-    groupSelectedItems(list, false, false, true);
+    }
+    groupSelectedItems(list, false, true);
 }
 
 void CustomTrackView::slotSelectAllClips()
 {
     m_scene->clearSelection();
     resetSelectionGroup();
-    groupSelectedItems(m_scene->items(), false, false, true);
+    groupSelectedItems(m_scene->items(), false, true);
 }
 
 void CustomTrackView::selectClip(bool add, bool group, int track, int pos)
@@ -7128,8 +7306,8 @@ void CustomTrackView::selectClip(bool add, bool group, int track, int pos)
     QList<QGraphicsItem *> selection = m_scene->items(rect);
     resetSelectionGroup(group);
     if (!group) m_scene->clearSelection();
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             selection.at(i)->setSelected(add);
             break;
         }
@@ -7143,8 +7321,8 @@ void CustomTrackView::selectTransition(bool add, bool group)
     QList<QGraphicsItem *> selection = m_scene->items(rect);
     resetSelectionGroup(group);
     if (!group) m_scene->clearSelection();
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == TRANSITIONWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == TransitionWidget) {
             selection.at(i)->setSelected(add);
             break;
         }
@@ -7158,8 +7336,8 @@ QStringList CustomTrackView::extractTransitionsLumas()
     QList<QGraphicsItem *> itemList = items();
     Transition *transitionitem;
     QDomElement transitionXml;
-    for (int i = 0; i < itemList.count(); i++) {
-        if (itemList.at(i)->type() == TRANSITIONWIDGET) {
+    for (int i = 0; i < itemList.count(); ++i) {
+        if (itemList.at(i)->type() == TransitionWidget) {
             transitionitem = static_cast <Transition*>(itemList.at(i));
             transitionXml = transitionitem->toXML();
             // luma files in transitions can be in "resource" or "luma" property
@@ -7185,8 +7363,8 @@ void CustomTrackView::checkTrackSequence(int track)
     QList<QGraphicsItem *> selection = m_scene->items(rect);
     QList <int> timelineList;
     timelineList.append(0);
-    for (int i = 0; i < selection.count(); i++) {
-        if (selection.at(i)->type() == AVWIDGET) {
+    for (int i = 0; i < selection.count(); ++i) {
+        if (selection.at(i)->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(selection.at(i));
             int start = clip->startPos().frames(m_document->fps());
             int end = clip->endPos().frames(m_document->fps());
@@ -7224,6 +7402,7 @@ void CustomTrackView::insertZoneOverwrite(QStringList data, int in)
 
 void CustomTrackView::clearSelection(bool emitInfo)
 {
+    if (m_dragItem) m_dragItem->setSelected(false);
     resetSelectionGroup();
     scene()->clearSelection();
     m_dragItem = NULL;
@@ -7253,7 +7432,7 @@ void CustomTrackView::removeTipAnimation()
     }
 }
 
-void CustomTrackView::setTipAnimation(AbstractClipItem *clip, OPERATIONTYPE mode, const double size)
+void CustomTrackView::setTipAnimation(AbstractClipItem *clip, OperationType mode, const double size)
 {
     if (m_visualTip == NULL) {
         QRectF rect = clip->sceneBoundingRect();
@@ -7262,22 +7441,22 @@ void CustomTrackView::setTipAnimation(AbstractClipItem *clip, OPERATIONTYPE mode
         m_animation->setScaleAt(1, 1, 1);
         QPolygon polygon;
         switch (mode) {
-        case FADEIN:
-        case FADEOUT:
+        case FadeIn:
+        case FadeOut:
             m_visualTip = new QGraphicsEllipseItem(-size, -size, size * 2, size * 2);
             ((QGraphicsEllipseItem*) m_visualTip)->setBrush(m_tipColor);
             ((QGraphicsEllipseItem*) m_visualTip)->setPen(m_tipPen);
-            if (mode == FADEIN)
+            if (mode == FadeIn)
                 m_visualTip->setPos(rect.x() + ((ClipItem *) clip)->fadeIn(), rect.y());
             else
                 m_visualTip->setPos(rect.right() - ((ClipItem *) clip)->fadeOut(), rect.y());
 
             m_animation->setScaleAt(.5, 2, 2);
             break;
-        case RESIZESTART:
-        case RESIZEEND:
+        case ResizeStart:
+        case ResizeEnd:
             polygon << QPoint(0, - size * 2);
-            if (mode == RESIZESTART)
+            if (mode == ResizeStart)
                 polygon << QPoint(size * 2, 0);
             else
                 polygon << QPoint(- size * 2, 0);
@@ -7287,17 +7466,17 @@ void CustomTrackView::setTipAnimation(AbstractClipItem *clip, OPERATIONTYPE mode
             m_visualTip = new QGraphicsPolygonItem(polygon);
             ((QGraphicsPolygonItem*) m_visualTip)->setBrush(m_tipColor);
             ((QGraphicsPolygonItem*) m_visualTip)->setPen(m_tipPen);
-            if (mode == RESIZESTART)
+            if (mode == ResizeStart)
                 m_visualTip->setPos(rect.x(), rect.y() + rect.height() / 2);
             else
                 m_visualTip->setPos(rect.right(), rect.y() + rect.height() / 2);
 
             m_animation->setScaleAt(.5, 2, 1);
             break;
-        case TRANSITIONSTART:
-        case TRANSITIONEND:
+        case TransitionStart:
+        case TransitionEnd:
             polygon << QPoint(0, - size * 2);
-            if (mode == TRANSITIONSTART)
+            if (mode == TransitionStart)
                 polygon << QPoint(size * 2, 0);
             else
                 polygon << QPoint(- size * 2, 0);
@@ -7307,7 +7486,7 @@ void CustomTrackView::setTipAnimation(AbstractClipItem *clip, OPERATIONTYPE mode
             m_visualTip = new QGraphicsPolygonItem(polygon);
             ((QGraphicsPolygonItem*) m_visualTip)->setBrush(m_tipColor);
             ((QGraphicsPolygonItem*) m_visualTip)->setPen(m_tipPen);
-            if (mode == TRANSITIONSTART)
+            if (mode == TransitionStart)
                 m_visualTip->setPos(rect.x(), rect.bottom());
             else
                 m_visualTip->setPos(rect.right(), rect.bottom());
@@ -7332,12 +7511,12 @@ bool CustomTrackView::hasAudio(int track) const
     QRectF rect(0, (double)(track * m_tracksHeight + 1), (double) sceneRect().width(), (double)(m_tracksHeight - 1));
     QList<QGraphicsItem *> collisions = scene()->items(rect, Qt::IntersectsItemBoundingRect);
     QGraphicsItem *item;
-    for (int i = 0; i < collisions.count(); i++) {
+    for (int i = 0; i < collisions.count(); ++i) {
         item = collisions.at(i);
         if (!item->isEnabled()) continue;
-        if (item->type() == AVWIDGET) {
+        if (item->type() == AVWidget) {
             ClipItem *clip = static_cast <ClipItem *>(item);
-            if (!clip->isVideoOnly() && (clip->clipType() == AUDIO || clip->clipType() == AV || clip->clipType() == PLAYLIST)) return true;
+            if (!clip->isVideoOnly() && (clip->clipType() == Audio || clip->clipType() == AV || clip->clipType() == Playlist)) return true;
         }
     }
     return false;
@@ -7398,16 +7577,16 @@ EffectsParameterList CustomTrackView::getEffectArgs(const QDomElement &effect)
     if (effect.hasAttribute("in")) parameters.addParam("in", effect.attribute("in"));
     if (effect.hasAttribute("out")) parameters.addParam("out", effect.attribute("out"));
     if (effect.attribute("id") == "region") {
-       QDomNodeList subeffects = effect.elementsByTagName("effect");
-       for (int i = 0; i < subeffects.count(); i++) {
-           QDomElement subeffect = subeffects.at(i).toElement();
-           int subeffectix = subeffect.attribute("region_ix").toInt();
-           parameters.addParam(QString("filter%1").arg(subeffectix), subeffect.attribute("id"));
-           parameters.addParam(QString("filter%1.tag").arg(subeffectix), subeffect.attribute("tag"));
-           parameters.addParam(QString("filter%1.kdenlive_info").arg(subeffectix), subeffect.attribute("kdenlive_info"));
-           QDomNodeList subparams = subeffect.elementsByTagName("parameter");
-           adjustEffectParameters(parameters, subparams, m_document->mltProfile(), QString("filter%1.").arg(subeffectix));
-       }
+        QDomNodeList subeffects = effect.elementsByTagName("effect");
+        for (int i = 0; i < subeffects.count(); ++i) {
+            QDomElement subeffect = subeffects.at(i).toElement();
+            int subeffectix = subeffect.attribute("region_ix").toInt();
+            parameters.addParam(QString("filter%1").arg(subeffectix), subeffect.attribute("id"));
+            parameters.addParam(QString("filter%1.tag").arg(subeffectix), subeffect.attribute("tag"));
+            parameters.addParam(QString("filter%1.kdenlive_info").arg(subeffectix), subeffect.attribute("kdenlive_info"));
+            QDomNodeList subparams = subeffect.elementsByTagName("parameter");
+            adjustEffectParameters(parameters, subparams, m_document->mltProfile(), QString("filter%1.").arg(subeffectix));
+        }
     }
 
     QDomNodeList params = effect.elementsByTagName("parameter");
@@ -7419,10 +7598,10 @@ EffectsParameterList CustomTrackView::getEffectArgs(const QDomElement &effect)
 
 void CustomTrackView::adjustEffectParameters(EffectsParameterList &parameters, QDomNodeList params, MltVideoProfile profile, const QString &prefix)
 {
-  QLocale locale;
-  for (int i = 0; i < params.count(); i++) {
+    QLocale locale;
+    for (int i = 0; i < params.count(); ++i) {
         QDomElement e = params.item(i).toElement();
-       QString paramname = prefix + e.attribute("name");
+        QString paramname = prefix + e.attribute("name");
         if (e.attribute("type") == "geometry" && !e.hasAttribute("fixed")) {
             // effects with geometry param need in / out synced with the clip, request it...
             parameters.addParam("_sync_in_out", "1");
@@ -7459,7 +7638,7 @@ void CustomTrackView::adjustEffectParameters(EffectsParameterList &parameters, Q
             QTextStream txtNeu(&neu);
             if (values.size() > 0)
                 txtNeu << (int)values[0].toDouble();
-            for (int i = 0; i < separators.size() && i + 1 < values.size(); i++) {
+            for (int i = 0; i < separators.size() && i + 1 < values.size(); ++i) {
                 txtNeu << separators[i];
                 txtNeu << (int)(values[i+1].toDouble());
             }
@@ -7493,14 +7672,14 @@ void CustomTrackView::updateTrackNames(int track, bool added)
     int audioTracks = 0;
     for (int i = max - 1; i >= 0; --i) {
         TrackInfo info = tracks.at(i);
-        if (info.type == VIDEOTRACK)
+        if (info.type == VideoTrack)
             videoTracks++;
         else
             audioTracks++;
 
         if (i <= docTrack) {
-            QString type = (info.type == VIDEOTRACK ? "Video " : "Audio ");
-            int typeNumber = (info.type == VIDEOTRACK ? videoTracks : audioTracks);
+            QString type = (info.type == VideoTrack ? "Video " : "Audio ");
+            int typeNumber = (info.type == VideoTrack ? videoTracks : audioTracks);
 
             if (added) {
                 if (i == docTrack || info.trackName == type + QString::number(typeNumber - 1)) {
@@ -7555,7 +7734,7 @@ void CustomTrackView::slotRefreshThumbs(const QString &id, bool resetThumbs)
     QList<QGraphicsItem *> list = scene()->items();
     ClipItem *clip = NULL;
     for (int i = 0; i < list.size(); ++i) {
-        if (list.at(i)->type() == AVWIDGET) {
+        if (list.at(i)->type() == AVWidget) {
             clip = static_cast <ClipItem *>(list.at(i));
             if (clip->clipProducer() == id) {
                 clip->refreshClip(true, resetThumbs);
@@ -7568,7 +7747,7 @@ void CustomTrackView::adjustEffects(ClipItem* item, ItemInfo oldInfo, QUndoComma
 {
     QMap<int, QDomElement> effects = item->adjustEffectsToDuration(m_document->width(), m_document->height(), oldInfo);
 
-    if (effects.count()) {
+    if (!effects.isEmpty()) {
         QMap<int, QDomElement>::const_iterator i = effects.constBegin();
         while (i != effects.constEnd()) {
             new EditEffectCommand(this, m_document->tracksCount() - item->track(), item->startPos(), i.value(), item->effect(i.key()), i.value().attribute("kdenlive_ix").toInt(), true, true, command);
@@ -7598,41 +7777,41 @@ void CustomTrackView::slotGotFilterJobResults(const QString &/*id*/, int startPo
         EditEffectCommand *command = new EditEffectCommand(this, m_document->tracksCount() - clip->track(), clip->startPos(), effect, newEffect, clip->selectedEffectIndex(), true, true);
         m_commandStack->push(command);
         emit clipItemSelected(clip);
-    }    
+    }
 }
 
 
-void CustomTrackView::slotImportClipKeyframes(GRAPHICSRECTITEM type)
+void CustomTrackView::slotImportClipKeyframes(GraphicsRectItem type)
 {
     ClipItem *item = NULL;
-    if (type == TRANSITIONWIDGET) {
-       // We want to import keyframes to a transition
-       if (!m_selectionGroup) {
-           emit displayMessage(i18n("You need to select one clip and one transition"), ErrorMessage);
-           return;
-       }
-       // Make sure there is no collision
-       QList<QGraphicsItem *> children = m_selectionGroup->childItems();
-       for (int i = 0; i < children.count(); i++) {
-           if (children.at(i)->type() == AVWIDGET) {
-               item = (ClipItem*) children.at(i);
-               break;
-           }
-       }
+    if (type == TransitionWidget) {
+        // We want to import keyframes to a transition
+        if (!m_selectionGroup) {
+            emit displayMessage(i18n("You need to select one clip and one transition"), ErrorMessage);
+            return;
+        }
+        // Make sure there is no collision
+        QList<QGraphicsItem *> children = m_selectionGroup->childItems();
+        for (int i = 0; i < children.count(); ++i) {
+            if (children.at(i)->type() == AVWidget) {
+                item = (ClipItem*) children.at(i);
+                break;
+            }
+        }
     }
     else {
-       // Import keyframes from current clip to its effect
-       if (m_dragItem) item = static_cast<ClipItem*> (m_dragItem);
+        // Import keyframes from current clip to its effect
+        if (m_dragItem) item = static_cast<ClipItem*> (m_dragItem);
     }
     
     if (!item) {
-       emit displayMessage(i18n("No clip found"), ErrorMessage);
-       return;
+        emit displayMessage(i18n("No clip found"), ErrorMessage);
+        return;
     }
     QMap <QString, QString> data = item->baseClip()->analysisData();
     if (data.isEmpty()) {
-       emit displayMessage(i18n("No keyframe data found in clip"), ErrorMessage);
-       return;
+        emit displayMessage(i18n("No keyframe data found in clip"), ErrorMessage);
+        return;
     }
     QPointer<QDialog> d = new QDialog(this);
     Ui::ImportKeyframesDialog_UI ui;
@@ -7642,15 +7821,15 @@ void CustomTrackView::slotImportClipKeyframes(GRAPHICSRECTITEM type)
     int ix = 0;
     QMap<QString, QString>::const_iterator i = data.constBegin();
     while (i != data.constEnd()) {
-       ui.data_list->insertItem(ix, i.key());
-       ui.data_list->setItemData(ix, i.value(), Qt::UserRole);
-       ++i;
-       ix++;
+        ui.data_list->insertItem(ix, i.key());
+        ui.data_list->setItemData(ix, i.value(), Qt::UserRole);
+        ++i;
+        ix++;
     }
 
     if (d->exec() != QDialog::Accepted) {
-       delete d;
-       return;
+        delete d;
+        return;
     }
     QString keyframeData = ui.data_list->itemData(ui.data_list->currentIndex()).toString();
     
@@ -7663,36 +7842,38 @@ void CustomTrackView::slotImportClipKeyframes(GRAPHICSRECTITEM type)
     newGeometry.insert(gitem);
     int pos = offset + 1;
     while (!geometry.next_key(&gitem, pos)) {
-       pos = gitem.frame();
-       gitem.frame(pos - offset);
-       pos++;
-       newGeometry.insert(gitem);
+        pos = gitem.frame();
+        gitem.frame(pos - offset);
+        pos++;
+        newGeometry.insert(gitem);
     }
     QStringList keyframeList = QString(newGeometry.serialise()).split(';', QString::SkipEmptyParts);
     
     QString result;
     if (ui.import_position->isChecked()) {
-       if (ui.import_size->isChecked()) {
-           foreach(QString key, keyframeList) {
-               if (key.count(':') > 1) result.append(key.section(':', 0, 1));
-               else result.append(key);
-               result.append(';');
-           }
-       }
-       else {
-           foreach(QString key, keyframeList) {
-               result.append(key.section(':', 0, 0));
-               result.append(';');
-           }
-       }
+        if (ui.import_size->isChecked()) {
+            foreach(QString key, keyframeList) {
+                if (key.count(':') > 1) result.append(key.section(':', 0, 1));
+                else result.append(key);
+                result.append(';');
+            }
+        }
+        else {
+            foreach(QString key, keyframeList) {
+                result.append(key.section(':', 0, 0));
+                result.append(';');
+            }
+        }
     }
     else if (ui.import_size->isChecked()) {
-       foreach(QString key, keyframeList) {
-           result.append(key.section(':', 1, 1));
-           result.append(';');
-       }
+        foreach(QString key, keyframeList) {
+            result.append(key.section(':', 1, 1));
+            result.append(';');
+        }
     }
     emit importKeyframes(type, result, ui.limit_keyframes->isChecked() ? ui.max_keyframes->value() : -1);
     delete d;
 }
 
+
+#include "customtrackview.moc"