-
Notifications
You must be signed in to change notification settings - Fork 5
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
Editorial suggestions from Myles #44
Comments
I created a PR for some of these comments at #45 |
For remaining points:
reworded: #57
made consistent: #57
rather than touching here, renamed title of another note, to make background clearer: #65
updated: #68
issue #74
added another line: #66
updated text #67
made consistent: #57
updated: #57 |
added lines from Bin-sensei's update (C-#44-6)
All relevant PRs have merged. |
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”.
The text was updated successfully, but these errors were encountered: