-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Automatic lookup of DOI at citation relations #13539
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
Changes from 5 commits
ec864d3
365b4fa
a24dd53
4ef4dd4
af8b918
5fa2d72
927664c
dfdec86
43e1ce7
487e1fb
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
package org.jabref.gui.entryeditor.citationrelationtab; | ||
|
||
import javafx.scene.control.Button; | ||
import javafx.scene.control.ProgressIndicator; | ||
|
||
import org.jabref.logic.importer.fetcher.citation.CitationFetcher; | ||
import org.jabref.model.entry.BibEntry; | ||
|
||
import org.controlsfx.control.CheckListView; | ||
|
||
public record CitationComponents( | ||
BibEntry entry, | ||
CheckListView<CitationRelationItem> listView, | ||
Button abortButton, | ||
Button refreshButton, | ||
CitationFetcher.SearchType searchType, | ||
Button importButton, | ||
ProgressIndicator progress) { | ||
} |
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2791,7 +2791,8 @@ Miscellaneous=Diversos | |
File-related=Arquivo relacionado | ||
|
||
Add\ selected\ entry(s)\ to\ library=Adicionar as referências selecionadas à biblioteca | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You have to go through crowdin web site to update translations.... There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. OK, let me have a look. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. When the PR is merged, the new translations will be synced with crowdin and you can translat them there https://crowdin.com/project/jabref There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @koppor @Siedlerchr Could you raise an issue to address the new translations with Crowdin and assign it to me, please? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do you need that for a university course? Typically Crowdin just works. |
||
The\ selected\ entry\ doesn't\ have\ a\ DOI\ linked\ to\ it.\ Lookup\ a\ DOI\ and\ try\ again.=A entrada selecionada não tem um DOI vinculado a ele. Verifique um DOI e tente novamente. | ||
The\ selected\ entry\ doesn't\ have\ a\ DOI\ linked\ to\ it.=A entrada selecionada não tem um DOI vinculado a ele. | ||
Look\ Up\ a\ DOI\ and\ try\ again.=Verifique um DOI e tente novamente. | ||
Cited\ By=Citado por | ||
Cites=Citar | ||
No\ articles\ found=Nenhum artigo encontrado | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UI component (ProgressIndicator) should not be directly included in a data structure. This violates separation of concerns and should be managed in a dedicated UI controller.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is simple parameter object - instead passing 7 parameters, use dedicated object.