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

[Workaround in comments] ESC removes information from entry editor #10764

Closed
2 tasks done
ilippert opened this issue Jan 10, 2024 · 17 comments · Fixed by #11190
Closed
2 tasks done

[Workaround in comments] ESC removes information from entry editor #10764

ilippert opened this issue Jan 10, 2024 · 17 comments · Fixed by #11190
Labels
bug Confirmed bugs or reports that are very likely to be bugs entry-editor keybinding

Comments

@ilippert
Copy link
Contributor

JabRef version

5.12 (latest release)

Operating system

GNU / Linux

Details on version and operating system

No response

Checked with the latest development build (copy version output from About dialog)

  • I made a backup of my libraries before testing the latest development version.
  • I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

I am on JabRef 5.12--2023-12-24--4233bad
Linux 6.6.10-200.fc39.x86_64 amd64
Java 21.0.1
JavaFX 21.0.1+6

  1. type some notes in a field in the entry editor, and then press ESC.
  2. The information in the field disappears
  3. cannot be brought back with CTRL+Z

That is sad because the user, at least I, might accidentally press ESC.

I consider this a bug because I think this is not intended. But am happy to revise thi as a feature request if you think so.

Appendix

No response

@Siedlerchr
Copy link
Member

Siedlerchr commented Jan 10, 2024

Does this also occur in the 5.13 were we downgraded javafx (is integreatd in latest main) again?

@ilippert
Copy link
Contributor Author

Hi, happy to try, but cannot find https://builds.jabref.org/pull/10747/merge anymore, I think.

@Siedlerchr
Copy link
Member

you can use latest main we merged it into main

@ilippert
Copy link
Contributor Author

Thanks,

on

JabRef 5.13--2024-01-09--01c3e15
Linux 6.6.10-200.fc39.x86_64 amd64
Java 21.0.1
JavaFX 20+19

it is still the case

@HoussemNasri
Copy link
Member

I haven't found the reason behind this behavior or a way to fix it but found a workaround.

Workaround: click on the entry again to remove the focus from the entry editor and press Ctrl+Z or press the undo button above in the main toolbar.

@HoussemNasri HoussemNasri added keybinding entry-editor bug Confirmed bugs or reports that are very likely to be bugs labels Jan 10, 2024
@Siedlerchr
Copy link
Member

I can confirm the same behavior under macOS. To me it seems like some javafx thingy default behavior. No idea though

@ror3d
Copy link
Contributor

ror3d commented Mar 18, 2024

Just stumbled into this with a more recent version (I use vim a lot and I have the pressing Esc is kind of automatic)

Sadly I don't know javafx enough to easily look into it

@jeanprbt
Copy link
Contributor

jeanprbt commented Apr 9, 2024

Hi, I can try to tackle that issue if you wish ! I was able to reproduce this behavior.

@Siedlerchr
Copy link
Member

@jeanprbt I think this more a javafx standard behavior.

@jeanprbt
Copy link
Contributor

jeanprbt commented Apr 9, 2024

@jeanprbt I think this more a javafx standard behavior.

Maybe we can close this issue then. :)

@ilippert
Copy link
Contributor Author

ilippert commented Apr 9, 2024 via email

@Siedlerchr Siedlerchr added the status: depends-on-external A bug or issue that depends on an update of an external library label Apr 9, 2024
@ThiloteE
Copy link
Member

ThiloteE commented Apr 9, 2024

I would suggest to leave it open, as a kind of "documentation", as "undo" is an important feature, but since issue might depend on external and there is also a workaround I would set the priority to low.

@koppor
Copy link
Member

koppor commented Apr 11, 2024

Hi, I can try to tackle that issue if you wish ! I was able to reproduce this behavior.

We need a minimal working example to report to the JavaFX community.

Maybe fork https://github.com/Siedlerchr/javafxreproducer and just add a text field @jeanprbt . Maybe with a label above the text field with the steps a user should make to reproduce the behavior and what is the accepted behavior.

I know, this is kind of trivial intended behavior. However, the JavaFX implementors are in their JavaFX world and we need to get close to their world to explain.

@jeanprbt
Copy link
Contributor

Hi, I can do that but what should I do next ? Should I file an issue on official Open JFX issue tracker with a link to my fork ?

@Siedlerchr
Copy link
Member

You can only submit a bug report via https://bugreport.java.com/bugreport/ if you can reproduce it

@HoussemNasri HoussemNasri changed the title ESC removes information from entry editor [Workaround in comments] ESC removes information from entry editor Apr 11, 2024
@jeanprbt
Copy link
Contributor

I tried to reproduce the bug using a minimalistic example, you can see here. However, using the original javafx.scene.control.TextFieldthe bug does not appear. Any idea how is JabRef implementing its field editors ?

@koppor
Copy link
Member

koppor commented Apr 13, 2024

@jeanprbt Thank you for building the reproducer and reporting that is a JabRef issue.

Now, I could narrow it down. Will submit a fix soon. This comment is only to prevent anyone else to work on this now. Focus on other things.

Any idea how is JabRef implementing its field editors ?

Let's look at org.jabref.gui.fieldeditors.FieldEditors?

I tested it. It occurres in all kinds of editors. ESC just clears the field.

No special code.

Then, let's investiage KeyBinding.java.

ESC appears twice. One unused. One used. Used in ALL txt fields (See TextInputKeyBindings). Introduced at #8829.

@koppor koppor removed the status: depends-on-external A bug or issue that depends on an update of an external library label Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bugs or reports that are very likely to be bugs entry-editor keybinding
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

7 participants