Convert comment-wrapper.tpl.php to HTML5

tagging!

section seems to make sense, nodes and comments as articles, with section as the wrapper,

<article class="node">
  <h1>Node Title</h1>
  
  <section id="comments">
    <h2>Comments</h2>
    <article class="comment">
      <h3>Comment Title</h3>
    </article>
  </section>
 
</article>

There is another double >> in the line:

+<section id="comments" class="<?php print $classes; ?>"<?php print $attributes; ?>>

Rereading carefully, skip the above... But now am wondering if print $attributes includes a space before first attribute if $attributes is not empty.

Only thing actually changed is the element (swapped div for section), everything else is the same.

Sweet! Nice work guys ;)

This is still RTBC, just updating the sprint tag to keep track of it while we wait.

Committed to 8.x. Thanks.

Assuming fixed? Hooray, my first patch for Drupal 8 - very chuffed :)

Sweet! Congrats Jeff :D

I see that the comment section still has an "id='comment'" attribute. Did we want to eliminate the use of ids in our markup or am I remembering that wrong.

Removing ID's is a totally separate issue, I dont know if there is an open issue for this.

Nothing else needs to be done here. #comments ID is a useful anchor that we shouldn't remove IMO.

Cleaning up tags/component.

added cross link to comment.tpl.php conversion issue