Open r12a opened 4 years ago
I created a PR for some of these comments at https://github.com/w3c/simple-ruby/pull/45
For remaining points:
modified in the light of any characters preceding and following
Light?
reworded: #57
line edge line head or the line end
Be consistent with terminology
made consistent: #57
In some cases, this document suggests optional methods to be allowed as implementation defined, such as that a ruby annotation wider than its base text should not overhang any preceding or following kana characters.
Awkward wording
rather than touching here, renamed title of another note, to make background clearer: #65
NOTE: Wrap opportunities
Seems like we need a whole section about how ruby interacts with justification. I don’t know why this is just a note.
updated: #68 placement of whole text along inline direction is not a part of placement of ruby.
points 1, 2, and 3 belong to the first step
The points are “considerations.” It’s not clear what “belong to” means here.
issue #74
then the end of the ruby annotation is aligned with the line’s end edge
… assuming the text is justified
added another line: #66
The spatial ratio of 1 unit before/after and 2 units between
This note doesn’t say anything distinct from content that comes right after it. No reason for the note to exist
updated text #67
base character string
Should use consistent terminology
made consistent: #57
points 1, 2, and 3 belong to the first step, and points 4 and 5 belong to the second.
This is true for all 3 forms, so there’s no need to call it out 3 different times. Also, better to use consistent formatting
updated: #57
All relevant PRs have merged. @kidayasuo close or not?
The following feedback was sent to me in email. Posting here with consent.
Too many pronouns. Unclear what this actually means.
Awkward phrasing
Run-on sentence. Also, the word “determined” is used twice in a row
Seems contrary to the thesis of this document, that “simple” placement is possible
What was the first hand?
Light?
Comma between “end” and “the”
Be consistent with terminology
Comma between “Also” and “methods”
Awkward wording
Seems like we need a whole section about how ruby interacts with justification. I don’t know why this is just a note.
Use either “font size” instead of “size”, or “ruby annotation characters” instead of “ruby"
Is this sentence necessary?
The points are “considerations.” It’s not clear what “belong to” means here.
What about if it consists of 0 or 1 characters?
These two paragraphs are almost identical. Surely there’s a better way rather than repetition
… assuming the text is justified
This note doesn’t say anything distinct from content that comes right after it. No reason for the note to exist
spacing
Should use consistent terminology
This is true for all 3 forms, so there’s no need to call it out 3 different times. Also, better to use consistent formatting
Reading this initially, I was confused because I thought it meant “all the ruby texts together in aggregate” rather than “each individual ruby text paired with its ruby base”. Perhaps this should say “each” instead of “all”.