Skip to content

Commit 1205212

Browse files
authored
[release/3.6] 修复使用 JavaFX 25 时游戏下载界面布局错误的问题 (#4533)
#4508
1 parent 0d9a275 commit 1205212

2 files changed

Lines changed: 14 additions & 3 deletions

File tree

HMCL/src/main/java/org/jackhuang/hmcl/ui/InstallerItem.java

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
import javafx.beans.binding.Bindings;
2323
import javafx.beans.property.ObjectProperty;
2424
import javafx.beans.property.SimpleObjectProperty;
25+
import javafx.beans.value.ChangeListener;
2526
import javafx.css.PseudoClass;
2627
import javafx.geometry.Insets;
2728
import javafx.geometry.Pos;
@@ -312,17 +313,24 @@ public InstallerItem[] getLibraries() {
312313
private static final class InstallerItemSkin extends SkinBase<InstallerItem> {
313314
private static final PseudoClass LIST_ITEM = PseudoClass.getPseudoClass("list-item");
314315
private static final PseudoClass CARD = PseudoClass.getPseudoClass("card");
315-
private static final WeakListenerHolder holder = new WeakListenerHolder();
316+
317+
@SuppressWarnings({"FieldCanBeLocal", "unused"})
318+
private final ChangeListener<Number> holder;
316319

317320
InstallerItemSkin(InstallerItem control) {
318321
super(control);
319322

320323
Pane pane;
321324
if (control.style == Style.CARD) {
322325
pane = new VBox();
323-
holder.add(FXUtils.onWeakChange(pane.widthProperty(), v -> FXUtils.setLimitHeight(pane, v.doubleValue() * 0.7)));
326+
327+
if (FXUtils.JAVAFX_MAJOR_VERSION == 8)
328+
holder = FXUtils.onWeakChange(pane.widthProperty(), v -> FXUtils.setLimitHeight(pane, v.doubleValue() * 0.7));
329+
else
330+
holder = FXUtils.onWeakChangeAndOperate(pane.widthProperty(), v -> FXUtils.setLimitHeight(pane, v.doubleValue() * 0.7));
324331
} else {
325332
pane = new HBox();
333+
holder = null;
326334
}
327335
pane.getStyleClass().add("installer-item");
328336
RipplerContainer container = new RipplerContainer(pane);

HMCL/src/main/java/org/jackhuang/hmcl/ui/download/AbstractInstallersPage.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ protected InstallersPageSkin(AbstractInstallersPage control) {
124124
libraryPane.setVgap(16);
125125
libraryPane.setHgap(16);
126126

127-
if (libraries.length <= 8) {
127+
if (FXUtils.JAVAFX_MAJOR_VERSION < 25 && libraries.length <= 8) {
128128
BorderPane.setMargin(libraryPane, new Insets(16, 0, 16, 0));
129129
root.setCenter(libraryPane);
130130
} else {
@@ -133,6 +133,9 @@ protected InstallersPageSkin(AbstractInstallersPage control) {
133133
scrollPane.setFitToHeight(true);
134134
BorderPane.setMargin(scrollPane, new Insets(16, 0, 16, 0));
135135
root.setCenter(scrollPane);
136+
137+
if (libraries.length <= 8)
138+
scrollPane.setVbarPolicy(ScrollPane.ScrollBarPolicy.NEVER);
136139
}
137140
}
138141

0 commit comments

Comments
 (0)