Skip to content

Customjfxpanel#3128

Merged
lenhard merged 14 commits into
masterfrom
customjfxpanel
Aug 21, 2017
Merged

Customjfxpanel#3128
lenhard merged 14 commits into
masterfrom
customjfxpanel

Conversation

@Siedlerchr

@Siedlerchr Siedlerchr commented Aug 18, 2017

Copy link
Copy Markdown
Member
  • Change in CHANGELOG.md described
  • Tests created for changes
  • Screenshots added (for bigger UI changes)
  • Manually tested changed features in running JabRef
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)
  • If you changed the localization: Did you run gradle localizationUpdate?

* upstream/master:
  Move event processing from source tab into entry editor (#3116)
  Update Menu_ja.properties (#3125)
  Update JabRef_ja.properties (#3126)
  Localization: French: Translation of new entries (#3124)
  Fix changelog - no date for fixes
  Fix markdown
  Additional filter functionality for Check Integrity popup (#3120)
  Workround: make snapcraft find jabref.desktop
  Enable publishing so that users do not need to execute --devmode
  Update jabref.desktop
  Enable publishing so that users do not need to execute --devmode
  Release beta version also on snapcraft
  Workround: make snapcraft find jabref.desktop
  Fix branch name
  Use newer image
  Set executable for all (and not for owner only)
  Add comment on version replacement
  Enable creation of an Ubuntu snap
@Siedlerchr Siedlerchr requested a review from tobiasdiez August 18, 2017 16:34
@@ -0,0 +1,136 @@
/*
* Copyright (c) 2013, Oracle and/or its affiliates. All rights reserved.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@koppor This is a direct copy from the openfjx code, no modifications, Is this compatible with our license?
http://hg.openjdk.java.net/openjfx/8/master/rt/file/tip/modules/swing/src/main/java/javafx/embed/swing/InputMethodSupport.java

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No. It is GPLv2. Not compatible. However, the complete JDK has the linking exception. See http://hg.openjdk.java.net/openjfx/8/master/file/b80799b32c69/LICENSE#l326.

To avoid discussions with Linux distributions, please make a separate repository org.jabref.gui.customjfx with the same license file, a README.md and a gradle build script. The resulting JAR is then copied into https://github.com/JabRef/jabref/tree/master/lib.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah I already thought that it will become a problem. Will do.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Siedlerchr

Copy link
Copy Markdown
Member Author

Tested under linux, works as expected

@koppor koppor added this to the v4.0 milestone Aug 19, 2017
@koppor koppor added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers platform: linux labels Aug 19, 2017
@lenhard

lenhard commented Aug 21, 2017

Copy link
Copy Markdown
Member

It seems that something is missing for the source pane, but otherwise the PR is functional: #3028 (comment)

@Siedlerchr

Siedlerchr commented Aug 21, 2017

Copy link
Copy Markdown
Member Author

I would vote for merging this in as it is. And I will see on thursday why this on the source tab thing is not working. I would create a follow up PR then

@koppor

koppor commented Aug 21, 2017

Copy link
Copy Markdown
Member

OK for me. At least, this heals the issue at most places.

@lenhard

lenhard commented Aug 21, 2017

Copy link
Copy Markdown
Member

Agreed. This repairs most of this problem for 4.0. At least it's now possible to enter the info in the normal entry editor

@lenhard lenhard merged commit d46fd5c into master Aug 21, 2017
@lenhard lenhard deleted the customjfxpanel branch August 21, 2017 17:30
Siedlerchr added a commit that referenced this pull request Aug 22, 2017
* upstream/master: (269 commits)
  Customjfxpanel (#3128)
  Make constants more obvious; improve logging
  Rename variables and make use of Java8 streams
  Re-add "Normalize to BibTex name format" context menu item and menu item tooltips
  Tweak build.gradle and external-libraries
  Work in feedback
  Files can be dropped to entry preview panel to attach a file again.
  Update IntelliJ howto to IntelliJ version 2017.2
  Move event processing from source tab into entry editor (#3116)
  Update Menu_ja.properties (#3125)
  Update JabRef_ja.properties (#3126)
  Localization: French: Translation of new entries (#3124)
  Fix changelog - no date for fixes
  Fix markdown
  Additional filter functionality for Check Integrity popup (#3120)
  Workround: make snapcraft find jabref.desktop
  Enable publishing so that users do not need to execute --devmode
  Update jabref.desktop
  Enable publishing so that users do not need to execute --devmode
  Release beta version also on snapcraft
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: linux 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.

4 participants