Testing and review items for Refactor unlinked files (PR #7209)

[koobs]

Testing and review items for Refactor unlinked files in PR #7209

JabRef 5.3--2021-01-24--ae43548
Windows 10 10.0 amd64 
Java 14.0.2 
JavaFX 15.0.1+1

• Regression: Searching for File extension: PDF file *.pdf appears not to match files in sub-directories during Scan directory. ANY_FILE appears to work recursively

PDF Search

ANY_FILE Search

• Start directory and File extension controls should set a minimum height (overlap during resize)

• Allow search results control to be minimized (on resize). Minimum size is too large.

Import results list:

• Import result list columns should be resizable
• Re-order columns (Status | File | Message)
• Create one row for each file processes (test case: processing one file results in 7 rows, 6 of which are empty)
• Expand/Collapse triangle moves location when switching between Currently linked files and Import results. Both triangles for each list should be located on the left (not right)

UI & Usability Tweaks

• Search button should have a minimum horizontal size set
• Directory and File type controls minimum size is too large (labels disappear during resize)

Control Labels:

• Rename Scan directory button label to Search (consistent with dialog and feature title)
• Rename Start directory to Directory (remove colon :)
• Rename File extension: to File type (remove colon :)
• Rename Currently unlinked files to Search results

Unless this is an issue with windows java framework features (rather than file type static names set in JabRef)

• Rename ANY_FILE (*.*) to All files (*.*)
• Rename BIBTEX_DB to Bibtex Database (*.bib)

Control types

• Replace Select/Unselect Collapse/Expand buttons with right-click context menus

References: #7205 #7206

[Siedlerchr]

Thanks for testing and reviewing!
Fixed some of your issues in a follow up PR #7385

I disagree with some of your wording suggestions regarding the name of the labels or the buttons.

Regarding the arrow: Although it jumps it has to be on the right side. JabRef follows Material Design guidelines.

Otherwise: The Empty rows are created by javafx by default and the TableView resizing is a fcking shit.

[koobs]

@Siedlerchr Much ❤️ and you're welcome. Do note that some of the items here are just my initial 'easy' tweak suggestions that form part of a broader UI/UX standardizing/convention document I have going for 'later use'. I'm coming into jabref with my 'new user' hat on, and lots of UI/UX/opensource experience to guide me. All of it is motivated by wanting to get jabref to the point where its extremely friendly and coherent, applying as many good patterns and practices as possible within the scope of its goals.

I also thought we might be restricted by JavaFX stuff, but wasn't sure, so feel free not to include anything that's too difficult until we can do more fundamental improvements in separate/distinct issues. :)

[koobs]

Regarding the arrow: Although it jumps it has to be on the right side. JabRef follows Material Design guidelines.

That's absolutely fine, but then we should be following it consistently everywhere else as well :) I haven't been able to find a design system / style guide / design guidelines document for JabRef, do you know if we have one?

[calixtus]

Devcall: Hi @koobs , we only have some rudimentary ui recommendations yet. We would love it, if someone would like to put some work in it. Would you be interested in working on this page (https://devdocs.jabref.org/advanced-reading/ui-recommendations) in close coordination with us?

[koobs]

@calixtus Would be my pleasure