[ui] AttributeEditor: remove SortFilterDelegateModel

Usage of DelegateModel for model filtering has not proven to be the most stable solution, and might be responsible for random crashes happening during engine's garbage collection.  Implement Loader-based alternative:
* first delegate is a Loader which creates the AttributeItemDelegate if necessary
* compensate spacing using negative height when element is hidden

Author: yann-lty

meshroom/ui/qml/GraphEditor/AttributeEditor.qml

@@ -19,29 +19,23 @@ ListView {
 
     implicitHeight: contentHeight
 
-    clip: true
     spacing: 2
+    clip: true
     ScrollBar.vertical: ScrollBar { id: scrollBar }
 
-    model: SortFilterDelegateModel {
+    model: attributes
 
-        model: attributes
-        filterRole: GraphEditorSettings.showAdvancedAttributes ? "" : "advanced"
-        filterValue: false
-
-        function modelData(item, roleName) {
-            return item.model.object.desc[roleName]
-        }
+    delegate: Loader {
+        active: !object.desc.advanced || GraphEditorSettings.showAdvancedAttributes
+        visible: active
+        height: item ? item.implicitHeight : -spacing // compensate for spacing if item is hidden
 
-        Component {
-            id: delegateComponent
-            AttributeItemDelegate {
-                width: ListView.view.width - scrollBar.width
-                readOnly: root.readOnly
-                labelWidth: root.labelWidth
-                attribute: object
-                onDoubleClicked: root.attributeDoubleClicked(mouse, attr)
-            }
+        sourceComponent: AttributeItemDelegate {
+            width: root.width - scrollBar.width
+            readOnly: root.readOnly
+            labelWidth: root.labelWidth
+            attribute: object
+            onDoubleClicked: root.attributeDoubleClicked(mouse, attr)
         }
     }