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

Pressing Escape does not close entry editor #2949

Closed
tobiasdiez opened this issue Jun 28, 2017 · 8 comments
Closed

Pressing Escape does not close entry editor #2949

tobiasdiez opened this issue Jun 28, 2017 · 8 comments
Assignees
Labels
component: entry-editor [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs
Milestone

Comments

@tobiasdiez
Copy link
Member

No description provided.

@tobiasdiez tobiasdiez added [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs component: entry-editor labels Jun 28, 2017
@tobiasdiez tobiasdiez added this to the v4.0 milestone Jun 28, 2017
@Siedlerchr
Copy link
Member

Siedlerchr commented Jun 28, 2017 via email

@koppor
Copy link
Member

koppor commented Jun 29, 2017

Somehow related:

  • Ctrl+S does not work. Neither when focused in the entry nor when the entry table is focused.
  • Star for modified database not shown if I change something in the entry editor

@Siedlerchr
Copy link
Member

This does not work when you are in an entry editor field.

@Siedlerchr Siedlerchr self-assigned this Jul 2, 2017
@Siedlerchr
Copy link
Member

Escape now works. For save, we first need to create a save action in the entry editor. I will create a follow up PR

@tobiasdiez
Copy link
Member Author

The save command shouldn't be handled by the entry editor but by a higher instance (probably JabRefMain).

@Siedlerchr
Copy link
Member

Yes, but the problem is that it is not that easy to filter out the command from the JFX panel, because we would need to map between awt keyEvents and javafx keyEvents/KeyCombination. Unfortuately there is no easy way to do this.
So I would suggest creating a save action analogous to the close action in EntryEditor/ setupKeyBindings

@tobiasdiez
Copy link
Member Author

This should be fixed in the latest development version. Could you please check the build from http://builds.jabref.org/master/. Thanks!

@koppor koppor mentioned this issue Nov 28, 2019
5 tasks
@koppor
Copy link
Member

koppor commented Apr 16, 2024

Since two years, this does not work. At #11190 (comment), we decided to stick with Ctrl+E to open/close the entry editor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: entry-editor [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs
Projects
None yet
Development

No branches or pull requests

3 participants