-
Notifications
You must be signed in to change notification settings - Fork 14
Why the range U+0320–U+03FF when computing spacing? #169
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
Comments
|
@fred-wang I think it's reasonable to ask though why that range, especially as it uses all the standard Greek code points. Why isn't a range from the Private use area used here, as it's just an internal mapping of the tables. |
AFAIK, it still possible to use PUA characters in |
ah 14 bits 03FF which explains the range, which I guess answers @NSoiffer's question. Maybe we should say that so it doesn't look like we are ignoring Greek. I agree it makes no difference in practice as single letter Greek, like single letter Latin is never going to need an opdict entry so the slots are "free" |
I'm not sure what's the next actionable step. AFAIK the text in the spec is correct and covered by tests. |
Choosing this range is cleaver but "random" (there are plenty of other ranges from other alphabets that I think could be used). I think an informative note (just one or two sentences similar to your comment) in the spec as to why this is done is appropriate. Specs should not have mysteries buried in them. |
This is separated out from #167 since the other issues are settled and it should be closed for CR.
Core says:
That ranges makes no sense to me. It covers part of the combining chars and also the Greek/Coptic chars. I think maybe it is trying to capture the combining chars, but the combining chars range is U+0300 - U+036F. There are additional combining chars 1AB0–1AFF and 1DC0–1DFF that maybe should be included.
And from a later comment:
I still don't see why U+0320–U+03FF makes sense. Why are some combining chars included in the range and not others? Why is a Greek alpha treated different than a latin a? Although you (@fred-wang) don't need include text in the spec why this is so, it seems like a bug to me so you should explain why it isn't a bug.
The text was updated successfully, but these errors were encountered: