Skip to content

Fixes koppor/jabref#521 - Allow to drag&drop selected bib entries to other library tabs#8982

Merged
koppor merged 7 commits into
JabRef:mainfrom
HITOSS:addfeature-issue-521
Jul 18, 2022
Merged

Fixes koppor/jabref#521 - Allow to drag&drop selected bib entries to other library tabs#8982
koppor merged 7 commits into
JabRef:mainfrom
HITOSS:addfeature-issue-521

Conversation

@GNAQ

@GNAQ GNAQ commented Jul 15, 2022

Copy link
Copy Markdown
Contributor

Fixed JabRef#521

by adding a new feature that allows to drag selected entries, and drop them to other opened inactive bib library tabs.

Behavior Description:

Now when dragging selected entries in main table, JabRef allows user to drop it on another opened library tab, and this item will be copied and added to the dropped library accordingly. Drop to current active tab is disallowed. If there're potential duplicated entries, JabRef will ask user to confirm and choose copy behavior (similar to ctrl-c/v copy and paste).

Screenshots for UI changes:

screenshot.gif

Checklist / TODOs :

  • Code implement of this new feature
  • 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.

@GNAQ GNAQ marked this pull request as ready for review July 15, 2022 15:26
@ThiloteE ThiloteE changed the title Fix issue koppor/jabref#521 (Add feature: allow drag&drop from selected bib entries to other library tabs) Fixes koppor/jabref#521 - Allow to drag&drop selected bib entries to other library tabs Jul 16, 2022
@ThiloteE ThiloteE added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Jul 16, 2022
@Siedlerchr Siedlerchr added this to the v5.7 milestone Jul 16, 2022
Comment thread CHANGELOG.md 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.

thanks for this PR! Codewise looks already good to me.
Before merging, a second maintainer needs to review your code as well

…ription)

Co-authored-by: Christoph <siedlerkiller@gmail.com>
@GNAQ

GNAQ commented Jul 17, 2022

Copy link
Copy Markdown
Contributor Author

thanks for this PR! Codewise looks already good to me. Before merging, a second maintainer needs to review your code as well

Thank you for your kindly review!

@koppor

koppor commented Jul 18, 2022

Copy link
Copy Markdown
Member

Thank you for the PR! We have a small nitpick that t should be named tab (to have a bit more readable local variable name).

We could not update it for ourselves, because editing by maintainers is disabled:

grafik

We merge nevertheless and will rename later (at some time ^^).

Thank you for the PR!

@koppor koppor merged commit 6d6423c into JabRef:main Jul 18, 2022
@koppor

koppor commented Jul 18, 2022

Copy link
Copy Markdown
Member

Rename happened at #8989

Siedlerchr added a commit to zkl-ai/jabref that referenced this pull request Jul 19, 2022
…failure-dialog

* upstream/main:
  Reworded t in pr (JabRef#8989)
  Fixed checkstyle (JabRef#8990)
  Fix RfcFetcherTest (JabRef#8988)
  Fixes JabRef#521 - Allow to drag&drop selected bib entries to other library tabs (JabRef#8982)

# Conflicts:
#	src/main/java/org/jabref/gui/maintable/MainTable.java
#	src/test/java/org/jabref/logic/importer/fetcher/JstorFetcherTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: ui status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

drag'n'drop to another library should be possible

5 participants