Skip to content

Support for non alphabetical unicode characters - #58

Open
nisanthchunduru wants to merge 10 commits into
hgmnz:masterfrom
SupportBee:better_unicode_support
Open

Support for non alphabetical unicode characters#58
nisanthchunduru wants to merge 10 commits into
hgmnz:masterfrom
SupportBee:better_unicode_support

Conversation

@nisanthchunduru

Copy link
Copy Markdown

Hi,

I've added support for some non alphabetical unicode characters like currency characters, math characters, symbols etc.

Previously, non alphabetical unicode characters were gobbled up

[2] pry(main)> utf8_html
=> "+<br />ー<br />〜<br />=<br />─<br />a (with double-byte space)<br />¥<br />&<br />%<br />#<br />$<br />!<br />?<br />><<br />・<br />/<br />「」<br />@<br />、。"
[3] pry(main)> TruncateHtml::HtmlTruncator.new(TruncateHtml::HtmlString.new(utf8_html), length: 1000).truncate
=> "<br />ー<br />〜<br /><br /><br />a(with double-byte space)<br /><br />&<br />%<br />#<br /><br />!<br />?<br /><br />・<br />/<br />「」<br />@<br />、。"

Now

[2] pry(main)> utf8_html
=> "+<br />ー<br />〜<br />=<br />─<br />a (with double-byte space)<br />¥<br />&<br />%<br />#<br />$<br />!<br />?<br />><<br />・<br />/<br />「」<br />@<br />、。"
[3] pry(main)> TruncateHtml::HtmlTruncator.new(TruncateHtml::HtmlString.new(utf8_html), length: 1000).truncate
=> "+<br />ー<br />〜<br />=<br />─<br />a (with double-byte space)<br />¥<br />&<br />%<br />#<br />$<br />!<br />?<br />><<br />・<br />/<br />「」<br />@<br />、。"

@crv

crv commented Nov 25, 2015

Copy link
Copy Markdown

This is useful. @nisanth074 Can you also add temperature degree symbol (e.g. °C) in the list of your special characters ?

@wwcline

wwcline commented Mar 17, 2016

Copy link
Copy Markdown

This might (at least partly) resolve #35.

@rob-mcgrail

Copy link
Copy Markdown

Is this going to be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants