Hi,
I wanted to share a fix.
I installed Ajax Comments, and the comments suddenly lost the indentation (but they were ordered correctly).
I fixed the issue changing ajax_comments.module on line 377 from
$prefix .= '<div class="ajax-comment-wrapper" id="comment-wrapper-' . $comment->cid . '"' . ($indent == TRUE ? ' class="indented"' : '') . '>';
to
$prefix .= '<div class="ajax-comment-wrapper' . ($indent == TRUE ? ' indented' : '') . '" id="comment-wrapper-' . $comment->cid . '">';

Before wasn't working cause (I use Chrome) the browser doesn't want multiple 'class' attribute in the same tag.

Hope it helps,
Stefano

CommentFileSizeAuthor
#1 indentation_fix-1792734-1.patch753 bytesavr

Comments

avr’s picture

avr commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new indentation_fix-1792734-1.patch753 bytes

I ran into this issue as well - here's an actual patch.

honza pobořil’s picture

honza pobořil commented
Status: Needs review » Reviewed & tested by the community

Patch #1 seems clean and works for me.

Stan.Ezersky’s picture

Stan.Ezersky commented

Neochief, include patch from comment #1 in module core, please.

brunodbo’s picture

brunodbo
he/him
commented

Confirming that this patch fixes the double class issue, and fixes indentation. thanks!

jean-eric’s picture

jean-eric commented

patch #1 works for me too

with a side effect however - which i can't say if caused by the patch or the views module, but expose here in case it could help anyone :
* i use views for displaying the last posts & last comments in the content region
* on the same page, i display blocks (my taxonomies) in a RIGHT sidebar region
* if the last comments view contains indented ones, the sidebar region displays below the content one !...

note that:
* this does not happen with a LEFT sidebar
* neither if not using views in the content region, but still displaying indented comments (article node)

muschpusch’s picture

muschpusch commented
Status: Reviewed & tested by the community » Closed (fixed)

committed to dev

muschpusch’s picture

muschpusch commented
Issue summary: View changes

Wrap html in code tag