Instead of having the "Testbed results" listed under the attachments what about merging the two tables? I'm not sure if it is possible (cleanly) as I haven't dove into that code yet, but if so would people be ok with that?

CommentFileSizeAuthor
#1 pift_attachment.png4.99 KBboombatower
pift_attachment.png22.06 KBboombatower

Comments

boombatower’s picture

boombatower commented
StatusFileSize
new pift_attachment.png4.99 KB

Initial look.

I got some agreement in IRC and figured out this isn't that hard.

boombatower’s picture

boombatower commented
Status: Postponed (maintainer needs more info) » Fixed

Need confirmation that:

$comment->comment = preg_replace('/<table class="comment-upload-attachments">.*?<\/table>/s', '', $comment->comment);

Isn't a performance issue as hunmonk asked.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.