Daniel Brötzmann pushed to branch master at gajim / gajim


Commits:
2a1bad9b by wurstsalat at 2023-03-29T23:03:58+02:00
fix: Update message merged state after deleting a message

Fixes #11438

- - - - -


1 changed file:

- gajim/gtk/conversation/view.py


Changes:

=====================================
gajim/gtk/conversation/view.py
=====================================
@@ -666,8 +666,17 @@ def _reduce_messages_after(self) -> None:
 
     def remove_message(self, log_line_id: int) -> None:
         row = self.get_row_by_log_line_id(log_line_id)
-        if row is not None:
-            row.destroy()
+        if row is None:
+            return
+
+        index = row.get_index()
+        row.destroy()
+        decendant_row = self._list_box.get_row_at_index(index)
+        if isinstance(decendant_row, MessageRow):
+            # Unset possible merged state if we delete a 'top level' message.
+            # Checks for same sender etc. are not necessary, since we simply
+            # unset merged state.
+            decendant_row.set_merged(False)
 
     def scroll_to_message_and_highlight(self, log_line_id: int) -> None:
         highlight_row = None



View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/commit/2a1bad9b5162ae40fa18913151206b28e6dbbc46

-- 
View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/commit/2a1bad9b5162ae40fa18913151206b28e6dbbc46
You're receiving this email because of your account on dev.gajim.org.


_______________________________________________
Commits mailing list
[email protected]
https://lists.gajim.org/cgi-bin/listinfo/commits

Reply via email to