-
Notifications
You must be signed in to change notification settings - Fork 321
Update error-identification.html #4694
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for wcag2 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Adding second requested change in the document
<a href="error-suggestion">3.3.3 Error Suggestion</a>. Often, the error description can be phrased so that it meets both SC 3.3.1 and SC 3.3.3 at the same time. For instance, "Please provide a valid email address" conveys that the email field is empty or invalid, and also how it can be fixed. | ||
</p> | ||
<p>An "input error" is information provided by the user that is not accepted. This includes:</p> | ||
<p>An "input error" is input data that has failed validation. This includes:</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<p>An "input error" is input data that has failed validation. This includes:</p> | |
<p>An "input error" is caused by information provided by the user that has failed validation. This includes:</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about >>An "input error" is caused by input data that has failed validation. This includes:<<
The data is not the error, but causes it. The data might have been entered by som tool or AI, not by the user.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can remove this sentence entirely, as the normative section already links to the definition: input error
IMO, we can remove enirely:
An "input error" is information provided by the user that is not accepted. This includes:
- information that is required by the web page but omitted by the user, or
- information that is provided by the user but that falls outside the required data format or allowed values.
And replace the sentence below:
For example:
with:
Examples of "input error" include:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The task force was still in the process of debating the wording of the change at line 33. Note that the original suggested wording in the issue was:
An "input error" is input data that has failed validation for the application preventing access to further processing. This includes:
Closes #4675 by making suggested modifications clarifying the difference between Error Identification and Error Suggestions, and acknowledging that it's possible to meet both in one message.