[bisq-network/bisq] Move volume formatting from DisplayUtils to VolumeUtil (#5740)
Stan
notifications at github.com
Sat Oct 2 22:37:13 CEST 2021
In practice, presentation/display logic should be kept in UIs, but the core api will need to duplicate a very small part of the code in `DisplayUtils` if a very small part of this refactoring is not done. Instead of moving a tiny bit of the volume formatting methods, all of them are moved to core's `VolumeUtil`.
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/5740
-- Commit Summary --
* <a href="https://github.com/bisq-network/bisq/pull/5740/commits/db6e2a51b2243c84a0be7cef7ae012d348359c1d">Move volume formatting from DisplayUtils to VolumeUtil</a>
-- File Changes --
M core/src/main/java/bisq/core/util/VolumeUtil.java (83)
M desktop/src/main/java/bisq/desktop/main/dao/wallet/send/BsqSendView.java (6)
M desktop/src/main/java/bisq/desktop/main/market/MarketView.java (3)
M desktop/src/main/java/bisq/desktop/main/market/offerbook/OfferBookChartView.java (67)
M desktop/src/main/java/bisq/desktop/main/market/offerbook/OfferBookChartViewModel.java (21)
M desktop/src/main/java/bisq/desktop/main/market/trades/TradeStatistics3ListItem.java (5)
M desktop/src/main/java/bisq/desktop/main/market/trades/TradesChartsView.java (3)
M desktop/src/main/java/bisq/desktop/main/market/trades/charts/volume/VolumeBar.java (4)
M desktop/src/main/java/bisq/desktop/main/offer/FeeUtil.java (7)
M desktop/src/main/java/bisq/desktop/main/offer/MutableOfferViewModel.java (8)
M desktop/src/main/java/bisq/desktop/main/offer/offerbook/OfferBookViewModel.java (3)
M desktop/src/main/java/bisq/desktop/main/offer/takeoffer/TakeOfferViewModel.java (3)
M desktop/src/main/java/bisq/desktop/main/overlays/windows/ContractWindow.java (62)
M desktop/src/main/java/bisq/desktop/main/overlays/windows/DisputeSummaryWindow.java (3)
M desktop/src/main/java/bisq/desktop/main/overlays/windows/OfferDetailsWindow.java (19)
M desktop/src/main/java/bisq/desktop/main/overlays/windows/SwiftPaymentDetails.java (9)
M desktop/src/main/java/bisq/desktop/main/overlays/windows/TradeDetailsWindow.java (5)
M desktop/src/main/java/bisq/desktop/main/portfolio/closedtrades/ClosedTradesViewModel.java (7)
M desktop/src/main/java/bisq/desktop/main/portfolio/failedtrades/FailedTradesViewModel.java (6)
M desktop/src/main/java/bisq/desktop/main/portfolio/openoffer/OpenOffersViewModel.java (5)
M desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesView.java (5)
M desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesViewModel.java (3)
M desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/steps/buyer/BuyerStep2View.java (4)
M desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/steps/seller/SellerStep3View.java (28)
M desktop/src/main/java/bisq/desktop/util/DisplayUtils.java (83)
M desktop/src/main/java/bisq/desktop/util/GUIUtil.java (7)
M desktop/src/test/java/bisq/desktop/util/DisplayUtilsTest.java (15)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/5740.patch
https://github.com/bisq-network/bisq/pull/5740.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/5740
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20211002/fa4b5f01/attachment.htm>
More information about the bisq-github
mailing list