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

Show if maker has xmr autoconf enabled #4578

Merged

Conversation

chimp1984
Copy link
Contributor

Implements #4569

If maker is seller and is a xmr offer and has autoConf enabled we set a flag ("1")
Add info to tooltip and to offer window
Add center css to actions column. It looks weird that title is left aligned and button is centered.
only "last-column", "avatar-column" worked, making an new style e.g. center-column did not succeed... css is not my friend....
Shorten "Time since signing" to "Signed since" header to avoid truncation
@ripcurlx ripcurlx added this to the v1.4.0 milestone Sep 30, 2020
@@ -338,7 +338,7 @@ offerbook.offerersAcceptedBankSeats=Accepted seat of bank countries (taker):\n {
offerbook.availableOffers=Available offers
offerbook.filterByCurrency=Filter by currency
offerbook.filterByPaymentMethod=Filter by payment method
offerbook.timeSinceSigning=Time since signing
offerbook.timeSinceSigning=Signed since
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you change this to save space?

@ripcurlx
Copy link
Contributor

@chimp1984 How about using the fast clock instead of the rocket?
Bildschirmfoto 2020-09-30 um 17 58 33

@chimp1984
Copy link
Contributor Author

Yes ok as well, though i felt the rocket might add more incentive is looks a bit more "special"... but feel free to use the clock, no strong opinion here...

@chimp1984
Copy link
Contributor Author

Maybe the clock could be used for all instant trade options?

@ghost
Copy link

ghost commented Sep 30, 2020

Expressing my preference for the rocket! I think it looks cool and may generate interest in the feature...

image

@chimp1984
Copy link
Contributor Author

Yes I think it sticks out a bit more than a more traditional icon. The "to the moon" crew will like it as well ;-)

@ripcurlx
Copy link
Contributor

ripcurlx commented Oct 1, 2020

Yes I think it sticks out a bit more than a more traditional icon. The "to the moon" crew will like it as well ;-)

ok - voted down and I accept my defeat 😜

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

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

ACK

As described in #4569 this sets for xmr trades the autoconf flag on offer creation. It won't be updated if the config is changed after the offer has been created.

@ripcurlx ripcurlx merged commit a1778ed into bisq-network:master Oct 1, 2020
@chimp1984 chimp1984 deleted the show-if-maker-has-xmr-autoconf-enabled branch October 1, 2020 15:32
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.

2 participants