Fix BulletGraph vertical centering of ranges.

KoalaPlot · Oct 5, 2024 · d9778bf · d9778bf
1 parent 32808e9
commit d9778bf
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -22,6 +22,7 @@ to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 ### Fixed
 
 - #81 Add coerceAtLeast to rememberIntLinearAxisModel to fix Zoom range limit when range is small
+- BulletGraph vertical centering of ranges
 
 ### Removed
 

diff --git a/src/commonMain/kotlin/io/github/koalaplot/core/bar/BulletBuilderScope.kt b/src/commonMain/kotlin/io/github/koalaplot/core/bar/BulletBuilderScope.kt
@@ -370,7 +370,9 @@ public class BulletBuilderScope<T>(private val axisModel: LinearAxisModel<T>) wh
                     }
                     HorizontalBarIndicator(SolidColor(Color(shade, shade, shade)))
                 } else {
-                    Box(modifier = Modifier.fillMaxSize()) { range.indicator.invoke() }
+                    Box(contentAlignment = Alignment.CenterStart, modifier = Modifier.fillMaxSize()) {
+                        range.indicator.invoke()
+                    }
                 }
             }
         }
@@ -400,10 +402,9 @@ public class BulletBuilderScope<T>(private val axisModel: LinearAxisModel<T>) wh
                                 Modifier.size(rangeHeight.toDp(), rangeHeight.toDp())
                             }
 
-                        Box(
-                            contentAlignment = Alignment.Center,
-                            modifier = sizeModifier
-                        ) { featuredMeasure.indicator() }
+                        Box(contentAlignment = Alignment.Center, modifier = sizeModifier) {
+                            featuredMeasure.indicator()
+                        }
                     }
                 }[0]
 
@@ -497,7 +498,7 @@ public class BulletBuilderScope<T>(private val axisModel: LinearAxisModel<T>) wh
                 } else {
                     rangeWidth * axisModel.computeOffset(value) - it.width / 2
                 }
-                it.place(xPos.roundToInt(), yPos + rangeHeight / 2 - it.height / 2)
+                it.place(xPos.roundToInt(), yPos)
             }
         }