jabref icon indicating copy to clipboard operation
jabref copied to clipboard

ESC removes information from entry editor

Open ilippert opened this issue 1 year ago • 6 comments

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)

  • [X] I made a backup of my libraries before testing the latest development version.
  • [X] 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

ilippert avatar Jan 10 '24 07:01 ilippert

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

Siedlerchr avatar Jan 10 '24 07:01 Siedlerchr

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

ilippert avatar Jan 10 '24 08:01 ilippert

you can use latest main we merged it into main

Siedlerchr avatar Jan 10 '24 08:01 Siedlerchr

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

ilippert avatar Jan 10 '24 09:01 ilippert

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 avatar Jan 10 '24 18:01 HoussemNasri

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

Siedlerchr avatar Jan 13 '24 19:01 Siedlerchr

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

ror3d avatar Mar 18 '24 14:03 ror3d

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

jeanprbt avatar Apr 09 '24 17:04 jeanprbt

@JEANPRBT I think this more a javafx standard behavior.

Siedlerchr avatar Apr 09 '24 18:04 Siedlerchr

@JEANPRBT I think this more a javafx standard behavior.

Maybe we can close this issue then. :)

jeanprbt avatar Apr 09 '24 18:04 jeanprbt

Then, I'd tag it as depending on external... But maybe checking, communicating this upstream and reporting here where the issue is placed would be good.

Sorry, am not of much help these weeks.

Sent from my LineageOS device with K-9 Mail. Please excuse my brevity.

On 9 April 2024 20:43:37 CEST, Jean Perbet @.***> wrote:

@JEANPRBT I think this more a javafx standard behavior.

Maybe we can close this issue then. :)

-- Reply to this email directly or view it on GitHub: https://github.com/JabRef/jabref/issues/10764#issuecomment-2045855969 You are receiving this because you authored the thread.

Message ID: @.***>

ilippert avatar Apr 09 '24 18:04 ilippert

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.

ThiloteE avatar Apr 09 '24 18:04 ThiloteE

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.

koppor avatar Apr 11 '24 09:04 koppor

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 ?

jeanprbt avatar Apr 11 '24 12:04 jeanprbt

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

Siedlerchr avatar Apr 11 '24 13:04 Siedlerchr

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 ?

jeanprbt avatar Apr 13 '24 11:04 jeanprbt

@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 https://github.com/JabRef/jabref/pull/8829.

koppor avatar Apr 13 '24 12:04 koppor