imo, the time ago text should really be wrapped in a span so that the themer has more control over styling

CommentFileSizeAuthor
#1 recent-comments.patch1006 bytesJacine
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jacine’s picture

Jacine
she/her
Location New York
CreditAttribution: Jacine commented
Status: Active » Needs review
FileSize
recent-comments.patch1006 bytes

Yeah, the <br /> is a really annoying and outdated assumption for how this data should be displayed. Here's a simple patch that removes the <br /> and wraps the date in a <span>.

bdawg8569’s picture

bdawg8569 CreditAttribution: bdawg8569 commented
Status: Needs review » Reviewed & tested by the community

Patch worked correctly for me.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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