Skip to content

Partial revert of #3715 since double click on group should expand/collapse it#3834

Merged
Siedlerchr merged 1 commit into
masterfrom
partialRevert3715
Mar 12, 2018
Merged

Partial revert of #3715 since double click on group should expand/collapse it#3834
Siedlerchr merged 1 commit into
masterfrom
partialRevert3715

Conversation

@tobiasdiez

Copy link
Copy Markdown
Member

This PR reverts the part of #3715, which introduced that a double click on a group opens the edit window. However, the default behavior is that double click expands or collapse the subtree and users probably profit from the default behavior more than from the easier way to edit a group.

Fixes (or better closes) JabRef#277.


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 11, 2018
@Siedlerchr Siedlerchr merged commit 9b310a7 into master Mar 12, 2018
@Siedlerchr Siedlerchr deleted the partialRevert3715 branch March 12, 2018 09:54
@Siedlerchr

Copy link
Copy Markdown
Member

Find this behaviour also more useful. You shoudl also cherry-pick that commit in maintable-beta

@tobiasdiez

Copy link
Copy Markdown
Member Author

Why cherry-pick? I thought we agreed to merge master regularly into maintable?

Siedlerchr added a commit that referenced this pull request Mar 14, 2018
…drop

* upstream/maintable-beta: (97 commits)
  Update Eclipse style to intellij style (#3827)
  fix some not on fx thread dialogs in preferences
  fix inital save error Convert last dialog to javafx refactor exception
  Merge changes for renamed actions
  New Crowdin translations (#3835)
  Partial revert of #3715 since double click on group should expand/collapse it (#3834)
  Add Spanish translations (#3833)
  update applicationsinsights
  update javafxsvg to 1.3.0
  Disable FTP Download on CI
  Apply copy linked files dialog fix from master
  Show dialog when copy files did not found file (#3826)
  Remove customjfx (#3779)
  Disable FTP Download on CI
  Remove color customization for maintable from preferences (#3808)
  Remove erroneous escape character (#3831)
  New translations JabRef_en.properties (French)
  New translations JabRef_en.properties (Tagalog)
  New translations JabRef_en.properties (Italian)
  New translations JabRef_en.properties (Indonesian)
  ...

# Conflicts:
#	src/main/java/org/jabref/gui/maintable/MainTable.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants