Fixed potential issue where closing latest notification could fail when dismissed

Instead of getting the first/last notification in the AnimatedList, we get the first/last non-destroyed item

Fixed potential issue where closing latest notification could fail when dismissed
42d06db3 · Erik Reider · 8112744d · 42d06db3
Commit 42d06db3 authored Jan 02, 2026 by Erik Reider
Hide whitespace changes
Inline Side-by-side

Showing with 14 additions and 6 deletions

animatedList.vala src/animatedList/animatedList.vala +14 -6

No files found.
--- a/src/animatedList/animatedList.vala
+++ b/src/animatedList/animatedList.vala
@@ -762,17 +762,25 @@ public class AnimatedList : Gtk.Widget, Gtk.Scrollable {
    }

    public unowned AnimatedListItem ?get_first_item () {
-        if (children.is_empty ()) {
-            return null;
+        for (unowned List<AnimatedListItem> ?link = children.first ();
+             link != null;
+             link = link.next) {
+            if (link.data != null && !link.data.destroying) {
+                return link.data;
+            }
        }
-        return children.first ().data;
+        return null;
    }

    public unowned AnimatedListItem ?get_last_item () {
-        if (children.is_empty ()) {
-            return null;
+        for (unowned List<AnimatedListItem> ?link = children.last ();
+             link != null;
+             link = link.prev) {
+            if (link.data != null && !link.data.destroying) {
+                return link.data;
+            }
        }
-        return children.last ().data;
+        return null;
    }

    private bool should_card_animate () {