For example truncating the HTML string "Hello <b>world</b>" to 10 characters would result in the string "Hello w" (with a bold "w"). Any non technical user would count 7 characters instead of 10. This module would be much more useful if it will only count visible characters.

CommentFileSizeAuthor
#2 2898983-trim-on-visible-characters.patch2.01 KBmauritsl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mauritsl created an issue. See original summary.

mauritsl’s picture

mauritsl CreditAttribution: mauritsl commented
Status: Active » Needs review
FileSize
2898983-trim-on-visible-characters.patch2.01 KB

Patch attached fixes this issue.

mauritsl’s picture

mauritsl CreditAttribution: mauritsl commented
Issue summary: View changes
kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented
Related issues: +#2923142: Strip HTML before counting characters for trimming.

This might be my issue as well.

kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented
Status: Needs review » Reviewed & tested by the community

This does fix my issue.

JonMcL’s picture

JonMcL CreditAttribution: JonMcL commented
Related issues: +#2975871: Number of characters should not include invisible HTML (8.x)

Thank you for putting the time and effort into this. Hopefully maintainer will commit eventually.

I made a 8.x version of this patch here: #2975871: Number of characters should not include invisible HTML (8.x)

  • azovsky committed 4f384b6 on 7.x-1.x 
    Issue #2898983 by mauritsl: FIX Number of characters should not include...
azovsky’s picture

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

Status: Fixed » Closed (fixed)

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