[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [kdenlive] src/timeline: Fix undo grouping broken
From:       Jean-Baptiste Mardelle <jb () kdenlive ! org>
Date:       2016-11-15 22:51:38
Message-ID: E1c6mZm-0004Ox-Ou () code ! kde ! org
[Download RAW message or body]

Git commit 12217947db4b232463cfb87d400f63a0453fbd22 by Jean-Baptiste Mardelle.
Committed on 15/11/2016 at 22:51.
Pushed by mardelle into branch 'master'.

Fix undo grouping broken
CCBUG: 370653

M  +3    -3    src/timeline/customtrackview.cpp

http://commits.kde.org/kdenlive/12217947db4b232463cfb87d400f63a0453fbd22

diff --git a/src/timeline/customtrackview.cpp b/src/timeline/customtrackview.cpp
index 0e9bdc6..0d112fc 100644
--- a/src/timeline/customtrackview.cpp
+++ b/src/timeline/customtrackview.cpp
@@ -4405,7 +4405,7 @@ void CustomTrackView::groupClips(bool group, \
QList<QGraphicsItem *> itemList, bo  // Expand groups
     int max = itemList.count();
     for (int i = 0; i < max; ++i) {
-        if (itemList.at(i)->type() == GroupWidget) {
+        if (itemList.at(i)->type() == GroupWidget && itemList.at(i) != \
m_selectionGroup) {  if (!existingGroups.contains(itemList.at(i))) {
                 existingGroups << itemList.at(i);
             }
@@ -4513,14 +4513,14 @@ void CustomTrackView::doGroupClips(QList <ItemInfo> \
clipInfos, QList <ItemInfo>  QList <QGraphicsItem *>list;
     for (int i = 0; i < clipInfos.count(); ++i) {
         ClipItem *clip = getClipItemAtStart(clipInfos.at(i).startPos, \
                clipInfos.at(i).track);
-        if (clip) {
+        if (clip && !list.contains(clip)) {
             list.append(clip);
             //clip->setSelected(true);
         }
     }
     for (int i = 0; i < transitionInfos.count(); ++i) {
         Transition *clip = getTransitionItemAt(transitionInfos.at(i).startPos, \
                transitionInfos.at(i).track);
-        if (clip) {
+        if (clip && !list.contains(clip)) {
             list.append(clip);
             //clip->setSelected(true);
         }


[prev in list] [next in list] [prev in thread] [next in thread]