LayoutDSL: Fix inconsistent rounding in arrangement position calculat…

…ions Source-Commit: ade472fdc50a18f98a0b5a84ba3e97404b593b08
EssentialGG · Feb 29, 2024 · 84bee44 · 84bee44
1 parent 430372f
commit 84bee44
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/unstable/layoutdsl/src/main/kotlin/gg/essential/elementa/layoutdsl/arrangement.kt b/unstable/layoutdsl/src/main/kotlin/gg/essential/elementa/layoutdsl/arrangement.kt
@@ -7,6 +7,7 @@ import gg.essential.elementa.utils.ObservableAddEvent
 import gg.essential.elementa.utils.ObservableClearEvent
 import gg.essential.elementa.utils.ObservableListEvent
 import gg.essential.elementa.utils.ObservableRemoveEvent
+import gg.essential.elementa.utils.roundToRealPixels
 
 abstract class Arrangement {
     internal lateinit var mainAxis: Axis
@@ -112,16 +113,16 @@ private open class SpacedArrangement(
     }
 
     override fun layoutPositions() {
-        val spacing = getSpacing(boundComponent)
-        var nextStart = boundComponent.getMainAxisStart() + getStartOffset(boundComponent, spacing)
+        val spacing = getSpacing(boundComponent).roundToRealPixels()
+        var nextStart = boundComponent.getMainAxisStart() + getStartOffset(boundComponent, spacing).roundToRealPixels()
         boundComponent.children.forEach {
             lastPosValues[it] = nextStart
             nextStart += it.getMainAxisSize() + spacing
         }
     }
 
     override fun getPadding(child: UIComponent): Float {
-        return if (child === boundComponent.children.last()) 0f else getSpacing(boundComponent)
+        return if (child === boundComponent.children.last()) 0f else getSpacing(boundComponent).roundToRealPixels()
     }
 }