JabRef / jabref

Graphical Java application for managing BibTeX and biblatex (.bib) databases
https://devdocs.jabref.org
MIT License
3.61k stars 2.57k forks source link

Increase size of "file" field in General Tab of Entry-Editor #8823

Open ThiloteE opened 2 years ago

ThiloteE commented 2 years ago

Problem:

  1. Have multiple files attached to an entry
  2. Try to select and remove an attached file from an entry

What happens:

Because the size of the field is so small, it has very hard to select a specific file (or specifc fileS), because the scrollbar often jumps too far or not far enough. Also, when scrolling over the end of the file field, the scrollbar of the general tab will get activated and the user will henceforth scroll within the general tab instead of the file field.

Ideas for Solution:

Additional context

grafik

claell commented 2 years ago

I like the idea of having a responsive height based on the amount of files (possibly with a maximum height at which the internal scroll bar is used again)

AEgit commented 2 years ago

Note, this issue has been reported before:

https://github.com/JabRef/jabref/issues/5105#issuecomment-509132201

Version 3.8.2 showed many more files than the current version (see screenshots in the linked post).

MVP-D77 commented 2 years ago

Hello, @[ThiloteE] , our group are interested in your problem and would like to work on this issue. Can we have a try? ——SE 2022 group haha, SUSTech

MVP-D77 commented 2 years ago

And for result, simply show! G7XGBAA`YQYNEG{H3V%5$6C 4`_022PM)2R69)ZBHX1 U)T

ThiloteE commented 2 years ago

hey, thanks yes this seems to be a lot less controversial issue than the other one :-) Thanks for the PR, I am going to try it :-)

MVP-D77 commented 2 years ago

Thanks, but also, I just take a example, not complete!

AEgit commented 2 years ago

JabRef 5.8--2022-09-03--416ba16 Windows 10 10.0 amd64 Java 18.0.2.1 JavaFX 18.0.2+2

Hmm, this does not appear to be (fully?) solved in the current dev version. There is just more white space below the linked files (see attached picture), but there are still only 3 linked files shown, despite the fact, that there are 5 linked files. Note, that the old version of JabRef could easily handle ca. 10 linked files or so (see https://github.com/JabRef/jabref/issues/5105#issuecomment-509132201)

Behaviour in current dev version: JabRef58_shownfiles

AEgit commented 2 years ago

@Siedlerchr Can you re-open this issue please? As far as I can tell, the problem has not been solved yet.

Siedlerchr commented 2 years ago

Yes, it still has a fixed size. We are currently investigating potential ways to improve the height calculation, but seems not that easy.

ThiloteE commented 2 years ago

To reiterate: There is still the problem that when there are more than 3 files attached, some files will be hidden from view.

Here is where the relevant files and where most of the code that is responsible for this behaviour lives:

JabRef/src/main/java/org/jabref/gui/fieldeditors/LinkedFilesEditor.fxml JabRef/src/main/java/org/jabref.gui/fieldeditors/LinkedFilesEditor.java JabRef/src/main/java/org/jabref/gui/fieldeditors/LinkedFilesEditorViewModel.java JabRef/src/main/java/org/jabref/gui/entryeditor/FieldsEditorTab.java

I used scenic-view to have a look at what exactly is there in the UI and what boxes and objects compete for space. I tracked it down to the JFXListViewSkin.class

com.jfoenix.skins.JFXListViewSkin

ThiloteE commented 1 year ago

Reminder: revisit this issue once gemsfx or some other library has replaced jfoenix.

koppor commented 12 months ago

For historical reference - here a screenshot of JabRef 3.8.2: https://github.com/JabRef/jabref/issues/5105#issuecomment-509132201