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

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

Open
ThiloteE opened this issue May 18, 2022 · 12 comments · Fixed by #9114
Open

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

ThiloteE opened this issue May 18, 2022 · 12 comments · Fixed by #9114

Comments

@ThiloteE
Copy link
Member

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

@claell
Copy link
Contributor

claell commented May 18, 2022

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
Copy link

AEgit commented May 19, 2022

Note, this issue has been reported before:

#5105 (comment)

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

@MVP-D77
Copy link

MVP-D77 commented May 27, 2022

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
Copy link

MVP-D77 commented May 27, 2022

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

@ThiloteE
Copy link
Member Author

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
Copy link

MVP-D77 commented May 27, 2022

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

@AEgit
Copy link

AEgit commented Sep 4, 2022

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 #5105 (comment))

Behaviour in current dev version:
JabRef58_shownfiles

@AEgit
Copy link

AEgit commented Sep 5, 2022

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

@Siedlerchr Siedlerchr reopened this Sep 5, 2022
@Siedlerchr
Copy link
Member

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

@ThiloteE
Copy link
Member Author

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

@koppor koppor moved this to Normal priority in Features & Enhancements Nov 7, 2022
@koppor koppor moved this to Normal priority in JabRef UI Improvements Nov 11, 2022
@ThiloteE
Copy link
Member Author

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

@koppor
Copy link
Member

koppor commented Oct 24, 2023

For historical reference - here a screenshot of JabRef 3.8.2: #5105 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Normal priority
6 participants