Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Contact Form Displays Error Prematurely #2404

Open
justinr86 opened this issue Oct 21, 2020 · 2 comments
Open

Contact Form Displays Error Prematurely #2404

justinr86 opened this issue Oct 21, 2020 · 2 comments
Assignees
Labels
bug Something isn't working medium priority not high, not low, but medium!

Comments

@justinr86
Copy link
Contributor

Summary

The Contact Info page validates the data with the following rules:

  1. Full name is required

  2. Either phone number or email address is required

  3. Both email and phone number are evaluated for valid format

The error message is displayed as soon as one field has been touched. We should display an error to the user in the following scenarios:

  1. Invalid data is present in the field (i.e. a malformed email address).

  2. The user attempts to submit a form missing required information.

This logic for presenting error messages is demonstrated on the When Did It Happen page.

Steps to reproduce

On Contact Info page focus on an input field by either clicking in the field with your mouse or using the Tab key. Move focus away from this input field and you will see error messages displayed.

Unresolved questions

N/A

@justinr86 justinr86 added bug Something isn't working needs prioritization We have to prioritize this issue labels Oct 21, 2020
@dengcn dengcn self-assigned this Oct 21, 2020
@lmcbhvu
Copy link
Contributor

lmcbhvu commented Oct 21, 2020

I see it too but I thought it is the way to design. Currently, the error is still displayed when the user enters data in required field.

@dengcn
Copy link
Contributor

dengcn commented Oct 21, 2020

It really depends on user's feeling.
Current error display is like real time prompt. Let's say user type in valid full name, and move away the mouse to focus on empty place, then warning pop up can prompt warning to user, you need to fill in other required fields before click "Continue"

We can take business team's feedback/requirement, to see if use this real time style approach or validate all only on when user click "Continue"

@ngosset ngosset added medium priority not high, not low, but medium! and removed needs prioritization We have to prioritize this issue labels Oct 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working medium priority not high, not low, but medium!
Projects
None yet
Development

No branches or pull requests

4 participants