Skip to content

Expanded UNICODE range for Latin characters from 33-255 to 33-383 #849

Merged
pissang merged 1 commit into
ecomfe:masterfrom
trajnisz:master
Nov 19, 2021
Merged

Expanded UNICODE range for Latin characters from 33-255 to 33-383 #849
pissang merged 1 commit into
ecomfe:masterfrom
trajnisz:master

Conversation

@trajnisz
Copy link
Copy Markdown

Hi, I'm doing a small project and noticed that in TextStyleProps we have overflow?: 'break' | 'breakAll' | 'truncate' | 'none'. The 'break' should break by word and it is, but not in all cases.

There is a list of unicode characters divided into blocks (https://en.wikipedia.org/wiki/List_of_Unicode_characters) and zrender only considers Latin characters in the range from 33 to 255 which is 1st and 2nd block. Many European characters are in the 3rd block (Latin Extended-A https://en.wikipedia.org/wiki/Latin_Extended-A).

Adding the 3rd UNICODE block should not cause any trouble (there are no characters that can be considered 'special') and would help a lot in using European languages because now some words are breaking in half :(

…k 'Latin Extended-A' - European characters)
@pissang
Copy link
Copy Markdown
Contributor

pissang commented Nov 19, 2021

Wow thanks for filling the gap! Sorry we didn't notice that!

@pissang pissang merged commit b5de6bd into ecomfe:master Nov 19, 2021
@trajnisz
Copy link
Copy Markdown
Author

Thank you for the quick reply. Keep up a good work!

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.

2 participants