Skip to content
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

Preview panel is now updated when an entry is cut/deleted #937

Merged
merged 1 commit into from
Mar 11, 2016
Merged

Preview panel is now updated when an entry is cut/deleted #937

merged 1 commit into from
Mar 11, 2016

Conversation

chriba
Copy link
Contributor

@chriba chriba commented Mar 11, 2016

Fixes #936.
When an entry is cut/deleted the preview panel will be hidden due to the fact that no entry is selected anymore.

  • Change in CHANGELOG.md described?
  • Changes in pull request outlined? (What, why, ...)
  • Tests created for changes?
  • Tests green?

@boceckts
Copy link
Contributor

Looks good 👍

@boceckts boceckts added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers and removed stupro-ready-for-internal-review labels Mar 11, 2016
@oscargus
Copy link
Contributor

👍

1 similar comment
@simonharrer
Copy link
Contributor

👍

@stefan-kolb
Copy link
Member

Does the panel show up again when selecting another entry after deleting one? If so 👍

@chriba
Copy link
Contributor Author

chriba commented Mar 11, 2016

Yes, it does.

@tobiasdiez
Copy link
Member

We really should consider to implement an event system...

tobiasdiez added a commit that referenced this pull request Mar 11, 2016
Preview panel is now updated when an entry is cut/deleted
@tobiasdiez tobiasdiez merged commit eea587a into JabRef:master Mar 11, 2016
@chriba chriba deleted the fix-936 branch March 11, 2016 15:40
@simonharrer
Copy link
Contributor

👍 for the event system (maybe use the stuff that comes with google guava.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

6 participants