jabref icon indicating copy to clipboard operation
jabref copied to clipboard

Remove checkbox for Enable field formatters

Open Siedlerchr opened this issue 2 weeks ago • 1 comments

We now have an apply button and this fixes the binding exception I encountered (see melting pot issue)

Follow up to https://github.com/JabRef/jabref/pull/13852 Fixes https://github.com/JabRef/jabref-issue-melting-pot/issues/1144

grafik

Steps to test

  1. Open quality -> Cleanup entries

Mandatory checks

  • [x] I own the copyright of the code submitted and I license it under the MIT license
  • [x] I manually tested my changes in running JabRef (always required)
  • [/[ I added JUnit tests for changes (if applicable)
  • [x] I added screenshots in the PR description (if change is visible to the user)
  • [/] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • [/] I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

Siedlerchr avatar Dec 10 '25 19:12 Siedlerchr

For further discussion: I think there should be warning dialog that Apply closes the dialog and executes the selected option. Maybe rename the button to Apply actions? I think only apply can be dangerous as it is not clear that he dialog closes

Siedlerchr avatar Dec 10 '25 19:12 Siedlerchr

For further discussion: I think there should be warning dialog that Apply closes the dialog and executes the selected option. Maybe rename the button to Apply actions? I think only apply can be dangerous as it is not clear that he dialog closes

Maybe "Run" or "Execute"? To be really different to the word "Apply" used in settings dialog.

koppor avatar Dec 11 '25 19:12 koppor

Yes, UI will probably need more love here. Maybe the button apply needs to be on the right with some chevron indicating - this is it, lets do it.

calixtus avatar Dec 11 '25 22:12 calixtus

Let's tackle this in a follow up pr

Siedlerchr avatar Dec 14 '25 20:12 Siedlerchr