Skip to content

updating#1

Merged
mpele merged 15 commits into
mpele:masterfrom
JabRef:master
Jul 10, 2017
Merged

updating#1
mpele merged 15 commits into
mpele:masterfrom
JabRef:master

Conversation

@mpele

@mpele mpele commented Jul 10, 2017

Copy link
Copy Markdown
Owner
  • 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?

jhshinn and others added 15 commits July 3, 2017 10:50
* Fix the function "Edit - Copy BibTeX key and link" to pass a hyperlink rather than an HTML statement.

* Update CHANGELOG.md

* Remove comment about non-working code

* Reorder the import statement using IntelliJ Configuration.
Rename Escape to ESC for javafx
Renamed Enum variables as key for the constant was amibiuous
* Try to fix other keybindings in entry editor

Try to match awt key events with keycombinations

* Finally fix keybinding
Remove + signs to make swing work again
Add + sign only in binding checker

* Extract method for event converting in keybinding repo
Adapt test
Add + between keybindings
* Propagate changes to the group tree to the MetaData

* Add Changelog entry
- Fix regression bug #1182
- Pass language and country as separate args for Locale constructor
* Remove <pre> tag from entries fetched using MathSciNet

* Fix tests
* Check for different editions in the duplicate check

* edited changelog

* Code Conventions
@mpele mpele merged commit c8d66cb into mpele:master Jul 10, 2017
mpele pushed a commit that referenced this pull request Jul 26, 2017
* Updated maven repo

* #1 fixed, in case the old file is not there we just save the new file

* #1 adjustments

* #1 adjustments

* fixes for checkstyle and improvements on file path verification
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants