Skip to content
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

Add audio, image in presentation contexts to content elements. #891

Closed
skynavga opened this issue Jul 15, 2018 · 3 comments
Closed

Add audio, image in presentation contexts to content elements. #891

skynavga opened this issue Jul 15, 2018 · 3 comments

Comments

@skynavga
Copy link
Collaborator

skynavga commented Jul 15, 2018

I reviewed all 99 instances of references to the term content element, and all of them should include and not exclude audio and image descendants of body. Furthermore, these elements need to be added to 10.4.4.4 step (1) and 10.2.1 (the list of elements on which @style applies - image was already present, but missing audio).

This is an editorial change to correct an inadvertent omission which is substantially backed up by normative text elsewhere, e.g., both audio and image accept style properties which are applied to them, and, therefore, style resolution must apply (in contradiction to the [filter] step of 10.4.4.4 and the [associate region] procedure (defined in terms of content elements). In addition, presentation processing explicitly applies, e.g., see [presentation related element].

@nigelmegitt
Copy link
Contributor

I am querying whether this is truly editorial - to make the argument clearly I think we need more details about the existing normative text that implies that these changes are not substantive. It's going to be tricky because if there are such parts of the text then these changes apparently fall under the third bullet of class 3 in the Process document's Classes of Change, i.e. they clear up an ambiguity, whose source was the previous inconsistency.

@skynavga skynavga modified the milestones: PR, CR3 Jul 19, 2018
@skynavga
Copy link
Collaborator Author

Taking a conservative position, I am relabeling this as substantive.

@skynavga skynavga removed their assignment Jul 20, 2018
@skynavga
Copy link
Collaborator Author

Merged early per WG resolution to move to CR3 CfC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants