revert: Remove compact mode for chat blocks

ChatView/qml/chatparts/CodeBlock.qml

@@ -28,14 +28,11 @@ Rectangle {
 
     property string code: ""
     property string language: ""
-
+    property bool expanded: false
-    enum DisplayMode { Collapsed, Compact, Expanded }
-    property int displayMode: CodeBlock.DisplayMode.Compact
-    property int compactHeight: 150
 
     property alias codeFontFamily: codeText.font.family
     property alias codeFontSize: codeText.font.pointSize
-    readonly property real headerHeight: copyButton.height + 10
+    readonly property real collapsedHeight: copyButton.height + 10
 
     color: palette.alternateBase
     border.color: root.color.hslLightness > 0.5 ? Qt.darker(root.color, 1.3)
@@ -49,17 +46,6 @@ Rectangle {
         NumberAnimation { duration: 200; easing.type: Easing.InOutQuad }
     }
 
-    implicitHeight: {
-        if (displayMode === CodeBlock.DisplayMode.Collapsed) {
-            return headerHeight
-        } else if (displayMode === CodeBlock.DisplayMode.Compact) {
-            let fullHeight = headerHeight + codeText.implicitHeight + 20
-            return Math.min(fullHeight, headerHeight + compactHeight)
-        } else {
-            return headerHeight + codeText.implicitHeight + 20
-        }
-    }
-
     ChatUtils {
         id: utils
     }
@@ -73,17 +59,9 @@ Rectangle {
         id: header
 
         width: parent.width
-        height: root.headerHeight
+        height: root.collapsedHeight
         cursorShape: Qt.PointingHandCursor
-        onClicked: {
+        onClicked: root.expanded = !root.expanded
-            if (root.displayMode === CodeBlock.DisplayMode.Collapsed) {
-                root.displayMode = CodeBlock.DisplayMode.Compact
-            } else if (root.displayMode === CodeBlock.DisplayMode.Compact) {
-                root.displayMode = CodeBlock.DisplayMode.Collapsed
-            } else {
-                root.displayMode = CodeBlock.DisplayMode.Compact
-            }
-        }
 
         Row {
             id: headerRow
@@ -105,81 +83,33 @@ Rectangle {
 
             Text {
                 anchors.verticalCenter: parent.verticalCenter
-                text: root.displayMode === CodeBlock.DisplayMode.Collapsed ? "▶" : "▼"
+                text: root.expanded ? "▼" : "▶"
                 font.pixelSize: 10
                 color: palette.mid
             }
         }
     }
 
-    Item {
+    TextEdit {
-        id: codeWrapper
+        id: codeText
 
         anchors {
             left: parent.left
             right: parent.right
             top: header.bottom
-            bottom: parent.bottom
             margins: 10
-            bottomMargin: expandButton.visible ? expandButton.height + 15 : 10
-        }
-        clip: true
-        visible: root.displayMode !== CodeBlock.DisplayMode.Collapsed
-
-        TextEdit {
-            id: codeText
-
-            width: parent.width
-            text: root.code
-            readOnly: true
-            selectByMouse: true
-            color: root.color.hslLightness > 0.5 ? "black" : "white"
-            wrapMode: Text.WordWrap
-            selectionColor: palette.highlight
-
-            MouseArea {
-                anchors.fill: parent
-                acceptedButtons: Qt.RightButton
-                onClicked: contextMenu.open()
-            }
-        }
-    }
-
-    Rectangle {
-        id: expandButton
-
-        property bool needsExpand: codeText.implicitHeight > compactHeight - 20
-
-        anchors {
-            bottom: parent.bottom
-            left: parent.left
-            right: parent.right
-            bottomMargin: 5
-            leftMargin: 10
-            rightMargin: 10
-        }
-        height: 24
-        radius: 4
-        color: palette.button
-        visible: needsExpand && root.displayMode !== CodeBlock.DisplayMode.Collapsed
-
-        Text {
-            anchors.centerIn: parent
-            text: root.displayMode === CodeBlock.DisplayMode.Expanded ? qsTr("▲ Show less") : qsTr("▼ Show more")
-            font.pixelSize: 11
-            color: palette.buttonText
         }
+        text: root.code
+        readOnly: true
+        selectByMouse: true
+        color: parent.color.hslLightness > 0.5 ? "black" : "white"
+        wrapMode: Text.WordWrap
+        selectionColor: palette.highlight
 
         MouseArea {
             anchors.fill: parent
-            cursorShape: Qt.PointingHandCursor
+            acceptedButtons: Qt.RightButton
-            onClicked: {
+            onClicked: contextMenu.open()
-                if (root.displayMode === CodeBlock.DisplayMode.Expanded) {
-                    root.displayMode = CodeBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = CodeBlock.DisplayMode.Expanded
-                }
-            }
         }
     }
 
@@ -197,26 +127,8 @@ Rectangle {
         Platform.MenuSeparator {}
 
         Platform.MenuItem {
-            text: root.displayMode === CodeBlock.DisplayMode.Collapsed ? qsTr("Expand") : qsTr("Collapse")
+            text: root.expanded ? qsTr("Collapse") : qsTr("Expand")
-            onTriggered: {
+            onTriggered: root.expanded = !root.expanded
-                if (root.displayMode === CodeBlock.DisplayMode.Collapsed) {
-                    root.displayMode = CodeBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = CodeBlock.DisplayMode.Collapsed
-                }
-            }
-        }
-
-        Platform.MenuItem {
-            text: root.displayMode === CodeBlock.DisplayMode.Expanded ? qsTr("Compact view") : qsTr("Full view")
-            enabled: root.displayMode !== CodeBlock.DisplayMode.Collapsed
-            onTriggered: {
-                if (root.displayMode === CodeBlock.DisplayMode.Expanded) {
-                    root.displayMode = CodeBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = CodeBlock.DisplayMode.Expanded
-                }
-            }
         }
     }
 
@@ -241,4 +153,21 @@ Rectangle {
             onTriggered: parent.text = qsTr("Copy")
         }
     }
+
+    states: [
+        State {
+            when: !root.expanded
+            PropertyChanges {
+                target: root
+                implicitHeight: root.collapsedHeight
+            }
+        },
+        State {
+            when: root.expanded
+            PropertyChanges {
+                target: root
+                implicitHeight: header.height + codeText.implicitHeight + 10
+            }
+        }
+    ]
 }

ChatView/qml/chatparts/ThinkingBlock.qml

@@ -24,12 +24,10 @@ Rectangle {
     id: root
 
     property string thinkingContent: ""
+    // property string signature: ""
     property bool isRedacted: false
-
+    property bool expanded: false
-    enum DisplayMode { Collapsed, Compact, Expanded }
+    
-    property int displayMode: ThinkingBlock.DisplayMode.Compact
-    property int compactHeight: 120
-
     property alias headerOpacity: headerRow.opacity
 
     radius: 6
@@ -40,32 +38,13 @@ Rectangle {
         NumberAnimation { duration: 200; easing.type: Easing.InOutQuad }
     }
 
-    implicitHeight: {
-        if (displayMode === ThinkingBlock.DisplayMode.Collapsed) {
-            return header.height
-        } else if (displayMode === ThinkingBlock.DisplayMode.Compact) {
-            let fullHeight = header.height + contentColumn.height + 20
-            return Math.min(fullHeight, header.height + compactHeight)
-        } else {
-            return header.height + contentColumn.height + 20
-        }
-    }
-
     MouseArea {
         id: header
 
         width: parent.width
         height: headerRow.height + 10
         cursorShape: Qt.PointingHandCursor
-        onClicked: {
+        onClicked: root.expanded = !root.expanded
-            if (root.displayMode === ThinkingBlock.DisplayMode.Collapsed) {
-                root.displayMode = ThinkingBlock.DisplayMode.Compact
-            } else if (root.displayMode === ThinkingBlock.DisplayMode.Compact) {
-                root.displayMode = ThinkingBlock.DisplayMode.Collapsed
-            } else {
-                root.displayMode = ThinkingBlock.DisplayMode.Compact
-            }
-        }
 
         Row {
             id: headerRow
@@ -88,96 +67,72 @@ Rectangle {
 
             Text {
                 anchors.verticalCenter: parent.verticalCenter
-                text: root.displayMode === ThinkingBlock.DisplayMode.Collapsed ? "▶" : "▼"
+                text: root.expanded ? "▼" : "▶"
                 font.pixelSize: 10
                 color: palette.mid
             }
         }
     }
 
-    Item {
+    Column {
-        id: contentWrapper
+        id: contentColumn
 
         anchors {
             left: parent.left
             right: parent.right
             top: header.bottom
-            bottom: parent.bottom
             margins: 10
-            bottomMargin: expandButton.visible ? expandButton.height + 15 : 10
         }
-        clip: true
+        spacing: 8
-        visible: root.displayMode !== ThinkingBlock.DisplayMode.Collapsed
-
-        Column {
-            id: contentColumn
-
-            width: parent.width
-            spacing: 8
-
-            Text {
-                visible: root.isRedacted
-                width: parent.width
-                text: qsTr("Thinking content was redacted by safety systems")
-                font.pixelSize: 11
-                font.italic: true
-                color: Qt.rgba(0.8, 0.4, 0.4, 1.0)
-                wrapMode: Text.WordWrap
-            }
-
-            TextEdit {
-                id: thinkingText
-
-                visible: !root.isRedacted
-                width: parent.width
-                text: root.thinkingContent
-                readOnly: true
-                selectByMouse: true
-                color: palette.text
-                wrapMode: Text.WordWrap
-                font.family: "monospace"
-                font.pixelSize: 11
-                selectionColor: palette.highlight
-            }
-        }
-    }
-
-    Rectangle {
-        id: expandButton
-
-        property bool needsExpand: contentColumn.height > compactHeight - 20
-
-        anchors {
-            bottom: parent.bottom
-            left: parent.left
-            right: parent.right
-            bottomMargin: 5
-            leftMargin: 10
-            rightMargin: 10
-        }
-        height: 24
-        radius: 4
-        color: palette.button
-        visible: needsExpand && root.displayMode !== ThinkingBlock.DisplayMode.Collapsed
 
         Text {
-            anchors.centerIn: parent
+            visible: root.isRedacted
-            text: root.displayMode === ThinkingBlock.DisplayMode.Expanded ? qsTr("▲ Show less") : qsTr("▼ Show more")
+            width: parent.width
+            text: qsTr("Thinking content was redacted by safety systems")
             font.pixelSize: 11
-            color: palette.buttonText
+            font.italic: true
+            color: Qt.rgba(0.8, 0.4, 0.4, 1.0)
+            wrapMode: Text.WordWrap
         }
 
-        MouseArea {
+        TextEdit {
-            anchors.fill: parent
+            id: thinkingText
-            cursorShape: Qt.PointingHandCursor
+
-            onClicked: {
+            visible: !root.isRedacted
-                if (root.displayMode === ThinkingBlock.DisplayMode.Expanded) {
+            width: parent.width
-                    root.displayMode = ThinkingBlock.DisplayMode.Compact
+            text: root.thinkingContent
-                } else {
+            readOnly: true
-                    root.displayMode = ThinkingBlock.DisplayMode.Expanded
+            selectByMouse: true
-                }
+            color: palette.text
-            }
+            wrapMode: Text.WordWrap
+            font.family: "monospace"
+            font.pixelSize: 11
+            selectionColor: palette.highlight
         }
+
+        // Rectangle {
+        //     visible: root.signature.length > 0 && root.expanded
+        //     width: parent.width
+        //     height: signatureText.height + 10
+        //     color: palette.alternateBase
+        //     radius: 4
+
+        //     Text {
+        //         id: signatureText
+
+        //         anchors {
+        //             left: parent.left
+        //             right: parent.right
+        //             verticalCenter: parent.verticalCenter
+        //             margins: 5
+        //         }
+        //         text: qsTr("Signature: %1").arg(root.signature.substring(0, Math.min(40, root.signature.length)) + "...")
+        //         font.pixelSize: 9
+        //         font.family: "monospace"
+        //         color: palette.mid
+        //         elide: Text.ElideRight
+        //     }
+        // }
     }
 
     MouseArea {
@@ -191,26 +146,8 @@ Rectangle {
         id: contextMenu
 
         Platform.MenuItem {
-            text: root.displayMode === ThinkingBlock.DisplayMode.Collapsed ? qsTr("Expand") : qsTr("Collapse")
+            text: root.expanded ? qsTr("Collapse") : qsTr("Expand")
-            onTriggered: {
+            onTriggered: root.expanded = !root.expanded
-                if (root.displayMode === ThinkingBlock.DisplayMode.Collapsed) {
-                    root.displayMode = ThinkingBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = ThinkingBlock.DisplayMode.Collapsed
-                }
-            }
-        }
-
-        Platform.MenuItem {
-            text: root.displayMode === ThinkingBlock.DisplayMode.Expanded ? qsTr("Compact view") : qsTr("Full view")
-            enabled: root.displayMode !== ThinkingBlock.DisplayMode.Collapsed
-            onTriggered: {
-                if (root.displayMode === ThinkingBlock.DisplayMode.Expanded) {
-                    root.displayMode = ThinkingBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = ThinkingBlock.DisplayMode.Expanded
-                }
-            }
         }
     }
 
@@ -225,4 +162,22 @@ Rectangle {
                                                                 : Qt.lighter(palette.alternateBase, 1.3))
         radius: root.radius
     }
+
+    states: [
+        State {
+            when: !root.expanded
+            PropertyChanges {
+                target: root
+                implicitHeight: header.height
+            }
+        },
+        State {
+            when: root.expanded
+            PropertyChanges {
+                target: root
+                implicitHeight: header.height + contentColumn.height + 20
+            }
+        }
+    ]
 }
+

ChatView/qml/chatparts/ToolBlock.qml

@@ -24,10 +24,7 @@ Rectangle {
     id: root
 
     property string toolContent: ""
-
+    property bool expanded: false
-    enum DisplayMode { Collapsed, Compact, Expanded }
-    property int displayMode: ToolBlock.DisplayMode.Compact
-    property int compactHeight: 120
 
     property alias headerOpacity: headerRow.opacity
 
@@ -43,32 +40,13 @@ Rectangle {
         NumberAnimation { duration: 200; easing.type: Easing.InOutQuad }
     }
 
-    implicitHeight: {
-        if (displayMode === ToolBlock.DisplayMode.Collapsed) {
-            return header.height
-        } else if (displayMode === ToolBlock.DisplayMode.Compact) {
-            let fullHeight = header.height + contentColumn.height + 20
-            return Math.min(fullHeight, header.height + compactHeight)
-        } else {
-            return header.height + contentColumn.height + 20
-        }
-    }
-
     MouseArea {
         id: header
 
         width: parent.width
         height: headerRow.height + 10
         cursorShape: Qt.PointingHandCursor
-        onClicked: {
+        onClicked: root.expanded = !root.expanded
-            if (root.displayMode === ToolBlock.DisplayMode.Collapsed) {
-                root.displayMode = ToolBlock.DisplayMode.Compact
-            } else if (root.displayMode === ToolBlock.DisplayMode.Compact) {
-                root.displayMode = ToolBlock.DisplayMode.Collapsed
-            } else {
-                root.displayMode = ToolBlock.DisplayMode.Compact
-            }
-        }
 
         Row {
             id: headerRow
@@ -90,84 +68,36 @@ Rectangle {
 
             Text {
                 anchors.verticalCenter: parent.verticalCenter
-                text: root.displayMode === ToolBlock.DisplayMode.Collapsed ? "▶" : "▼"
+                text: root.expanded ? "▼" : "▶"
                 font.pixelSize: 10
                 color: palette.mid
             }
         }
     }
 
-    Item {
+    Column {
-        id: contentWrapper
+        id: contentColumn
 
         anchors {
             left: parent.left
             right: parent.right
             top: header.bottom
-            bottom: parent.bottom
             margins: 10
-            bottomMargin: expandButton.visible ? expandButton.height + 15 : 10
         }
-        clip: true
+        spacing: 8
-        visible: root.displayMode !== ToolBlock.DisplayMode.Collapsed
 
-        Column {
+        TextEdit {
-            id: contentColumn
+            id: resultText
 
             width: parent.width
-            spacing: 8
+            text: root.toolResult
-
+            readOnly: true
-            TextEdit {
+            selectByMouse: true
-                id: resultText
+            color: palette.text
-
+            wrapMode: Text.WordWrap
-                width: parent.width
+            font.family: "monospace"
-                text: root.toolResult
-                readOnly: true
-                selectByMouse: true
-                color: palette.text
-                wrapMode: Text.WordWrap
-                font.family: "monospace"
-                font.pixelSize: 11
-                selectionColor: palette.highlight
-            }
-        }
-    }
-
-    Rectangle {
-        id: expandButton
-
-        property bool needsExpand: contentColumn.height > compactHeight - 20
-
-        anchors {
-            bottom: parent.bottom
-            left: parent.left
-            right: parent.right
-            bottomMargin: 5
-            leftMargin: 10
-            rightMargin: 10
-        }
-        height: 24
-        radius: 4
-        color: palette.button
-        visible: needsExpand && root.displayMode !== ToolBlock.DisplayMode.Collapsed
-
-        Text {
-            anchors.centerIn: parent
-            text: root.displayMode === ToolBlock.DisplayMode.Expanded ? qsTr("▲ Show less") : qsTr("▼ Show more")
             font.pixelSize: 11
-            color: palette.buttonText
+            selectionColor: palette.highlight
-        }
-
-        MouseArea {
-            anchors.fill: parent
-            cursorShape: Qt.PointingHandCursor
-            onClicked: {
-                if (root.displayMode === ToolBlock.DisplayMode.Expanded) {
-                    root.displayMode = ToolBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = ToolBlock.DisplayMode.Expanded
-                }
-            }
         }
     }
 
@@ -196,26 +126,8 @@ Rectangle {
         Platform.MenuSeparator {}
 
         Platform.MenuItem {
-            text: root.displayMode === ToolBlock.DisplayMode.Collapsed ? qsTr("Expand") : qsTr("Collapse")
+            text: root.expanded ? qsTr("Collapse") : qsTr("Expand")
-            onTriggered: {
+            onTriggered: root.expanded = !root.expanded
-                if (root.displayMode === ToolBlock.DisplayMode.Collapsed) {
-                    root.displayMode = ToolBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = ToolBlock.DisplayMode.Collapsed
-                }
-            }
-        }
-
-        Platform.MenuItem {
-            text: root.displayMode === ToolBlock.DisplayMode.Expanded ? qsTr("Compact view") : qsTr("Full view")
-            enabled: root.displayMode !== ToolBlock.DisplayMode.Collapsed
-            onTriggered: {
-                if (root.displayMode === ToolBlock.DisplayMode.Expanded) {
-                    root.displayMode = ToolBlock.DisplayMode.Compact
-                } else {
-                    root.displayMode = ToolBlock.DisplayMode.Expanded
-                }
-            }
         }
     }
 
@@ -229,4 +141,22 @@ Rectangle {
                                              : Qt.lighter(palette.alternateBase, 1.3)
         radius: root.radius
     }
+
+
+    states: [
+        State {
+            when: !root.expanded
+            PropertyChanges {
+                target: root
+                implicitHeight: header.height
+            }
+        },
+        State {
+            when: root.expanded
+            PropertyChanges {
+                target: root
+                implicitHeight: header.height + contentColumn.height + 20
+            }
+        }
+    ]
 }