fix: Guild Member List & Guild Search Lists are now fully clickable + Grid fixes

src/main/kotlin/com/mineinabyss/guiy/components/Grid.kt

@@ -5,6 +5,7 @@ import com.mineinabyss.guiy.layout.Layout
 import com.mineinabyss.guiy.layout.MeasurePolicy
 import com.mineinabyss.guiy.layout.MeasureResult
 import com.mineinabyss.guiy.modifiers.Modifier
+import kotlin.math.ceil
 
 /**
  * A grid layout component that finds the largest child size, and then places children in a grid based on this size.
@@ -47,22 +48,19 @@ fun gridMeasurePolicy(vertical: Boolean) = MeasurePolicy { measurables, constrai
         constraints.minWidth,
         constraints.minHeight
     ) {}
-
     // Get width and height divisible by cellWidth, cellHeight
     val itemsPerLine =
         if (vertical) constraints.maxWidth / cellWidth
         else constraints.maxHeight / cellHeight
-
     val (width, height) = if (vertical) {
         val w = itemsPerLine * cellWidth
-        val h = ((placeables.size / itemsPerLine) + 1) * cellHeight
+        val h = (ceil(placeables.size / itemsPerLine.toFloat()).toInt()) * cellHeight
         w to h
     } else {
-        val w = ((placeables.size / itemsPerLine) + 1) * cellWidth
+        val w = (ceil(placeables.size / itemsPerLine.toFloat()).toInt())  * cellWidth
         val h = itemsPerLine * cellHeight
         w to h
     }
-
     MeasureResult(width, height) {
         var placeAtX = 0
         var placeAtY = 0

src/main/kotlin/com/mineinabyss/guiy/components/lists/Paginated.kt

@@ -3,6 +3,7 @@ package com.mineinabyss.guiy.components.lists
 import androidx.compose.runtime.*
 import com.mineinabyss.guiy.components.Item
 import com.mineinabyss.guiy.components.Spacer
+import com.mineinabyss.guiy.components.VerticalGrid
 import com.mineinabyss.guiy.jetpack.Alignment
 import com.mineinabyss.guiy.jetpack.Arrangement
 import com.mineinabyss.guiy.layout.Box
@@ -35,14 +36,33 @@ fun <T> Paginated(
     content: @Composable (page: List<T>) -> Unit,
 ) {
     var size by remember { mutableStateOf(Size(0, 0)) }
+    var clearSize by remember { mutableStateOf(Size(0, 0)) }
     val itemsPerPage = size.width * size.height
+
     Box(Modifier.fillMaxSize()) {
         val start = page * itemsPerPage
         val end = (page + 1) * itemsPerPage
         val pageItems = remember(items, start, end) {
             if (start < 0) emptyList()
             else items.subList(start, end.coerceAtMost(items.size))
         }
+
+        // Extract original size of contents
+        Box(Modifier.onSizeChanged{
+            size = it
+        }) {
+            content(pageItems)
+        }
+
+        // Clear out the previous Box
+        Box(Modifier.onSizeChanged{
+            clearSize = it
+        }.fillMaxSize()) {
+            VerticalGrid(){
+                MutableList(clearSize.width * clearSize.height) {Item(null)}
+            }
+        }
+
         NavbarLayout(
             position = navbarPosition,
             navbar = {
@@ -54,9 +74,8 @@ fun <T> Paginated(
                 }
             },
             content = {
-                Box(Modifier.onSizeChanged {
-                    size = it
-                }) {
+                // Actually render the correct amount of items into a box that can fit them including offsets
+                Box(Modifier.fillMaxSize()) {
                     content(pageItems)
                 }
             }

src/main/kotlin/com/mineinabyss/guiy/components/lists/Scrollable.kt

@@ -1,12 +1,12 @@
 package com.mineinabyss.guiy.components.lists
 
 import androidx.compose.runtime.*
+import com.mineinabyss.guiy.components.Item
 import com.mineinabyss.guiy.components.Spacer
+import com.mineinabyss.guiy.components.VerticalGrid
 import com.mineinabyss.guiy.layout.Box
 import com.mineinabyss.guiy.layout.Size
-import com.mineinabyss.guiy.modifiers.Modifier
-import com.mineinabyss.guiy.modifiers.fillMaxSize
-import com.mineinabyss.guiy.modifiers.onSizeChanged
+import com.mineinabyss.guiy.modifiers.*
 import com.mineinabyss.idofront.items.editItemMeta
 import org.bukkit.Material
 import org.bukkit.inventory.ItemStack
@@ -36,15 +36,35 @@ fun <T> Scrollable(
     content: @Composable (page: List<T>) -> Unit,
 ) {
     var size by remember { mutableStateOf(Size(0, 0)) }
+    var clearSize by remember { mutableStateOf(Size(0, 0)) }
+
     val itemsPerLine = if (scrollDirection == ScrollDirection.VERTICAL) size.width else size.height
     val totalLines = if (scrollDirection == ScrollDirection.VERTICAL) size.height else size.width
-    Box(Modifier.fillMaxSize()) {
+
+    Box(Modifier.fillMaxSize().onSizeChanged {println("Exterior box size: $it")}) {
         val start = line * itemsPerLine
         val end = start + (itemsPerLine * totalLines)
         val pageItems = remember(items, start, end) {
             if (start < 0) emptyList()
             else items.subList(start, end.coerceAtMost(items.size))
         }
+
+        // Extract original size of contents
+        Box(Modifier.onSizeChanged{
+            size = it
+        }) {
+            content(pageItems)
+        }
+
+        // Clear out the previous Box
+        Box(Modifier.onSizeChanged{
+            clearSize = it
+        }.fillMaxSize()) {
+            VerticalGrid(){
+                MutableList(clearSize.width * clearSize.height) {Item(null)}
+            }
+        }
+
         NavbarLayout(
             position = navbarPosition,
             navbar = {
@@ -56,12 +76,12 @@ fun <T> Scrollable(
                 }
             },
             content = {
-                Box(Modifier.onSizeChanged {
-                    size = it
-                }) {
+                // Actually render the correct amount of items into a box that can fit them including offsets
+                Box(Modifier.fillMaxSize()) {
                     content(pageItems)
                 }
             }
+
         )
     }
-}
+}

src/main/kotlin/com/mineinabyss/guiy/layout/Layout.kt

@@ -24,7 +24,7 @@ inline fun Layout(
             set(measurePolicy) { this.measurePolicy = it }
             set(renderer) { this.renderer = it }
             //TODO dunno if this works
-            set(canvas) { this.canvas = it}
+            set(canvas) { this.canvas = it }
             set(modifier) { this.modifier = it }
         },
         content = content,