Walletwasabi: BTC send amount field should be limited to 8 sub decimals.

Created on 29 Aug 2019  路  7Comments  路  Source: zkSNACKs/WalletWasabi

I'm running very old binaries, so not sure if it's been resolved in the meanwhile.

When entering an amount of ie 0.2345 I would randomly hit the numpad to add entropy to the last digits.

image

UI good first issue

Most helpful comment

looks like the PR made by @benthecarman here should make this very easy to resolve. Avalonia doesn't support MaxLength property out of the box at the moment.

That won't actually help, MaxLength defines the max length of the entire text in the box. We want to limit the text after the decimal point so MaxLength won't help here.

All 7 comments

ACK, not sure if it's fixed already or not.

It is not fixed yet.
In addition we should limit the field to accept only numbers and the decimal point.

It currently does this behind the scenes but does not show up in the GUI

looks like the PR made by @benthecarman here should make this very easy to resolve. Avalonia doesn't support MaxLength property out of the box at the moment.

looks like the PR made by @benthecarman here should make this very easy to resolve. Avalonia doesn't support MaxLength property out of the box at the moment.

That won't actually help, MaxLength defines the max length of the entire text in the box. We want to limit the text after the decimal point so MaxLength won't help here.

This probably needs a coercion like API either in Wasabi or Avalonia (possible: https://github.com/AvaloniaUI/Avalonia/pull/3287)

this is already implemented - closing.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

molnard picture molnard  路  3Comments

nopara73 picture nopara73  路  3Comments

UkolovaOlga picture UkolovaOlga  路  3Comments

MaxHillebrand picture MaxHillebrand  路  3Comments

yahiheb picture yahiheb  路  3Comments