Comment template: Call comment_class() before recursing into child comments

[michalczaplinski]

What?

The odd and even CSS classes were not being applied correctly to the comments in the Comment Template.

I've moved the call to comment_class() to BEFORE we recurse into the children so that it's in line with how comments are being rendered in the WordPress core.

Screenshots

I've added this CSS so that you can see the difference:

.even { 
  background-color: aquamarine;
}

.odd {
  background-color: lightblue;
}

This is what the core comments look like. The colors alternate, regardless of the comment level:

Before (left) and after (right)

Comments rendered with the Comments Query Loop:

[Mamaduka]

Left a small suggestion (non-blocking); otherwise, this looks good to me 👍

Thanks, @michalczaplinski.

[ockham]

Great catch @michalczaplinski! And thanks @Mamaduka for your review!

Going to merge once we go green 🙂

[ockham]

Oh, one more note, maybe we can add (or extend) a unit test in phpunit/class-block-library-comment-template-test.php to cover this behavior 🤔 I'll have a quick look.

[ockham]

Oh, one more note, maybe we can add (or extend) a unit test in phpunit/class-block-library-comment-template-test.php to cover this behavior 🤔 I'll have a quick look.

(I'll keep that for a follow-up.)

[gziolo]

Cherry-picked to the wp/6.0 branch for WordPress Beta 3 release.