Concordium / concordium-reference-wallet-ios

Reference wallet for the Concordium blockchain
Apache License 2.0
11 stars 10 forks source link

Scheduled transfers with memo have cut off text for amounts with many decimal places #69

Closed concordium-cl closed 3 years ago

concordium-cl commented 3 years ago

Bug Description

Scheduled transfers with memo have cut off text for amounts with many decimal places, e.g. on sender account as the fee usually has many decimal places. This happens both in transfer list and in the transfer details when clicking on a transfer in transfer list, so the full text will never been shown anywhere.

IMG-4025 IMG-4026

Steps to Reproduce

Expected Result Text is fully shown, at least in transfer details.

Actual Result Text is cut off.

Versions

concordium-cl commented 3 years ago

@jens-concordium , maybe have a shorter text?

kristiyandobrev commented 3 years ago

@jens-concordium

I think the best we can do here is to make the text breaking into multiple lines. What do you think ?

jens-concordium commented 3 years ago

@concordium-cl @kristiyandobrev

I think we might have to leave it as it is in the transaction log for now, as it probably would be a quite large task to redesign and implement, but maybe we can break it into more lines in the transaction details. Something like the sketch below. What do you think @concordium-cl? We would have to do the same on Android to keep consistency.

image

concordium-cl commented 3 years ago

It's sufficient to have the full text in the transaction details.

jens-concordium commented 3 years ago

@jensvesti, what do you think? I'll add it to the corresponding Android task as well, if you think this is doable.