-
Notifications
You must be signed in to change notification settings - Fork 249
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
Normative wording inconsistencies for Accessible Authentication SCs #2715
Comments
Thanks @mbgower. I think both suggestions make sense. I would prefer adding “exists” for the second part. |
With regard to (1), I think it may have been sub optimal for 1.4.3 to have the (minimum) qualifier. We could go either way at this point. To be contrary, I will point out that we could have (No Exception) variants of other SC and it seems better to me that 2.1.1 is not named Keyboard (Minimum). For (2) I think if you add "exists" to the first bullet, I think you will find that you need to a bunch more words. to qualify and provide context wrt "step in a process". |
??? |
For (1), I agree it is odd. I think we updated the structure without re-considering the names of the SCs. I'd suggest leaving the AA SC and adding " (Enhanced)" to the AAA version. For (2), I think it works better as-is. The preamble is asking "that step provides at least one of the following:", and you can read each as a thing provided in the authentication step. I've created branch Issue2715-accessible-auth-name that includes the name-change, please use that for other updates related to this issue. |
For both 3.3.7 and 3.3.8, there are a number of trivial language considerations that could be advanced to improve consistency.
Depending on which way we want to go, I'm happy to do a PR for these
The text was updated successfully, but these errors were encountered: