Rewrite the logic for adding/removing/moving rows

This fixes several issues and improve the timeline performance.

Task-number: QT3DS-1496
Task-number: QT3DS-1493
Task-number: QT3DS-1476
Change-Id: I1e838d0107317abd6dc7267a8eb82fa56e076390
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>

1 files changed, 1 insertions, 13 deletions

diff --git a/src/Authoring/Studio/Palettes/TimelineGraphicsView/RowMover.cpp b/src/Authoring/Studio/Palettes/TimelineGraphicsView/RowMover.cpp
index 44a521dd..c63d71fd 100644
--- a/src/Authoring/Studio/Palettes/TimelineGraphicsView/RowMover.cpp
+++ b/src/Authoring/Studio/Palettes/TimelineGraphicsView/RowMover.cpp
@@ -71,21 +71,11 @@ void RowMover::resetInsertionParent(RowTree *newTarget)
     }
 }
 
-int RowMover::sourceIndex() const
-{
-    return m_sourceIndex;
-}
-
 int RowMover::targetIndex() const
 {
     return m_targetIndex;
 }
 
-bool RowMover::movingDown() const
-{
-    return m_targetIndex >= m_sourceIndex;
-}
-
 RowTree *RowMover::sourceRow() const
 {
     return m_sourceRow;
@@ -96,10 +86,9 @@ bool RowMover::isActive()
     return m_active;
 }
 
-void RowMover::start(RowTree *row, int index)
+void RowMover::start(RowTree *row)
 {
     m_sourceRow = row;
-    m_sourceIndex = index;
     m_active = true;
 
     m_sourceRow->setMoveSourceRecursive(true);
@@ -114,7 +103,6 @@ void RowMover::end()
 
         m_active = false;
         m_sourceRow = nullptr;
-        m_sourceIndex = -1;
 
         setVisible(false);
         resetInsertionParent();