fix for #633

haxeui · Sep 23, 2024 · dcb9c92 · dcb9c92
1 parent 22636f2
commit dcb9c92
Showing 1 changed file with 14 additions and 16 deletions.
diff --git a/haxe/ui/core/Component.hx b/haxe/ui/core/Component.hx
@@ -660,31 +660,29 @@ class Component extends ComponentImpl
     }
 
     private function assignPositionClasses(invalidate:Bool = true) {
-        if (childComponents.length == 1) {
-            childComponents[0].addClasses(["first", "last"], invalidate);
-            return;
-        }
-        var effectiveChildCount = 0;
-        for (i in 0...childComponents.length) {
-            var c = childComponents[i];
-            if (!c.includeInLayout) {
+        var effectiveChildren = [];
+        for (c in childComponents) {
+            if (!c.includeInLayout || c.hidden) {
                 continue;
             }
-            effectiveChildCount++;
+            effectiveChildren.push(c);
+        }
+
+        if (effectiveChildren.length == 1) {
+            effectiveChildren[0].addClasses(["first", "last"], invalidate);
+            return;
         }
+
         var n = 0;
-        for (i in 0...childComponents.length) {
-            var c = childComponents[i];
-            if (!c.includeInLayout) {
-                continue;
-            }
-            if (i == 0) {
+        for (c in effectiveChildren) {
+            if (n == 0) {
                 c.swapClass("first", "last", invalidate);
-            } else if (i == effectiveChildCount - 1) {
+            } else if (n == effectiveChildren.length - 1) {
                 c.swapClass("last", "first", invalidate);
             } else {
                 c.removeClasses(["first", "last"], invalidate);
             }
+
             n++;
         }
     }