redroot / holmes

Holmes is stand-alone diagnostic CSS stylesheet that can highlight potentially invalid or erroneous HTML(5) markup by adding one class
http://www.red-root.com/sandbox/holmes/index.html
698 stars 28 forks source link

Table Summary #21

Closed gnovaro closed 11 years ago

gnovaro commented 11 years ago

The summary attribute on the table element is obsolete. http://www.w3.org/TR/html-markup/table.html

justmarkup commented 11 years ago

Good catch, wasn't aware of this one. As far as I understand <caption> should be used instead, right?

Would be great if you could make a PR.

gnovaro commented 11 years ago

Insted of use caption element

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/table

<table>
  <caption>Awesome caption</caption>
  <tr>
    <td>Awesome data</td>
  </tr>
</table>    
justmarkup commented 11 years ago

Ok, so we should remove all warnings/errors for the summary attribute on the table element.

Regarding <caption> the only thing we can detect is if it's empy table > caption:empty {border: 1px solid red;} as with CSS we can not detect "table without a caption children element ".

Anything else missing regarding the table element?

gnovaro commented 11 years ago

Good. I think no. But if see other thing I will made a new report bug or a pull request with my own code. Thanks