asgiom / cr8-brm-mobile-issues

0 stars 1 forks source link

[BUG] Allow only decimal and specific special chars for "amount" fields. #121

Closed t-jacobi closed 4 years ago

t-jacobi commented 4 years ago

Describe the bug You can enter alphanumerical characters into the amount field in the area of sending and recieving. Also the keyboard layout should be the same in this two areas. (Send: direct numerical, recieve: alpha layout)

To Reproduce Steps to reproduce the behavior:

  1. Go to 'Send
  2. Click on Amount
  3. Switch to alpha characters

or

  1. Go to 'Recieve'
  2. Click on Amount
  3. Alpha keyboard layout will be shown

Expected behavior Allow only numbers and , and .

Screenshots If applicable, add screenshots to help explain your problem.

Smartphone (please complete the following information):

Additional context This solves a unrealstic issue but looks professional IMO.

t3nshi commented 4 years ago

Thatś right the implementation is for numeric, but it seems that numeric also allows hexadecimal in the HTML5 ref. I will check that. Thanks for noticing.

t-jacobi commented 4 years ago

I can confirm that this is done on 0.5.8