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

Add some javadoc and fix the preview update issue #6516

Merged
merged 1 commit into from
May 25, 2020

Conversation

leitianjian
Copy link
Contributor

Fixes #6447

I have taken a long time considering how to update the preview when preference change. After I found that the PreviewPanel will be bind to entry in FieldsEditorTab.java, then we can just update the layout in PreviewPanel.java#setEntry()

Some ideas:
Can we using observer pattern to refactor the code: JabRefPreferences as Observable object and component need preferences can register as Observer. If the preferences were changed, we can notify the affected component to do an update?

  • Change in CHANGELOG.md described (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 documentation: Is the information available and up to date? If not created an issue at https://github.com/JabRef/user-documentation/issues or, even better, submitted a pull request to the documentation repository.

@leitianjian leitianjian marked this pull request as ready for review May 24, 2020 12:47
@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label May 24, 2020
@Siedlerchr
Copy link
Member

Thanks, codewise looks good to me. We are currently in the process of refactoring the preferences in general and trying to solve the "updating of preferences" without restarting JabRef as well.

@leitianjian
Copy link
Contributor Author

leitianjian commented May 24, 2020

Thanks, codewise looks good to me. We are currently in the process of refactoring the preferences in general and trying to solve the "updating of preferences" without restarting JabRef as well.

What is the current idea of refactoring these preferences code? I wonder where I can see these discussions? Thanks!

@calixtus
Copy link
Member

@leitianjian : developer discussion happens here: https://gitter.im/JabRef/jabref .
Feel free to participate or even contribute. If you like, we can talk a bit about the principles behind my current attempt in refactoring the preferences.

@leitianjian
Copy link
Contributor Author

@leitianjian : developer discussion happens here: https://gitter.im/JabRef/jabref .
Feel free to participate or even contribute. If you like, we can talk a bit about the principles behind my current attempt in refactoring the preferences.

Thanks for your kindness. Recently I'm busy with other things. So, I cannot fully participate in it, but, I will keep track of your pr and the chat in gitter. Maybe I can learn something or provide some ideas

@Siedlerchr Siedlerchr merged commit 86b04a7 into JabRef:master May 25, 2020
@Siedlerchr
Copy link
Member

Thanks again for your contribution! I took the freedom to merge it ;)

@leitianjian
Copy link
Contributor Author

Thanks!

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.

Changes in entry preview are not saved
3 participants