Guarantee widget.Form tracks all entry validation changes, fixes #1965

[fpabl0]

Description:

Fixes #1965

Checklist:

• Tests included.
• Lint and formatter run with no errors.
• Tests all pass.

[andydotxyz]

The bug fix looks good, but the resulting test images show an invalid state - two focused elements at once. I don't know that fixing this is in scope for the ticket, but we should not commit invalid test images, so perhaps a line or to for Canvas.Focus() might help in the meantime?

[fpabl0]

Yes, I am aware of it, sorry about that, I tried to unfocus them, but have no success.

Canvas.Focus() might help in the meantime?

Oh, I will try with that, later today.

[fpabl0]

@andydotxyz Unfortunately, Canvas.Focus() didn't work, so I manually unfocused one entry, do now the test images look good? Or should I unfocus both of them?

[andydotxyz]

@andydotxyz Unfortunately, Canvas.Focus() didn't work, so I manually unfocused one entry, do now the test images look good? Or should I unfocus both of them?

Yeah that's OK for now. I suspect the focus issue requires a fix in the text code which we can look at separately.

[andydotxyz]

Looks good to me, though I would appreciate @Jacalz thoughts as well