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

BondsView: Fix sorting of numeric columns #4222

Merged
merged 1 commit into from
May 5, 2020

Conversation

cd2357
Copy link
Contributor

@cd2357 cd2357 commented May 1, 2020

The table in the BondsView uses string sorting by default. This results in unexpected behavior when sorting non-string columns.

This commit adds custom comparators to the numeric columns to address that.

Fixes #3231

@cd2357 cd2357 force-pushed the fix-sorting-bondsview branch from 3c7b443 to 341833b Compare May 1, 2020 18:25
The table in the BondsView uses string sorting by default. This results in unexpected behavior when sorting non-string columns.

This commit adds custom comparators to the numeric columns to address that.

Fixes bisq-network#3231
@cd2357 cd2357 force-pushed the fix-sorting-bondsview branch from 341833b to 12dec2c Compare May 1, 2020 18:33
Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

For UI fixes it's always a good idea to include before and after screen shots to show the fix. Really helps the reviewer.

@sqrrm sqrrm merged commit cdad01a into bisq-network:master May 5, 2020
@boring-cyborg
Copy link

boring-cyborg bot commented May 5, 2020

Awesome work, congrats on your first merged pull request!

@cd2357 cd2357 deleted the fix-sorting-bondsview branch May 5, 2020 15:15
@ripcurlx ripcurlx added this to the v1.3.5 milestone Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Table view columns should use a numeric-sort algorithm
3 participants