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

br is omitted from timeContainer list #171

Closed
nigelmegitt opened this issue Aug 1, 2016 · 2 comments
Closed

br is omitted from timeContainer list #171

nigelmegitt opened this issue Aug 1, 2016 · 2 comments

Comments

@nigelmegitt
Copy link
Contributor

In §12.2.4 timeContainer the list of elements for which timeContainer applies omits br, yet in §8.1.7 br it is clear that br should be in the list.

@skynavga
Copy link
Collaborator

skynavga commented Aug 1, 2016

Yes, since it can contain animation child elements, which may be timed.

On Mon, Aug 1, 2016 at 5:04 AM, Nigel Megitt notifications@github.com
wrote:

In §12.2.4 timeContainer the list of elements for which timeContainer
applies omits br, yet in §8.1.7 br it is clear that br should be in the
list.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#171, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXCb_pVXj4wNXLK_G3cb3ZDjzSCE8HIks5qbdKzgaJpZM4JZecV
.

@nigelmegitt
Copy link
Contributor Author

Note that I've provided a fix for it in PR #172 (hopefully that's already obvious but I think the github UI can maybe make this a bit subtle, that the reference is actually from a PR that fixes the issue)

@skynavga skynavga added this to the Editor's WR Work List milestone May 25, 2017
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