This is a variation on a previous issue (I'll add it later) that requested the text be trimmed without including the HTML in the count. Patch to come soon.

CommentFileSizeAuthor
#4 html_trim-2363123-4.patch14.72 KBmarkie
#2 TruncateHTML.php_.txt5.94 KBmarkie
#1 2363123-html_trim-1.patch5.63 KBmarkie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markie’s picture

markie CreditAttribution: markie commented
FileSize
2363123-html_trim-1.patch5.63 KB

as promised. Patch..

markie’s picture

markie CreditAttribution: markie commented
FileSize
TruncateHTML.php_.txt5.94 KB

Diff did not add the TruncateHTML class. So here it is in all it's txt glory.

markie’s picture

markie CreditAttribution: markie commented
Status: Active » Needs review

bah! Review please.

markie’s picture

markie CreditAttribution: markie commented
FileSize
html_trim-2363123-4.patch14.72 KB

Updated patch to fix some code sniffer issues and properly insert TruncateHTML class into patch.

  • 3e01ab3 committed on 8.x-1.x 
    Issue #2363123 by markie: Trim excluding HTML (without stripping)
markie’s picture

markie CreditAttribution: markie at Mediacurrent commented
Status: Needs review » Fixed

patch was already committed. Closing out

Status: Fixed » Closed (fixed)

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