Skip to content
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

モノルビとグループルビの使い分けに対応した #111

Merged
merged 2 commits into from
Oct 26, 2015
Merged

Conversation

inao
Copy link
Collaborator

@inao inao commented Oct 23, 2015

辟易(へき えき)のような文字単位でルビをふるモノルビと、
欠伸(あくび)のような熟語単位でルビをふるグループルビの使い分けに対応しました。

従来は一律グループルビにし、モノルビにすべきものは手動で変換していました。

@murashitas
変換があっているかや、ほかの文字でもうまくいくかを、InDesignに取り込んでみてご確認いただけますでしょうか?

辟易(へき えき)のような文字単位でルビをふるモノルビと、
欠伸(あくび)のような熟語単位でルビをふるグループルビの使い分けに対応しました。

従来は一律グループルビにし、モノルビにすべきものは手動で変換していました。
@murashitas
Copy link

現状のルビ系の用語統一にあるもので試してみました。

グループルビが「所以」しかないのでわかりづらいですが(最初とかに置けばよかったですね……すみません)うまくいっているようです。

@inao inao changed the title [WIP] モノルビとグループルビの使い分けに対応した モノルビとグループルビの使い分けに対応した Oct 26, 2015
@inao
Copy link
Collaborator Author

inao commented Oct 26, 2015

@murashitas
ありがとうございました!

@naoya
動作的には問題ないようです。
コード的にも大丈夫でしたら、マージしていただけませんでしょうか?

@murashitas
Copy link

あ、READMEも更新しなきゃですね。以下のような感じでしょうか。(forkするほどでもないので、お願いしてもよいですか? @inao


モノルビ(一つの漢字ごとに読みを示す)を振りたい場合はmonorubyクラスを使用し、カッコ内で一文字ごとに半角スペースを挟んでください。

<span class='monoruby'>辟易(へき えき)</span>

グループルビ(熟字訓など、まとめて読みを示す)を振りたい場合はgrouprubyクラスを使用してください。

<span class='groupruby'>欠伸(あくび)</span>

@inao
Copy link
Collaborator Author

inao commented Oct 26, 2015

@murashitas

ありがとうございます!
そのまんまコピペさせていただきました。

@naoya

今度こそ、コード的に問題なければマージしていただいて大丈夫です 🙇

@naoya
Copy link
Owner

naoya commented Oct 26, 2015

LGTM

naoya added a commit that referenced this pull request Oct 26, 2015
モノルビとグループルビの使い分けに対応した
@naoya naoya merged commit 9e7dfc2 into master Oct 26, 2015
@naoya naoya deleted the ruby branch October 26, 2015 10:47
@inao
Copy link
Collaborator Author

inao commented Oct 26, 2015

ありがとうございます!

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.

None yet

3 participants