-
Notifications
You must be signed in to change notification settings - Fork 16
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
conformsTo single and/or array of literals #106
Comments
Yes, it seems that the processing algorithm should begin by looking at this explicitly, a bit like it does for context, and the user agent should determine whether it can do something or not with that profile(s). |
Maybe the easy way out of this is to note in the explanation that at this stage of the processing algorithm, the value could be either a string or an array so the actual processing will need to take that into account. I don't like the idea of altering the value type just to account for this one quirk in the algorithm. |
I'm not sure I understand... |
I'm envisaging wording to obtain the profile like you put in #101, but also explaining why we talk about single values and lists in the explanation (i.e., that the normalization happens later). That way we leave conformsTo as an "Array of Literals" even though it won't (always) be at processing time, or not until the end of the algorithm. |
@mattgarrish, this seems to be settled in the version after the merge of #103 (modulo a separate comment #101 (comment)). If you agree, we should close the issue. |
Closing this with #116 |
conformsTo
can be a string:but for the processing algorithm to make sense we can't rely on an array translation, otherwise the profile can't be determined until after the value normalization steps (too late), or would have to occur in the middle of them so any extension steps can be run (very odd).
In this one case, we should probably explicitly allow either a single value or array of values and leave it to user agents to check the value type while obtaining the profile.
The text was updated successfully, but these errors were encountered: