Fix regression for HLL support

Author: uglide

src/qml/value-editor/ValueTableActions.qml

@@ -54,8 +54,9 @@ ColumnLayout {
                         source: keyTab.keyModel ? Editor.getEditorByTypeString(keyType) : ""
 
                         onLoaded: {
-                            item.state = "add"
+                            item.state = "add"                            
                             item.initEmpty()
+                            keyTab.addRowDialog = addRowDialog
                         }
                     }
                 }

src/qml/value-editor/ValueTabs.qml

@@ -54,6 +54,7 @@ Repeater {
         property var tabButton
         property bool loadingModel: showLoader
         property variant keyModel: keyViewModel
+        property var addRowDialog
 
         onKeyModelChanged: {
             console.log("keyModel changed")
@@ -447,11 +448,7 @@ Repeater {
                                     function clear() {
                                         if (valueEditor.item) {
                                             currentRow = -1
-
-                                            if (valueEditor.item.keyType !== undefined) {
-                                                valueEditor.item.keyType = keyType
-                                            }
-
+                                            valueEditor.item.keyType = Qt.binding(function() { return keyType });
                                             valueEditor.item.reset()
                                         }
                                     }

src/qml/value-editor/editors/MultilineEditor.qml

@@ -413,7 +413,8 @@ Item
                 textRole: "text"
 
                 visible: {
-                    return binaryFlag.visible && qmlUtils.binaryStringLength(root.value) <= appSettings.valueSizeLimit
+                    console.log("keyType:", keyType)
+                    return binaryFlag.visible && keyType != "hyperloglog" && qmlUtils.binaryStringLength(root.value) <= appSettings.valueSizeLimit
                             || root.valueCompression > 0
                 }
 
@@ -505,34 +506,29 @@ Item
                 color: "red"
             }
 
+            BetterButton {
+                iconSource: PlatformUtils.getThemeIcon("add.svg")
+                Layout.alignment: Qt.AlignHCenter
+
+                text: qsTranslate("RESP","Add Element to HLL");
+                visible: keyType === "hyperloglog"
+
+                onClicked: {
+                    keyTab.addRowDialog.open()
+                }
+            }
+
             RowLayout {
                 id: valueEditorToolBar
                 Layout.preferredWidth: isMultiRow ? 200 : 208
                 Layout.maximumWidth: isMultiRow ? 200 : 208
 
-                visible: showToolBar                
+                visible: showToolBar
 
                 RowLayout {
                     Layout.fillWidth: true
                     Layout.preferredWidth: 98
 
-                    ImageButton {
-                        iconSource: PlatformUtils.getThemeIcon("add.svg")
-                        implicitWidth: imgBtnWidth
-                        implicitHeight: imgBtnHeight
-                        imgWidth: imgBtnWidth
-                        imgHeight: imgBtnHeight
-
-                        Layout.alignment: Qt.AlignHCenter
-
-                        tooltip: qsTranslate("RESP","Add Element to HLL");
-                        visible: keyType === "hyperloglog"
-
-                        onClicked: {
-                            addRowDialog.open()
-                        }
-                    }
-
                     ImageButton {
                         id: copyValueToClipboardBtn
                         iconSource: PlatformUtils.getThemeIcon("copy_2.svg")