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

UI: Replace java.util.Collections with FXCollections #4388

Closed
cd2357 opened this issue Jul 31, 2020 · 6 comments · Fixed by #4875
Closed

UI: Replace java.util.Collections with FXCollections #4388

cd2357 opened this issue Jul 31, 2020 · 6 comments · Fixed by #4875

Comments

@cd2357
Copy link
Contributor

cd2357 commented Jul 31, 2020

As per a JavaFX performance tuning thread, the use of the FXCollections API is encouraged when optimizing for performance.

As per the FXCollections javadoc:

Utility class that consists of static methods that are 1:1 copies of java.util.Collections methods.

The utility methods are here mainly for performance reasons. All methods are optimized in a way that they yield only limited number of notifications. On the other hand, java.util.Collections methods might call "modification methods" on an ObservableList multiple times, resulting in a number of notifications.

Replace all usages of java.util.Collections with javafx.collections.FXCollections in the bisq-desktop project (the only one where JavaFX is used).

@cd2357
Copy link
Contributor Author

cd2357 commented Jul 31, 2020

This is a low-hanging fruit sort of change, minimal effort required.

Could even be a "good first issue".

@ripcurlx
Copy link
Contributor

ripcurlx commented Aug 1, 2020

Yes, might make sense to tackle a couple of this performance optimizations. But especially it would be good to update first to the new JavaFX version.

@vvkpd
Copy link

vvkpd commented Oct 6, 2020

I would like to work on this card please let me know if required.

Thanks

@cd2357
Copy link
Contributor Author

cd2357 commented Oct 6, 2020

Yes sure, you can take it. Please submit a PR when you have it ready.

@vvkpd
Copy link

vvkpd commented Oct 13, 2020

@cd2357 Thanks for assigning me on this task. I just started working on it.
Just a small suggestion: Could you please add hacktoberfest topic to repo and hacktoberfest as label in this task?
Background: I am participating in hacktoberfest 2020.
Thanks

@deusmax
Copy link
Contributor

deusmax commented Nov 29, 2020

@vvkpd, @cd2357
Is work in this being done ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants