Skip to content

Commit

Permalink
Revert "Fix #5615 side pane not remembering position (#5629)"
Browse files Browse the repository at this point in the history
This reverts commit 9ab537f.
  • Loading branch information
LinusDietz committed Dec 3, 2019
1 parent 9d620f1 commit dc9c5e9
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
1 change: 0 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ We refer to [GitHub issues](https://github.com/JabRef/jabref/issues) by using `#
- We fixed visibility issues with the scrollbar and group selection highlight in "Dark" mode, and enabled "Dark" mode for the OpenOffice preview in the style selection window. [#5522](https://github.com/JabRef/jabref/issues/5522)
- 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

Expand Down
3 changes: 1 addition & 2 deletions src/main/java/org/jabref/gui/JabRefFrame.java
Original file line number Diff line number Diff line change
Expand Up @@ -410,6 +410,7 @@ 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
Expand All @@ -422,8 +423,6 @@ public void changed(ObservableValue<? extends Boolean> 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();
Expand Down

0 comments on commit dc9c5e9

Please sign in to comment.