Skip to content

replace "SearchAll" in web search by "Search Selected"#10611

Merged
Siedlerchr merged 10 commits into
JabRef:mainfrom
lisongxuan:Issue#10556-Search-Selected
Nov 9, 2023
Merged

replace "SearchAll" in web search by "Search Selected"#10611
Siedlerchr merged 10 commits into
JabRef:mainfrom
lisongxuan:Issue#10556-Search-Selected

Conversation

@lisongxuan

Copy link
Copy Markdown
Contributor

Closes #10556

  • Implemented "Search Selected".
    image

  • Enabled configuring the catalogs to be searched.
    image

  • Updated menu and other corresponding UI
    image

  • Updated corresponding tests

Mandatory checks

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

see:JabRef#10556
Implemented "Search Selected".
Enabled configuring the catalogs to be searched.
Updated menu and other corresponding UI
Updated corresponding tests
@lisongxuan lisongxuan marked this pull request as draft November 1, 2023 09:13
@lisongxuan lisongxuan marked this pull request as ready for review November 1, 2023 10:17
@lisongxuan

Copy link
Copy Markdown
Contributor Author

No clue about what happened to fetcher tests:( any help is appreciated!

Comment thread src/main/java/org/jabref/logic/importer/ImporterPreferences.java Outdated
Comment thread src/main/java/org/jabref/gui/importer/fetcher/WebSearchPaneViewModel.java Outdated
Comment thread src/main/java/org/jabref/gui/importer/fetcher/WebSearchPaneViewModel.java Outdated
Comment thread src/main/java/org/jabref/gui/preferences/websearch/WebSearchTabViewModel.java Outdated

@Siedlerchr Siedlerchr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks already good, just a few code style things

@lisongxuan lisongxuan requested a review from Siedlerchr November 2, 2023 11:58
koppor
koppor previously requested changes Nov 2, 2023
Comment thread CHANGELOG.md Outdated
Comment thread src/main/java/org/jabref/gui/preferences/websearch/WebSearchTab.fxml Outdated
Comment thread src/main/java/org/jabref/logic/importer/fetcher/CompositeSearchBasedFetcher.java Outdated
Comment thread src/main/java/org/jabref/logic/importer/fetcher/CompositeSearchBasedFetcher.java Outdated
Comment thread src/main/java/org/jabref/logic/importer/fetcher/CompositeSearchBasedFetcher.java Outdated
Comment thread src/main/java/org/jabref/logic/importer/fetcher/CompositeSearchBasedFetcher.java Outdated
Comment thread src/test/java/org/jabref/logic/importer/WebFetchersTest.java
Siedlerchr and others added 3 commits November 9, 2023 21:03
…Selected

* upstream/main:
  Update to JavaFX 21.01 (JabRef#10627)
  Fix JabRef#10498 Create Fetcher and Transformer for ScholarArchive (JabRef#10549)
  fix tests
  Bump org.jsoup:jsoup from 1.16.1 to 1.16.2 (JabRef#10625)
  Bump org.junit.jupiter:junit-jupiter from 5.10.0 to 5.10.1
  Bump org.fxmisc.flowless:flowless from 0.7.1 to 0.7.2
  Bump com.dlsc.gemsfx:gemsfx from 1.84.0 to 1.90.0
  Bump com.tngtech.archunit:archunit-junit5-api from 1.1.0 to 1.2.0
  Change linter (JabRef#10614)
@Siedlerchr Siedlerchr enabled auto-merge November 9, 2023 21:40
@Siedlerchr Siedlerchr added this pull request to the merge queue Nov 9, 2023
Merged via the queue into JabRef:main with commit c1cb2d7 Nov 9, 2023
Siedlerchr added a commit to xuanan20020/jabref that referenced this pull request Nov 10, 2023
* upstream/main: (24 commits)
  Add support for scite.ai (JabRef#10617)
  Update tests.yml (JabRef#10631)
  replace "SearchAll" in web search by "Search Selected" (JabRef#10611)
  Use System preference for light dark theme (JabRef#10593)
  Fix NPE in lookup identifer
  Add minimum requirements to CHANGELOG.md (JabRef#10628)
  Update to JavaFX 21.01 (JabRef#10627)
  Fix JabRef#10498 Create Fetcher and Transformer for ScholarArchive (JabRef#10549)
  fix tests
  Bump org.jsoup:jsoup from 1.16.1 to 1.16.2 (JabRef#10625)
  Bump org.junit.jupiter:junit-jupiter from 5.10.0 to 5.10.1
  Bump org.fxmisc.flowless:flowless from 0.7.1 to 0.7.2
  Bump com.dlsc.gemsfx:gemsfx from 1.84.0 to 1.90.0
  Bump com.tngtech.archunit:archunit-junit5-api from 1.1.0 to 1.2.0
  Change linter (JabRef#10614)
  change codecov (JabRef#10616)
  Add entry based on ISSN number JabRef#10124 (JabRef#10178)
  Enable journal information fetcher directly in popup (JabRef#10598)
  Make generate button wider (JabRef#10588)
  make openRewrite stable again
  ...

# Conflicts:
#	CHANGELOG.md
#	src/main/resources/l10n/JabRef_en.properties
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web Search: "SearchAll" should be replaced by "Search Selected"

5 participants