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

#62 の受け入れテスト #94

Merged
merged 6 commits into from
May 19, 2014
Merged

#62 の受け入れテスト #94

merged 6 commits into from
May 19, 2014

Conversation

inao
Copy link
Collaborator

@inao inao commented May 13, 2014

@gfx
#62 の受け入れテストを書きました。

テストにミスがあるかもしれません。
ご確認願います。

@inao inao added the high label May 14, 2014
@gfx
Copy link
Collaborator

gfx commented May 18, 2014

@inao 対応しました。箇条書きと引用も CodeStyle:イタリック になりますが、それでいいですか?(テストの修正 529643a 参考)

@inao
Copy link
Collaborator Author

inao commented May 18, 2014

超ありがとうございます!

箇条書きは、元のテストのままのほうが助かるのですが、難しそうでしょうか?

CodeStyle:イタリックは、本文のように周囲が明朝体のところでは合うのですが、箇条書きのようにゴシック体のところでは合わないので、イタリック(変形斜体)にする必要があるのです。

引用は、明朝体なこともゴシック体なこともあるので、 CodeStyle:イタリックでかまいません。

@gfx
Copy link
Collaborator

gfx commented May 18, 2014

了解です。 b17dfe7 で対応しました。

ちなみに引用部分のフォントをどちらにすべきかはどういう条件で決まるんでしょう?

@gfx
Copy link
Collaborator

gfx commented May 18, 2014

あ、コラム本文も イタリック(変形書体) ですよね…。直しました。

@gfx
Copy link
Collaborator

gfx commented May 19, 2014

@inao ping!

@inao
Copy link
Collaborator Author

inao commented May 19, 2014

@gfx
すみませんすみません、どうもありがとうございました!

ちなみに引用部分のフォントをどちらにすべきかはどういう条件で決まるんでしょう?

フォーマット作成時にデザイナさん(たいていは @n-yuji )と相談して決めます。

いま手元のをパラパラと見ましたところ、WEB+DB PRESSの引用文はゴシックなんですが、書籍は明朝(今回の実装)なもののほうが多かったですね。

@gfx
Copy link
Collaborator

gfx commented May 19, 2014

なるほどー。じゃあひとまずこのP-Rはマージしても大丈夫でしょうか?

@inao
Copy link
Collaborator Author

inao commented May 19, 2014

はい、マージします!
おかげさまで今号から本文のイタリックが綺麗になりますっ。

inao added a commit that referenced this pull request May 19, 2014
#62 の受け入れテスト
@inao inao merged commit 66f8b42 into master May 19, 2014
@inao inao deleted the issue62 branch May 19, 2014 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants