{"action":"created","issue":{"url":"https://api.github.com/repos/whatwg/wattsi/issues/94","repository_url":"https://api.github.com/repos/whatwg/wattsi","labels_url":"https://api.github.com/repos/whatwg/wattsi/issues/94/labels{/name}","comments_url":"https://api.github.com/repos/whatwg/wattsi/issues/94/comments","events_url":"https://api.github.com/repos/whatwg/wattsi/issues/94/events","html_url":"https://github.com/whatwg/wattsi/pull/94","id":378376895,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI5MDk2OTI3","number":94,"title":"Refine handling of MDN annotations","user":{"login":"sideshowbarker","id":194984,"node_id":"MDQ6VXNlcjE5NDk4NA==","avatar_url":"https://avatars2.githubusercontent.com/u/194984?v=4","gravatar_id":"","url":"https://api.github.com/users/sideshowbarker","html_url":"https://github.com/sideshowbarker","followers_url":"https://api.github.com/users/sideshowbarker/followers","following_url":"https://api.github.com/users/sideshowbarker/following{/other_user}","gists_url":"https://api.github.com/users/sideshowbarker/gists{/gist_id}","starred_url":"https://api.github.com/users/sideshowbarker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sideshowbarker/subscriptions","organizations_url":"https://api.github.com/users/sideshowbarker/orgs","repos_url":"https://api.github.com/users/sideshowbarker/repos","events_url":"https://api.github.com/users/sideshowbarker/events{/privacy}","received_events_url":"https://api.github.com/users/sideshowbarker/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-11-07T16:51:52Z","updated_at":"2018-12-18T22:54:48Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/whatwg/wattsi/pulls/94","html_url":"https://github.com/whatwg/wattsi/pull/94","diff_url":"https://github.com/whatwg/wattsi/pull/94.diff","patch_url":"https://github.com/whatwg/wattsi/pull/94.patch"},"body":"The commits here ensure that MDN annos for headings go into right split\r\n\r\nThis commits here:\r\n\r\n* Ensure that in multipage output, any MDN annotations headings associated with heading IDs show up in the same output file (split) as the heading.  Otherwise without this change, MDN annos for headings can end up orphaned in the previous (split) output file.\r\n\r\n* Cause the markup for MDN annos for `p`-element-descendant elements to be inserted after the p ancestor (`</p>` end tag).  Without this change, the annos otherwise get inserted before the `p` ancestor, which is suboptimal because in many cases that results in the anno overlapping with the spec text (paragraph content) of the spec. Inserting the annos after the p ancestor minimizes the number of cases of such overlap."},"comment":{"url":"https://api.github.com/repos/whatwg/wattsi/issues/comments/448402537","html_url":"https://github.com/whatwg/wattsi/pull/94#issuecomment-448402537","issue_url":"https://api.github.com/repos/whatwg/wattsi/issues/94","id":448402537,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0ODQwMjUzNw==","user":{"login":"domenic","id":617481,"node_id":"MDQ6VXNlcjYxNzQ4MQ==","avatar_url":"https://avatars0.githubusercontent.com/u/617481?v=4","gravatar_id":"","url":"https://api.github.com/users/domenic","html_url":"https://github.com/domenic","followers_url":"https://api.github.com/users/domenic/followers","following_url":"https://api.github.com/users/domenic/following{/other_user}","gists_url":"https://api.github.com/users/domenic/gists{/gist_id}","starred_url":"https://api.github.com/users/domenic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/domenic/subscriptions","organizations_url":"https://api.github.com/users/domenic/orgs","repos_url":"https://api.github.com/users/domenic/repos","events_url":"https://api.github.com/users/domenic/events{/privacy}","received_events_url":"https://api.github.com/users/domenic/received_events","type":"User","site_admin":false},"created_at":"2018-12-18T22:54:48Z","updated_at":"2018-12-18T22:54:48Z","author_association":"MEMBER","body":"Sorry for the delay on this.\r\n\r\nIt seems like there's a simpler version of this that just uses the existing code, but uses TargetAncestor.NextSibling, inside InsertMDNAnnotationForElement. Let me post a draft to show what I'm thinking, and you can tell me if I'm crazy or not."}}
{"actor":{"display_login":"domenic"}}