Skip to content

Commit

Permalink
Add comment for @ripcurlx to add some visual element
Browse files Browse the repository at this point in the history
If we have multiple payment accounts we should show some info/warning to
make sure the user has selected the account he wants to use.
To use a popup is not recommended as we have already 3 popups at the
first time (if user has not clicked
don't show again).
@ripcurlx Maybe a similar warn triangle as used in create offer
might work?

See: bisq-network#1733
  • Loading branch information
ManfredKarrer committed Sep 28, 2018
1 parent afdd14e commit bd44aea
Showing 1 changed file with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -241,6 +241,12 @@ protected void activate() {
paymentAccountsComboBox.getSelectionModel().select(0);

paymentAccountTitledGroupBg.setText(Res.get("shared.selectTradingAccount"));

// TODO if we have multiple payment accounts we should show some info/warning to
// make sure the user has selected the account he wants to use.
// To use a popup is not recommended as we have already 3 popups at the first time (if user has not clicked
// don't show again).
// @Christoph Maybe a similar warn triangle as used in create offer might work?
}

balanceTextField.setTargetAmount(model.dataModel.getTotalToPayAsCoin().get());
Expand Down

0 comments on commit bd44aea

Please sign in to comment.