{"action":"opened","issue":{"url":"https://api.github.com/repos/htacg/tidy-html5/issues/210","labels_url":"https://api.github.com/repos/htacg/tidy-html5/issues/210/labels{/name}","comments_url":"https://api.github.com/repos/htacg/tidy-html5/issues/210/comments","events_url":"https://api.github.com/repos/htacg/tidy-html5/issues/210/events","html_url":"https://github.com/htacg/tidy-html5/issues/210","id":72806799,"number":210,"title":"Is it time to bring back the table missing summary warning?","user":{"login":"geoffmcl","id":1333435,"avatar_url":"https://avatars.githubusercontent.com/u/1333435?v=3","gravatar_id":"","url":"https://api.github.com/users/geoffmcl","html_url":"https://github.com/geoffmcl","followers_url":"https://api.github.com/users/geoffmcl/followers","following_url":"https://api.github.com/users/geoffmcl/following{/other_user}","gists_url":"https://api.github.com/users/geoffmcl/gists{/gist_id}","starred_url":"https://api.github.com/users/geoffmcl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geoffmcl/subscriptions","organizations_url":"https://api.github.com/users/geoffmcl/orgs","repos_url":"https://api.github.com/users/geoffmcl/repos","events_url":"https://api.github.com/users/geoffmcl/events{/privacy}","received_events_url":"https://api.github.com/users/geoffmcl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-05-03T13:30:07Z","updated_at":"2015-05-03T13:30:07Z","closed_at":null,"body":"A post on `html-tidy@w3.org` https://lists.w3.org/Archives/Public/html-tidy/2015AprJun/0009.html caused me to look deeper into the code concerning this missing table summary warning.\r\n\r\nAs stated in my reply, there are perhaps 2, or more, bugs here!\r\n\r\nCase 1: First at present tidy5 no longer emits this warning in **ANY** case, which may be fine for html5 mode... But is it?\r\n\r\nCase 2: Conversely, if the document **IS** html5, and you **DO** add a summary attribute, tidy5 does **NOT** warn that the summary attribute is not supported in HTML5 ;=(( Should it?\r\n\r\nCase 3: Further, if the doctype declares the document as an earlier doctype, like say 4.01 strict, or transitional, htm4--, tidy5 still does not emit the warning ;=(( Should it?\r\n\r\nThe missing summary warning was **REMOVED** back on Nov 21, 2011, commit/4fdc30c (https://github.com/htacg/tidy-html5/commit/4fdc30c) by @sideshowbarker, with the comment \"summary attribute is not required on table in HTML5\". That seems fine if tidy5 was only ever meant to support html5...\r\n\r\nBut now we want it to support **ALL** html, so is it time to bring back this warning? \r\n\r\nShould it **ALWAYS** be there as it was in tidy before 2011? \r\n\r\nOr should an appropriate warning **ONLY** be issued for Cases 2 and/or 3 above?\r\n\r\nOr other ideas... \r\n\r\nI seek comments in this before jumping down a path ;=)).\r\n"}}