Added preset for new entry keybindings and reintroduced defaults

CHANGELOG.md

@@ -22,6 +22,7 @@ Note that this project **does not** adhere to [Semantic Versioning](http://semve
 - We added a feature that allows the user to copy highlighted text in the preview window. [#6962](https://github.com/JabRef/jabref/issues/6962)
 - We added a feature that allows you to create new BibEntry via paste arxivId [#2292](https://github.com/JabRef/jabref/issues/2292)
 - We added a feature that allows the user to choose whether to trust the target site when unable to find a valid certification path from the file download site. [#7616](https://github.com/JabRef/jabref/issues/7616)
+- We added a keybinding preset for new entries. [#7705](https://github.com/JabRef/jabref/issues/7705)
 
 ### Changed
 
@@ -36,6 +37,7 @@ Note that this project **does not** adhere to [Semantic Versioning](http://semve
 - Automatically found pdf files now have the linking button to the far left and uses a link icon with a plus instead of a briefcase. The file name also has lowered opacity(70%) until added. [#3607](https://github.com/JabRef/jabref/issues/3607)
 - We simplified the select entry type form by splitting it into two parts ("Recommended" and "Others") based on internal usage data. [#6730](https://github.com/JabRef/jabref/issues/6730)
 - The export to MS Office XML now uses the month name for the field `Month` instead of the two digit number [forum#2685](https://discourse.jabref.org/t/export-month-as-text-not-number/2685)
+- We reintroduced missing default keybindings for new entries. [#7346](https://github.com/JabRef/jabref/issues/7346) [#7439](https://github.com/JabRef/jabref/issues/7439)
 
 ### Fixed
 

src/main/java/org/jabref/gui/entryeditor/EntryEditor.java

@@ -189,7 +189,7 @@ private void setupKeyBindings() {
                         event.consume();
                         break;
                     case CLOSE:
-                    case CLOSE_ENTRY:
+                    case EDIT_ENTRY:
                         close();
                         event.consume();
                         break;

src/main/java/org/jabref/gui/keyboard/KeyBinding.java

@@ -29,7 +29,6 @@ public enum KeyBinding {
     CHECK_INTEGRITY("Check integrity", Localization.lang("Check integrity"), "ctrl+F8", KeyBindingCategory.QUALITY),
     CLEANUP("Cleanup", Localization.lang("Cleanup entries"), "alt+F8", KeyBindingCategory.QUALITY),
     CLOSE_DATABASE("Close library", Localization.lang("Close library"), "ctrl+W", KeyBindingCategory.FILE),
-    CLOSE_ENTRY("Close entry", Localization.lang("Close entry"), "ctrl+E", KeyBindingCategory.VIEW),
     CLOSE("Close dialog", Localization.lang("Close dialog"), "Esc", KeyBindingCategory.VIEW),
     COPY("Copy", Localization.lang("Copy"), "ctrl+C", KeyBindingCategory.EDIT),
     COPY_TITLE("Copy title", Localization.lang("Copy title"), "ctrl+shift+alt+T", KeyBindingCategory.EDIT),
@@ -44,7 +43,7 @@ public enum KeyBinding {
     DELETE_ENTRY("Delete entry", Localization.lang("Delete entry"), "DELETE", KeyBindingCategory.BIBTEX),
     DEFAULT_DIALOG_ACTION("Execute default action in dialog", Localization.lang("Execute default action in dialog"), "ctrl+ENTER", KeyBindingCategory.VIEW),
     DOWNLOAD_FULL_TEXT("Download full text documents", Localization.lang("Download full text documents"), "alt+F7", KeyBindingCategory.QUALITY),
-    EDIT_ENTRY("Edit entry", Localization.lang("Edit entry"), "ctrl+E", KeyBindingCategory.BIBTEX),
+    EDIT_ENTRY("Open / close entry editor", Localization.lang("Open / close entry editor"), "ctrl+E", KeyBindingCategory.VIEW),
     EXPORT("Export", Localization.lang("Export"), "ctrl+alt+e", KeyBindingCategory.FILE),
     EXPORT_SELECTED("Export Selected", Localization.lang("Export selected entries"), "ctrl+shift+e", KeyBindingCategory.FILE),
     EDIT_STRINGS("Edit strings", Localization.lang("Edit strings"), "ctrl+T", KeyBindingCategory.BIBTEX),
@@ -61,17 +60,19 @@ public enum KeyBinding {
     IMPORT_INTO_CURRENT_DATABASE("Import into current library", Localization.lang("Import into current library"), "ctrl+I", KeyBindingCategory.FILE),
     IMPORT_INTO_NEW_DATABASE("Import into new library", Localization.lang("Import into new library"), "ctrl+alt+I", KeyBindingCategory.FILE),
     MERGE_ENTRIES("Merge entries", Localization.lang("Merge entries"), "ctrl+M", KeyBindingCategory.TOOLS),
+
     NEW_ARTICLE("New article", Localization.lang("New article"), "ctrl+shift+A", KeyBindingCategory.BIBTEX),
     NEW_BOOK("New book", Localization.lang("New book"), "ctrl+shift+B", KeyBindingCategory.BIBTEX),
     NEW_ENTRY("New entry", Localization.lang("New entry"), "ctrl+N", KeyBindingCategory.BIBTEX),
     NEW_ENTRY_FROM_PLAIN_TEXT("New entry from plain text", Localization.lang("New entry from plain text"), "ctrl+shift+N", KeyBindingCategory.BIBTEX),
-    NEW_INBOOK("New inbook", Localization.lang("New inbook"), "", KeyBindingCategory.BIBTEX),
-    NEW_MASTERSTHESIS("New mastersthesis", Localization.lang("New mastersthesis"), "", KeyBindingCategory.BIBTEX),
-    NEW_PHDTHESIS("New phdthesis", Localization.lang("New phdthesis"), "", KeyBindingCategory.BIBTEX),
-    NEW_PROCEEDINGS("New proceedings", Localization.lang("New proceedings"), "", KeyBindingCategory.BIBTEX),
-    NEW_UNPUBLISHED("New unpublished", Localization.lang("New unpublished"), "", KeyBindingCategory.BIBTEX),
+    NEW_INBOOK("New inbook", Localization.lang("New inbook"), "Ctrl+shift+I", KeyBindingCategory.BIBTEX),
+    NEW_MASTERSTHESIS("New mastersthesis", Localization.lang("New mastersthesis"), "Ctrl+shift+M", KeyBindingCategory.BIBTEX),
+    NEW_PHDTHESIS("New phdthesis", Localization.lang("New phdthesis"), "Ctrl+shift+T", KeyBindingCategory.BIBTEX),
+    NEW_PROCEEDINGS("New proceedings", Localization.lang("New proceedings"), "Ctrl+shift+P", KeyBindingCategory.BIBTEX),
+    NEW_UNPUBLISHED("New unpublished", Localization.lang("New unpublished"), "Ctrl+shift+U", KeyBindingCategory.BIBTEX),
     NEW_TECHREPORT("New technical report", Localization.lang("New technical report"), "", KeyBindingCategory.BIBTEX),
     NEW_INPROCEEDINGS("New inproceesings", Localization.lang("New inproceedings"), "", KeyBindingCategory.BIBTEX),
+
     NEXT_PREVIEW_LAYOUT("Next preview layout", Localization.lang("Next preview layout"), "F9", KeyBindingCategory.VIEW),
     NEXT_LIBRARY("Next library", Localization.lang("Next library"), "ctrl+PAGE_DOWN", KeyBindingCategory.VIEW),
     OPEN_CONSOLE("Open terminal here", Localization.lang("Open terminal here"), "ctrl+shift+L", KeyBindingCategory.TOOLS),

src/main/java/org/jabref/gui/preferences/keybindings/KeyBindingsTabViewModel.java

@@ -21,6 +21,7 @@
 import org.jabref.gui.preferences.PreferenceTabViewModel;
 import org.jabref.gui.preferences.keybindings.presets.BashKeyBindingPreset;
 import org.jabref.gui.preferences.keybindings.presets.KeyBindingPreset;
+import org.jabref.gui.preferences.keybindings.presets.NewEntryBindingPreset;
 import org.jabref.gui.util.OptionalObjectProperty;
 import org.jabref.logic.l10n.Localization;
 import org.jabref.preferences.PreferencesService;
@@ -45,6 +46,7 @@ public KeyBindingsTabViewModel(KeyBindingRepository keyBindingRepository, Dialog
         this.preferences = Objects.requireNonNull(preferences);
 
         keyBindingPresets.add(new BashKeyBindingPreset());
+        keyBindingPresets.add(new NewEntryBindingPreset());
     }
 
     /**

src/main/java/org/jabref/gui/preferences/keybindings/presets/NewEntryBindingPreset.java

@@ -0,0 +1,40 @@
+package org.jabref.gui.preferences.keybindings.presets;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import org.jabref.gui.keyboard.KeyBinding;
+
+public class NewEntryBindingPreset implements KeyBindingPreset {
+
+    private static final Map<KeyBinding, String> KEY_BINDINGS = new HashMap<>();
+
+    static {
+        // Clear conflicting default presets
+        KEY_BINDINGS.put(KeyBinding.PULL_CHANGES_FROM_SHARED_DATABASE, "");
+        KEY_BINDINGS.put(KeyBinding.COPY_PREVIEW, "");
+
+        // Add new entry presets
+        KEY_BINDINGS.put(KeyBinding.NEW_ARTICLE, "Ctrl+shift+A");
+        KEY_BINDINGS.put(KeyBinding.NEW_BOOK, "Ctrl+shift+B");
+        KEY_BINDINGS.put(KeyBinding.NEW_ENTRY, "Ctrl+N");
+        KEY_BINDINGS.put(KeyBinding.NEW_ENTRY_FROM_PLAIN_TEXT, "Ctrl+shift+N");
+        KEY_BINDINGS.put(KeyBinding.NEW_INBOOK, "Ctrl+shift+I");
+        KEY_BINDINGS.put(KeyBinding.NEW_INPROCEEDINGS, "Ctrl+shift+C");
+        KEY_BINDINGS.put(KeyBinding.NEW_MASTERSTHESIS, "Ctrl+shift+M");
+        KEY_BINDINGS.put(KeyBinding.NEW_PHDTHESIS, "Ctrl+shift+T");
+        KEY_BINDINGS.put(KeyBinding.NEW_PROCEEDINGS, "Ctrl+shift+P");
+        KEY_BINDINGS.put(KeyBinding.NEW_TECHREPORT, "Ctrl+shift+R");
+        KEY_BINDINGS.put(KeyBinding.NEW_UNPUBLISHED, "Ctrl+shift+U");
+    }
+
+    @Override
+    public String getName() {
+        return "New Entries";
+    }
+
+    @Override
+    public Map<KeyBinding, String> getKeyBindings() {
+        return KEY_BINDINGS;
+    }
+}

src/main/resources/l10n/JabRef_en.properties

@@ -137,7 +137,7 @@ Clear=Clear
 
 Clear\ fields=Clear fields
 
-Close\ entry=Close entry
+Open\ /\ close\ entry\ editor=Open / close entry editor
 
 Close\ dialog=Close dialog
 
@@ -269,7 +269,6 @@ Each\ line\ must\ be\ of\ the\ following\ form\:\ \'tab\:field1;field2;...;field
 
 Edit=Edit
 
-Edit\ entry=Edit entry
 Edit\ file\ type=Edit file type
 
 Edit\ group=Edit group