Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor externalprefs #6509

Merged
merged 5 commits into from
May 23, 2020
Merged

Refactor externalprefs #6509

merged 5 commits into from
May 23, 2020

Conversation

calixtus
Copy link
Member

Follow up to #6489
Made some minor cosmetic changes to the ExternalTab.
Everything else should be self explaining.

grafik

  • Change in CHANGELOG.md described (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked documentation: Is the information available and up to date? If not created an issue at https://github.com/JabRef/user-documentation/issues or, even better, submitted a pull request to the documentation repository.

@calixtus calixtus added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label May 22, 2020
…prefs

# Conflicts:
#	src/main/java/org/jabref/preferences/JabRefPreferences.java
#	src/main/java/org/jabref/preferences/PreferencesService.java
Copy link
Member

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

I vaguely remember that there was a reason why there was an extra dialog to specify the detailed information for an application (path + cmd args). Is this still necessary or can this be actually inlined in the preference dialog?

@calixtus
Copy link
Member Author

Only reason why I can think of is because the content depends on the application. Emacs e.g. has an additional textfield for c l arguments etc.

@tobiasdiez
Copy link
Member

ok, if you think it would be easy to integrate, then this would be a valuable improvement.

@tobiasdiez tobiasdiez merged commit 03e3663 into master May 23, 2020
@tobiasdiez tobiasdiez deleted the refactor_externalprefs branch May 23, 2020 22:21
@calixtus calixtus mentioned this pull request May 24, 2020
5 tasks
Siedlerchr added a commit that referenced this pull request May 26, 2020
* upstream/master: (76 commits)
  Fixes missing library properties context menu on library tab (#6508)
  Bump flexmark-ext-gfm-strikethrough from 0.61.30 to 0.61.32
  Bump archunit-junit5-api from 0.13.1 to 0.14.1
  Bump flexmark from 0.61.30 to 0.61.32
  Bump flexmark-ext-gfm-tasklist from 0.61.30 to 0.61.32
  Add javadoc and fix the preview update issue
  Refactor externalprefs (#6509)
  Extend the bib file for better screenshots
  Remove Grobid also from tests
  Fix help file tests
  Update ActionHelper.java
  Adjusted fix by using StateManager for clearing search bar
  fix checkstyle
  Return true in action helper if file is online link
  Reenable caching of gradle
  Refactor BibtexKeyPatternPreferences (#6489)
  Update CHANGELOG.md
  Add changelog entry and remove unnecessary code
  EasyBind revision part two
  Fix Drag and Drop on empty database
  ...
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.

3 participants