diff --git a/CHANGELOG.md b/CHANGELOG.md index 3bdd6c5c3f7..51879c2a3b8 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -62,6 +62,7 @@ We refer to [GitHub issues](https://github.com/JabRef/jabref/issues) by using `# - We fixed an issue where the merge dialog showed the wrong text colour in "Dark" mode [#5516](https://github.com/JabRef/jabref/issues/5516) - We fixed an issue where the author field was not correctly parsed during bibtex key-generation. [#5551](https://github.com/JabRef/jabref/issues/5551) - We fixed an issue where notifications where shown during autosave. [#5555](https://github.com/JabRef/jabref/issues/5555) +- We fixed an issue where the side pane was not remembering its position. [#5615](https://github.com/JabRef/jabref/issues/5615) ### Removed diff --git a/src/main/java/org/jabref/gui/JabRefFrame.java b/src/main/java/org/jabref/gui/JabRefFrame.java index 97165cfa8e6..93337ecb5ef 100644 --- a/src/main/java/org/jabref/gui/JabRefFrame.java +++ b/src/main/java/org/jabref/gui/JabRefFrame.java @@ -410,7 +410,6 @@ private void initLayout() { head.setCenter(createToolbar()); setTop(head); - SplitPane.setResizableWithParent(sidePane, Boolean.FALSE); splitPane.getItems().addAll(sidePane, tabbedPane); // We need to wait with setting the divider since it gets reset a few times during the initial set-up @@ -423,6 +422,8 @@ public void changed(ObservableValue observable, Boolean oldVa EasyBind.subscribe(sidePane.visibleProperty(), visible -> { if (visible) { + // Run SplitPane.setResizableWithParent later to avoid miscalculation during initial layouting + Platform.runLater(() -> SplitPane.setResizableWithParent(sidePane, Boolean.FALSE)); if (!splitPane.getItems().contains(sidePane)) { splitPane.getItems().add(0, sidePane); setDividerPosition();