{"action":"created","issue":{"url":"https://api.github.com/repos/whatwg/html/issues/3934","repository_url":"https://api.github.com/repos/whatwg/html","labels_url":"https://api.github.com/repos/whatwg/html/issues/3934/labels{/name}","comments_url":"https://api.github.com/repos/whatwg/html/issues/3934/comments","events_url":"https://api.github.com/repos/whatwg/html/issues/3934/events","html_url":"https://github.com/whatwg/html/pull/3934","id":351554241,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA5MTE3MDc1","number":3934,"title":"Revamp how fieldset and legend rendering is defined","user":{"login":"zcorpan","id":244772,"node_id":"MDQ6VXNlcjI0NDc3Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/244772?v=4","gravatar_id":"","url":"https://api.github.com/users/zcorpan","html_url":"https://github.com/zcorpan","followers_url":"https://api.github.com/users/zcorpan/followers","following_url":"https://api.github.com/users/zcorpan/following{/other_user}","gists_url":"https://api.github.com/users/zcorpan/gists{/gist_id}","starred_url":"https://api.github.com/users/zcorpan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zcorpan/subscriptions","organizations_url":"https://api.github.com/users/zcorpan/orgs","repos_url":"https://api.github.com/users/zcorpan/repos","events_url":"https://api.github.com/users/zcorpan/events{/privacy}","received_events_url":"https://api.github.com/users/zcorpan/received_events","type":"User","site_admin":false},"labels":[{"id":1016503498,"node_id":"MDU6TGFiZWwxMDE2NTAzNDk4","url":"https://api.github.com/repos/whatwg/html/labels/topic:%20fieldset","name":"topic: fieldset","color":"009800","default":false},{"id":349366641,"node_id":"MDU6TGFiZWwzNDkzNjY2NDE=","url":"https://api.github.com/repos/whatwg/html/labels/topic:%20rendering","name":"topic: rendering","color":"009800","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":14,"created_at":"2018-08-17T11:12:15Z","updated_at":"2018-08-29T09:12:50Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/whatwg/html/pulls/3934","html_url":"https://github.com/whatwg/html/pull/3934","diff_url":"https://github.com/whatwg/html/pull/3934.diff","patch_url":"https://github.com/whatwg/html/pull/3934.patch"},"body":"TODO explain what this does.\n\nFixes #3912, fixes #3955, fixes #3930, fixes #3929, fixes #3928, fixes #3927, fixes #3915, fixes #3913, fixes #3660, fixes #3331, fixes #2756.\n\n\n<!--\n    This comment and the below content is programatically generated.\n    You may add a comma-separated list of anchors you'd like a\n    direct link to below (e.g. #idl-serializers, #idl-sequence):\n\n    Don't remove this comment or modify anything below this line.\n    If you don't want a preview generated for this pull request,\n    just replace the whole of this comment's content by \"no preview\"\n    and remove what's below.\n-->\n***\n<a href=\"https://whatpr.org/html/3934/rendering.html\" title=\"Last updated on Aug 24, 2018, 2:58 PM GMT (e1ea70d)\">/rendering.html</a>  ( <a href=\"https://whatpr.org/html/3934/6f769b8...e1ea70d/rendering.html\" title=\"Last updated on Aug 24, 2018, 2:58 PM GMT (e1ea70d)\">diff</a> )"},"comment":{"url":"https://api.github.com/repos/whatwg/html/issues/comments/416883188","html_url":"https://github.com/whatwg/html/pull/3934#issuecomment-416883188","issue_url":"https://api.github.com/repos/whatwg/html/issues/3934","id":416883188,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNjg4MzE4OA==","user":{"login":"mstensho","id":13527206,"node_id":"MDQ6VXNlcjEzNTI3MjA2","avatar_url":"https://avatars0.githubusercontent.com/u/13527206?v=4","gravatar_id":"","url":"https://api.github.com/users/mstensho","html_url":"https://github.com/mstensho","followers_url":"https://api.github.com/users/mstensho/followers","following_url":"https://api.github.com/users/mstensho/following{/other_user}","gists_url":"https://api.github.com/users/mstensho/gists{/gist_id}","starred_url":"https://api.github.com/users/mstensho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mstensho/subscriptions","organizations_url":"https://api.github.com/users/mstensho/orgs","repos_url":"https://api.github.com/users/mstensho/repos","events_url":"https://api.github.com/users/mstensho/events{/privacy}","received_events_url":"https://api.github.com/users/mstensho/received_events","type":"User","site_admin":false},"created_at":"2018-08-29T09:12:50Z","updated_at":"2018-08-29T09:12:50Z","author_association":"NONE","body":"Thanks @MatsPalmgren . Sorry for being unclear, but my test case doesn't demonstrate any bug in Gecko. It works great! Padding is resolved correctly with respect to the containing block, the outer DIV. So, Gecko creates an anonymous HTMLScroll(fieldset) with a Block(fieldset) inside. So, padding is inherited from FieldSet(fieldset) into the HTMLScroll and further into the Block(fieldset), and there it's apparently correctly resolved against the DIV, and at the same time ignored on both the FieldsSet and the HTMLScroll. I see that you have some special code for this. If we say in the spec that we use an anonymous child, we also need to spec this special padding handling somehow."}}
{"actor":{"display_login":"mstensho"}}