fix: easier amount input

mpetrunic commented 3 months ago

Description

Related Issue Or Context

Closes: #150

How Has This Been Tested? Testing details.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have ensured that all acceptance criteria (or expected behavior) from issue are met
[ ] I have updated the documentation locally and in sygma-docs.
[ ] I have added tests to cover my changes.
[ ] I have ensured that all the checks are passing and green, I've signed the CLA bot

wainola commented 3 months ago

there is a small issue with the cursor when deleting the amount, it goes to the start of the input and leave some numbers there

mpetrunic commented 3 months ago

there is a small issue with the cursor when deleting the amount, it goes to the start of the input and leave some numbers there

This is default behavior of number type input. If we switch to text, it will not do that but it will also allow non number inputs and won't respect locale. I'm fine with switching to text input if you think it's better

Lykhoyda commented 3 months ago

@mpetrunic @wainola I think we can keep the type number for now, it's not causing big issues. If we want to change it, we can open a new issue. I will approve this one

sygmaprotocol / sygma-widget