New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: rewrite to meet w3c typography #68
Conversation
BREAKING CHANGE: * rename `characters` with `letters` * functions are now returning a Charset (https://github.com/ikatyang/regexp-util#charset)
src/index.ts
Outdated
const punctuation_charset = unicode({ | ||
const cjk_letters = unicode({ | ||
Script: ['Han', 'Katakana', 'Hiragana', 'Hangul', 'Bopomofo'], | ||
General_Category: ['Other_Letter', 'Letter_Number', 'Other_Symbol'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Modifier_Letter
should be included such as <U+3005> 々
Codecov Report
@@ Coverage Diff @@
## master #68 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 1 1
Lines 9 9
=====================================
Hits 9 9
Continue to review full report at Codecov.
|
d87ae92
to
53e2937
Compare
LGTM. The PR should be sufficient for the context of While I think we should wait for the graduation of css-text-level 4 and the standard alignment should be done on the side of Edits: As for this PR, it is good to merge. |
Fixes #48
Closes #47
@JLHwung
Sorry for the delay, I somehow missed this issue. And I'm not too familiar with unicode, would you be able to review this PR?