Merge pull request #2673 from alicevision/fix/chunksWarnings

[qml] Fix QML warnings related to chunks

Author: yann-lty

meshroom/ui/qml/GraphEditor/ChunksListView.qml

@@ -2,7 +2,7 @@ import QtQuick
 import QtQuick.Controls
 import QtQuick.Layouts
 
-import "common.js" as Common
+import Utils 1.0
 
 /**
  * ChunkListView
@@ -86,7 +86,7 @@ ColumnLayout {
             Rectangle {
                 width: 4
                 height: parent.height
-                color: Common.getChunkColor(parent.chunk)
+                color: Colors.getChunkColor(parent.chunk)
             }
         }
     }

meshroom/ui/qml/GraphEditor/NodeDocumentation.qml

@@ -4,8 +4,6 @@ import QtQuick.Layouts
 
 import Controls 1.0
 
-import "common.js" as Common
-
 /**
  * Displays Node documentation
  */

meshroom/ui/qml/GraphEditor/NodeStatus.qml

@@ -22,7 +22,7 @@ FocusScope {
         clip: true
         anchors.fill: parent
 
-        property string currentFile: (root.currentChunkIndex >= 0) ? root.currentChunk["statusFile"] : ""
+        property string currentFile: (root.currentChunkIndex >= 0 && root.currentChunk !== undefined) ? root.currentChunk["statusFile"] : ""
         property url sourceFile: Filepath.stringToUrl(currentFile)
 
         sourceComponent: statViewerComponent
@@ -90,7 +90,7 @@ FocusScope {
 
                 delegate: Rectangle {
                     color: activePalette.window
-                    width: parent.width
+                    width: statusListView.width
                     height: childrenRect.height
                     RowLayout {
                         width: parent.width

meshroom/ui/qml/GraphEditor/common.js

@@ -48,6 +48,8 @@ QtObject {
     ]
 
     function getChunkColor(chunk, overrides) {
+        if (chunk === undefined)
+            return "transparent"
         if (overrides && chunk.statusName in overrides) {
             return overrides[chunk.statusName]
         } else if (chunk.execModeName === "EXTERN" && chunk.statusName in statusColorsExternOverrides) {