jabref icon indicating copy to clipboard operation
jabref copied to clipboard

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

Open ThiloteE opened this issue 3 years ago • 12 comments

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:

  • Slightly increase size of "file" field in General Tab
  • Size of General Tab should reflect the amount of files that are attached to an entry. More files --> greater size. No or only one file is attached --> small size.

Additional context

grafik

ThiloteE avatar May 18 '22 12:05 ThiloteE

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)

claell avatar May 18 '22 13:05 claell

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).

AEgit avatar May 19 '22 20:05 AEgit

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 avatar May 27 '22 11:05 MVP-D77

And for result, simply show! G7XGBAAYQYNEG{H3V%5$6C](https://user-images.githubusercontent.com/74488290/170689766-c5c469bd-3588-4584-970d-ef83f631da52.JPG) ![4_022PM)2R69)ZBHX1 U)T

MVP-D77 avatar May 27 '22 11:05 MVP-D77

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 :-)

ThiloteE avatar May 27 '22 11:05 ThiloteE

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

MVP-D77 avatar May 27 '22 11:05 MVP-D77

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 avatar Sep 04 '22 13:09 AEgit

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

AEgit avatar Sep 05 '22 20:09 AEgit

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

Siedlerchr avatar Sep 05 '22 20:09 Siedlerchr

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 avatar Oct 18 '22 18:10 ThiloteE

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

ThiloteE avatar Dec 19 '22 21:12 ThiloteE

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

koppor avatar Oct 24 '23 12:10 koppor