-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Display files from referenced crossref in entry table (HighCrit version) #9717
base: main
Are you sure you want to change the base?
Conversation
Found a bug in our implementation needs more work
I like the idea, I wonder if there is a possibility to visually indicate this somehow that this is a crossRef file coming from a a parent/linked entry. |
I think we can use the |
Hello @HighCrit @bsVermeiden, thank you for taking a look at this issue! It was, unfortunately, quite a while since I looked at JabRef's source code, and I don't currently understand,
What is the entire table? The main table or the bib entry table? As long as it isn't the first one, it might be ok to listen to changes to the whole table. "P.S.", love that there are already tests 🥳 |
Hey @k3KAW8Pnf7mkmdSMPHz27, thank you for taking a look.
We now believe that we only need to observe the changes in the entry itself. Namely, changes in crossref field and files field. However, we cannot seem to figure this out.
It concerns area 1. We figured, since the other fields are also not propegated in area 2 for crossrefed values that only area 1 is desired. |
I believe at least part of it was looked at in #7754. I'll see if I can re-acquaint myself with #7754 soonish as well, perhaps there is something you can use from it in your PR, either through cherry-picking commits, merging the branch into this one, or the ideas discussed in it. I'll get back to you on this one.
I honestly don't know. It might be worth thinking about having grayed out/default text, e.g., from Google search |
Sorry that I haven't gotten back to you earlier; I've had some issues I've had to deal with and haven't had enough time for open-source :/ For now, I've merged If you have specific questions I am usually able to answer quite quickly. |
Fixes #7731.
Now display attached files from referenced crossref entry in the entry table. By providing a
getFiles
method inBibEntry
that takes in a database. From which referenced crossref files are fetched.Known Bugs:
Mandatory checks
CHANGELOG.md
described in a way that is understandable for the average user (if applicable)