From 49f94aaa8816ba22e29871efafbdfb84c253425e Mon Sep 17 00:00:00 2001
From: Pavel Baksy <beval@centrum.cz>
Date: Wed, 24 Dec 2025 13:40:56 +0100
Subject: [PATCH] Fix template errors - remove invalid cursor-name property and
 set cursor programmatically, remove debug output

---
 src/widgets/history-item.ui |  2 --
 src/widgets/history_item.py | 18 ++++++++++++------
 2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/widgets/history-item.ui b/src/widgets/history-item.ui
index 203c89d..d71e546 100644
--- a/src/widgets/history-item.ui
+++ b/src/widgets/history-item.ui
@@ -143,7 +143,6 @@
                 <property name="halign">fill</property>
                 <property name="margin-top">3</property>
                 <property name="margin-bottom">3</property>
-                <property name="cursor-name">pointer</property>
                 <child>
                   <object class="GtkSeparator">
                     <property name="hexpand">True</property>
@@ -229,7 +228,6 @@
                 <property name="halign">fill</property>
                 <property name="margin-top">3</property>
                 <property name="margin-bottom">3</property>
-                <property name="cursor-name">pointer</property>
                 <child>
                   <object class="GtkSeparator">
                     <property name="hexpand">True</property>
diff --git a/src/widgets/history_item.py b/src/widgets/history_item.py
index 117c4e6..3bae25c 100644
--- a/src/widgets/history_item.py
+++ b/src/widgets/history_item.py
@@ -95,6 +95,10 @@ class HistoryItem(Gtk.Box):
         needs_expander = num_lines > 3 or len(body_str) > 150
         self.request_expander.set_visible(needs_expander)
 
+        # Set cursor to pointer for clickability hint
+        if needs_expander:
+            self.request_expander.set_cursor_from_name("pointer")
+
         # Details - Response
         if self.entry.response:
             self.response_headers_label.set_text(self.entry.response.headers)
@@ -106,6 +110,10 @@ class HistoryItem(Gtk.Box):
             num_lines = response_body.count('\n') + 1
             needs_expander = num_lines > 3 or len(response_body) > 150
             self.response_expander.set_visible(needs_expander)
+
+            # Set cursor to pointer for clickability hint
+            if needs_expander:
+                self.response_expander.set_cursor_from_name("pointer")
         elif self.entry.error:
             self.response_headers_label.set_text("(error)")
             error_text = self.entry.error
@@ -116,6 +124,10 @@ class HistoryItem(Gtk.Box):
             needs_expander = num_lines > 3 or len(error_text) > 150
             self.response_expander.set_visible(needs_expander)
 
+            # Set cursor to pointer for clickability hint
+            if needs_expander:
+                self.response_expander.set_cursor_from_name("pointer")
+
     @Gtk.Template.Callback()
     def on_clicked(self, gesture, n_press, x, y):
         """Toggle expansion when clicked."""
@@ -129,7 +141,6 @@ class HistoryItem(Gtk.Box):
     @Gtk.Template.Callback()
     def on_request_expander_clicked(self, gesture, n_press, x, y):
         """Toggle request body expansion."""
-        print(f"DEBUG: Request expander clicked! Current state: {self.request_expanded}")
         self.request_expanded = not self.request_expanded
 
         if self.request_expanded:
@@ -138,19 +149,16 @@ class HistoryItem(Gtk.Box):
             self.request_body_scroll.set_max_content_height(300)
             self.request_body_scroll.set_min_content_height(300)
             self.request_expander_label.set_text("Collapse request")
-            print("DEBUG: Expanded request to 300px")
         else:
             # Collapse to show only few lines
             # Set min first, then max to avoid assertion errors
             self.request_body_scroll.set_min_content_height(60)
             self.request_body_scroll.set_max_content_height(60)
             self.request_expander_label.set_text("Show full request")
-            print("DEBUG: Collapsed request to 60px")
 
     @Gtk.Template.Callback()
     def on_response_expander_clicked(self, gesture, n_press, x, y):
         """Toggle response body expansion."""
-        print(f"DEBUG: Response expander clicked! Current state: {self.response_expanded}")
         self.response_expanded = not self.response_expanded
 
         if self.response_expanded:
@@ -159,14 +167,12 @@ class HistoryItem(Gtk.Box):
             self.response_body_scroll.set_max_content_height(300)
             self.response_body_scroll.set_min_content_height(300)
             self.response_expander_label.set_text("Collapse response")
-            print("DEBUG: Expanded response to 300px")
         else:
             # Collapse to show only few lines
             # Set min first, then max to avoid assertion errors
             self.response_body_scroll.set_min_content_height(60)
             self.response_body_scroll.set_max_content_height(60)
             self.response_expander_label.set_text("Show full response")
-            print("DEBUG: Collapsed response to 60px")
 
     def toggle_expanded(self):
         """Toggle between collapsed and expanded view."""