Skip to content

Improve handling SMS verification field input

Matti Viljanen requested to merge improve-sms-code-filtering into master

Simplify the SMS verification code text field handling. Tested with following:

  • Type '123456'
  • Type '123-456' (you can't type the dash separately however)
  • Paste '123456'
  • Paste '123-456'
  • Paste '123456789' » '123-456'
  • Paste 'Your SIGNAL verification code is: 123456' » '123-456'

Not tested in-application, but with a mock-up application instead.

Fixes #443 (closed)

The second commit replaces using errorHighlight property (defaults to !acceptableInput) with rightItem which shows a checkbox instead, when the input is correct. As above, the changes were tested in a mock-up application.

The only logical change is to the initial password fields; the second field shows the checkbox only if it's non-empty and its text matches the first fields text.

Fixes #444 (closed)

Edited by Matti Viljanen

Merge request reports