Skip to content

Add back keyboard shortcut features (#9) #10

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

Open
wants to merge 66 commits into
base: master
Choose a base branch
from
Open

Add back keyboard shortcut features (#9) #10

wants to merge 66 commits into from

Conversation

muachilin
Copy link
Collaborator

@muachilin muachilin commented Feb 25, 2020

Add back previously-supported keyboard shortcut features.

@muachilin muachilin changed the title Add back keyboard shortcut features Add back keyboard shortcut features (#9) Feb 25, 2020
@krisgun
Copy link
Collaborator

krisgun commented Feb 25, 2020

My guess is that it's going to take a bit more work than just restoring the file from the old version of the project. The current GUI library that is used is JavaFX and just looking at the imports in EmacsKeyBindings there are a lot of Swing imports, so there would probably have to be some porting done

@muachilin
Copy link
Collaborator Author

I see! I will look into handling the new GUI framework. Thanks!

krisgun and others added 26 commits February 26, 2020 10:49
This commit adds a new preference under the Entry editor tab in the preferences. This also maps to the preferences:

EDITOR_EMACS_KEYBINDINGS
EDITOR_EMACS_KEYBINDINGS_REBIND_CA
EDITOR_EMACS_KEYBINDINGS_REBIND_CF

[Issue: #9]
The text strings describing the emacs options in preferences were missing, and are now added.

[Issue: #9]
Ths commit adds the first step for re-integrating emac keybindings.
In this version emacs works globly but dosent have all features yet.

[issue: #9]
Emacs key bindings can now be toggled through a preference option in the Entry Editor preference tab

[Issue: #9]
There are sub-options deciding whether C-a and C-f  should be rebinded as well, which now work as intended

[Issue: #9]
Imports were not conforming to the CheckStyle and some localization keys for the Emacs commands were missing

[Issue: #9]
This moves all the emacs moves to its own class.

[issue: #9]
The old EmacsKeyBindings class is now removed and replaced with a working one. Missing localization keys were also added.

[Issue: #9]
If the enabledEmacsKeyBindings checkbox is unchecked, its children will now be disabled to further convey that the children settings are not applied anymore.

[Issue: #9 ]
krisgun and others added 30 commits February 27, 2020 13:58
A lot of preferences were passed from JabRefGUI instead of getting them directly from the preferences.

[Issue: #9]
This commit adds a new class to handle borth caret postion and result text for the emacs comands.

[issue: #9]
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.

4 participants