Allow drag-select keyframes from an object row

Currently only possible to drag-select keyframes from property rows

Task-number: QT3DS-1886
Change-Id: I6673e5ec52a73d48f0c9cd98abc7a53a63915ee0
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>

1 files changed, 3 insertions, 4 deletions

diff --git a/src/Authoring/Studio/Palettes/TimelineGraphicsView/KeyframeManager.cpp b/src/Authoring/Studio/Palettes/TimelineGraphicsView/KeyframeManager.cpp
index e234e99e..570c3981 100644
--- a/src/Authoring/Studio/Palettes/TimelineGraphicsView/KeyframeManager.cpp
+++ b/src/Authoring/Studio/Palettes/TimelineGraphicsView/KeyframeManager.cpp
@@ -173,11 +173,10 @@ void KeyframeManager::selectKeyframesInRect(const QRectF &rect)
     m_scene->rowManager()->clampIndex(idx1);
     m_scene->rowManager()->clampIndex(idx2);
 
-    RowTimeline *rowTimeline;
     for (int i = idx1; i <= idx2; ++i) {
-        rowTimeline = m_scene->rowManager()->rowTimelineAt(i);
+        RowTimeline *rowTimeline = m_scene->rowManager()->rowTimelineAt(i);
 
-        if (rowTimeline != nullptr) {
+        if (rowTimeline) {
             const auto keyframes = rowTimeline->getKeyframesInRange(rect.left(), rect.right());
             for (auto keyframe : keyframes) {
                 if (!m_selectedKeyframes.contains(keyframe)) {
@@ -402,7 +401,7 @@ bool KeyframeManager::hasSelectedKeyframes() const
 
 bool KeyframeManager::hasCopiedKeyframes() const
 {
-    return m_pasteKeyframeCommandHelper != nullptr &&
+    return m_pasteKeyframeCommandHelper &&
            m_pasteKeyframeCommandHelper->HasCopiedKeyframes();
 }