From 58d3540ab745e0dd59d1225a6e7dd17b5a7b442d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Danielle=20For=C3=A9?= <danielle@elementary.io>
Date: Thu, 9 Apr 2026 13:38:34 -0700
Subject: [PATCH 1/2] VAutoHider: cleanup size_allocate

---
 src/Grid/VAutoHider.vala | 44 ++++++++++++++--------------------------
 1 file changed, 15 insertions(+), 29 deletions(-)

diff --git a/src/Grid/VAutoHider.vala b/src/Grid/VAutoHider.vala
index aca35a0d3..d66483bc8 100644
--- a/src/Grid/VAutoHider.vala
+++ b/src/Grid/VAutoHider.vala
@@ -49,57 +49,43 @@ public class Maya.View.VAutoHider : Gtk.Bin {
 
     public override void size_allocate (Gtk.Allocation allocation) {
         base.size_allocate (allocation);
-        int global_height = allocation.height;
-        int children_length = (int)main_box.get_children ().length () - 1;
+        int children_length = (int) main_box.get_children ().length () - 1;
         if (children_length == 0) {
             more_label.hide ();
             return;
         }
 
-        int height = 0;
         int more_label_height;
-        int shown_children = 0;
         more_label.show ();
         more_label.vexpand = false;
         more_label.get_preferred_height (out more_label_height, null);
-        more_label.vexpand = true;
         more_label.hide ();
+
+        int shown_children_height = 0;
+        int hidden_children = 0;
         foreach (var child in main_box.get_children ()) {
-            if (child == more_label)
+            if (child == more_label) {
                 continue;
-
-            bool last = (shown_children == children_length - 1);
+            }
 
             int child_height;
             child.show ();
             child.get_preferred_height (out child_height, null);
-            ((Maya.View.EventButton) child).hide_without_animate ();
-
-            bool should_hide;
-            if (global_height - more_label_height < child_height + height) {
-                should_hide = true;
-                if (last && (global_height >= child_height + height)) {
-                    should_hide = false;
-                }
-            } else {
-                should_hide = false;
-                height += child_height;
-            }
 
-            if (should_hide) {
+            if (allocation.height - more_label_height < shown_children_height + child_height) {
                 ((Maya.View.EventButton) child).hide_without_animate ();
-            } else {
-                ((Maya.View.EventButton) child).show_without_animate ();
-                shown_children++;
+                hidden_children++;
+                continue;
             }
+
+            ((Maya.View.EventButton) child).show_without_animate ();
+            shown_children_height += child_height;
         }
 
-        int more = children_length - shown_children;
-        if (shown_children != children_length && more > 0) {
+        if (hidden_children > 0) {
             more_label.show ();
-            more_label.set_label (_("%u more…").printf ((uint)more));
-        } else {
-            more_label.hide ();
+            more_label.label = _("%i more…").printf (hidden_children);
+            more_label.vexpand = true;
         }
     }
 

From ff4466644a5dacdb9db7d2fa3dffdc63afababbc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Danielle=20For=C3=A9?= <danielle@elementary.io>
Date: Fri, 10 Apr 2026 11:28:53 -0700
Subject: [PATCH 2/2] Fix review comment

---
 src/Grid/VAutoHider.vala | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/Grid/VAutoHider.vala b/src/Grid/VAutoHider.vala
index d66483bc8..f8ea73b8f 100644
--- a/src/Grid/VAutoHider.vala
+++ b/src/Grid/VAutoHider.vala
@@ -49,6 +49,7 @@ public class Maya.View.VAutoHider : Gtk.Bin {
 
     public override void size_allocate (Gtk.Allocation allocation) {
         base.size_allocate (allocation);
+
         int children_length = (int) main_box.get_children ().length () - 1;
         if (children_length == 0) {
             more_label.hide ();
@@ -61,8 +62,8 @@ public class Maya.View.VAutoHider : Gtk.Bin {
         more_label.get_preferred_height (out more_label_height, null);
         more_label.hide ();
 
+        int shown_children = 0;
         int shown_children_height = 0;
-        int hidden_children = 0;
         foreach (var child in main_box.get_children ()) {
             if (child == more_label) {
                 continue;
@@ -72,16 +73,20 @@ public class Maya.View.VAutoHider : Gtk.Bin {
             child.show ();
             child.get_preferred_height (out child_height, null);
 
-            if (allocation.height - more_label_height < shown_children_height + child_height) {
-                ((Maya.View.EventButton) child).hide_without_animate ();
-                hidden_children++;
-                continue;
+            if (shown_children_height + child_height > allocation.height - more_label_height) {
+                var last = shown_children == children_length - 1;
+                if (!last || shown_children_height + child_height > allocation.height) {
+                    ((Maya.View.EventButton) child).hide_without_animate ();
+                    continue;
+                }
             }
 
             ((Maya.View.EventButton) child).show_without_animate ();
+            shown_children++;
             shown_children_height += child_height;
         }
 
+        var hidden_children = children_length - shown_children;
         if (hidden_children > 0) {
             more_label.show ();
             more_label.label = _("%i more…").printf (hidden_children);