-
Notifications
You must be signed in to change notification settings - Fork 70
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
Bisq avatar improvements #1706
Bisq avatar improvements #1706
Conversation
6122280
to
ff59130
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK
Nice improvements! Now this is much more flexible when it comes to number of images and buckets.
...op/src/main/java/bisq/desktop/overlay/onboarding/create_profile/CreateProfileController.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
Changes in the following files cause a merge conflict:
|
Use an integer array instead of encoding the data into a long which added limitations and made the code more complicate.
Use SIZE instead of width()/height() methods
Add BucketGenerator Increase *_COUNT values by 1 (was wrong before to use max index)
a3d3d21
to
18dc6ee
Compare
@alvasw Rebased on main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
Based on #1705
Rebased on current main