fix(hui): hare buttons flowpane sizing

software-challenge · Jul 4, 2024 · b6f06af · b6f06af
1 parent 7e2c5d2
commit b6f06af
Showing 1 changed file with 6 additions and 9 deletions.
diff --git a/src/main/kotlin/sc/gui/view/HuIBoard.kt b/src/main/kotlin/sc/gui/view/HuIBoard.kt
@@ -6,14 +6,14 @@ import javafx.geometry.HPos
 import javafx.geometry.Orientation
 import javafx.geometry.Point2D
 import javafx.geometry.Pos
+import javafx.scene.Group
 import javafx.scene.Node
 import javafx.scene.control.Button
 import javafx.scene.control.Label
 import javafx.scene.control.Tooltip
 import javafx.scene.effect.ColorAdjust
 import javafx.scene.input.KeyEvent
 import javafx.scene.layout.*
-import javafx.scene.shape.Rectangle
 import javafx.util.Duration
 import sc.api.plugins.Team
 import sc.gui.AppStyle
@@ -256,17 +256,14 @@ class HuIBoard: GameBoard<GameState>() {
                     if(currentPos + maxAdvance < targetPos || state.checkAdvance(distance) != null)
                         return@forEachIndexed
                     val flow = FlowPane(Orientation.HORIZONTAL).apply {
-                        maxWidthProperty().bind(graphicSize)
-                        clip = Rectangle().apply {
-                            widthProperty().bind(graphicSize)
-                            heightProperty().bind(graphicSize)
-                        }
-                        hgap = AppStyle.miniSpacing
-                        vgap = AppStyle.miniSpacing
+                        this.prefWidthProperty().bind(graphicSize)
+                        this.maxWidthProperty().bind(graphicSize)
+                        this.hgap = AppStyle.miniSpacing
+                        this.vgap = AppStyle.miniSpacing
                     }
                     var totalCards = 0
                     state.possibleCardMoves(distance)?.also {
-                        putOnPosition(flow, targetPos)
+                        putOnPosition(Group(Group(flow).apply { this.isManaged = false }), targetPos)
                         totalCards = it.sumOf { it.getCards().size }
                     }?.forEach { advance ->
                         val cards = advance.getCards()