I think we should use $variables['comment_classes'] instead of $variables['status'] in Drupal 6.x versions, because the comment's status transform into a comment's class in Drupal 7 only:

    if ($variables['status'] != 'comment-published') {
      $variables['classes_array'][] = $variables['status'];

Compare to Drupal 6

So in Drupal 6 the dimming does not work.

CommentFileSizeAuthor
#2 vud_comment-6.x-3.x-dev-dimming-1375068.patch564 bytesquotesbro
vud_comment.module.patch532 bytesquotesbro

Comments

Status: Needs review » Needs work

The last submitted patch, vud_comment.module.patch, failed testing.

quotesbro’s picture

quotesbro commented
StatusFileSize
new vud_comment-6.x-3.x-dev-dimming-1375068.patch564 bytes
quotesbro’s picture

quotesbro commented
Status: Needs work » Needs review
marvil07’s picture

marvil07 commented
Status: Needs review » Closed (cannot reproduce)

I can not reproduce the error, please reopen if you can reproduce this on a clean install and/or if you can provide details about your configuration.

+++ b/vud_comment/vud_comment.module
@@ -147,7 +147,7 @@ function vud_comment_preprocess_comment(&$variables) {
-    $variables['status'] .= ' vud-comment-dimmed';
+    $variables['comment_classes'] .= ' vud-comment-dimmed';

There is no such a $comment_classes variable rendered on the template nor referred on the preprocess.

BarisW’s picture

BarisW commented

There is in D7, maybe that's what caused the confusion. See: #1999978: Fix comment dimming