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

#24のテストコード #74

Merged
merged 5 commits into from
Mar 9, 2014
Merged

#24のテストコード #74

merged 5 commits into from
Mar 9, 2014

Conversation

inao
Copy link
Collaborator

@inao inao commented Mar 3, 2014

#24 のテストコードです

@gfx
Copy link
Collaborator

gfx commented Mar 3, 2014

#24 の受け入れテストですね。了解です。

@inao
Copy link
Collaborator Author

inao commented Mar 4, 2014

@gfx
はい、受け入れテストになります。
よろしければご対応くださいますと、ものすごく助かります(汗。

@inao
Copy link
Collaborator Author

inao commented Mar 4, 2014

なお最後のコミット555a95f は、 #24 は無関係です……。
いっしょくたにpull requestを出してしまい申し訳ございません。

@gfx
Copy link
Collaborator

gfx commented Mar 4, 2014

no problem!

@gfx
Copy link
Collaborator

gfx commented Mar 4, 2014

@inao <ParaStyle:半行アキ> が使われるのって何か意図ありますか? ふつうに変換すると <ParaStyle:本文> になるのでどうしたものかなと。

@inao
Copy link
Collaborator Author

inao commented Mar 4, 2014

どこかに<ParaStyle:半行アキ><ParaStyle:本文>の切り替えスイッチがあると思います。

以下の「InDesign出力」では、「空行半行アキ」(<ParaStyle:半行アキ>)と「空行1行アキ」(<ParaStyle:本文>)の切り替えスイッチがございますので。

http://md2inao.bloghackers.net/

上記実装周りについて詳しくは僕ではわかりませんので、 @naoya さんお願いできますでしょうか?

なお、テストコードを<ParaStyle:半行アキ>のほうで書いたのは、前後のテストもそうなっていたからです。

ちなみに<ParaStyle:半行アキ>の意図ですが、WEB+DB PRESS本誌や大部分のWEB+DB PRESS plusシリーズの書籍では、箇条書きや、(別ボックスの「リスト」や「図」ではない)本文中のコードやコマンドブロックの前行は、見栄え調整のため<ParaStyle:半行アキ>を使っているからです。<ParaStyle:本文>だと空行が広すぎますので。

@naoya
Copy link
Owner

naoya commented Mar 4, 2014

$c->blank_stylehalf のときは半行アキになるようになってますね。

@gfx
Copy link
Collaborator

gfx commented Mar 9, 2014

というと、半行アキ/本文についてはテストする必要はなく、テストのほうを 本文 になおしていいってことですね。

@gfx
Copy link
Collaborator

gfx commented Mar 9, 2014

あ、なんか勘違いしてました。問題なく実装できました。

@gfx
Copy link
Collaborator

gfx commented Mar 9, 2014

#14 もこのfenced codeを使えば解決できると思われます!

gfx added a commit that referenced this pull request Mar 9, 2014
@gfx gfx merged commit a5a3479 into master Mar 9, 2014
@gfx gfx deleted the inao20140304 branch March 9, 2014 14:30
gfx added a commit that referenced this pull request Mar 9, 2014
Changelog diff is:

diff --git a/Changes b/Changes
index 360d840..2964115 100644
--- a/Changes
+++ b/Changes
@@ -1,4 +1,7 @@

+0.09 2014-03-09 23:33:52+0900
+    - Resolve #24, #74, #81, #79, #80, #78
+
 0.08 2014-03-04 21:55:40+0900
     - Fix a packaging issue for older perls
gfx added a commit that referenced this pull request Mar 9, 2014
Changelog diff is:

diff --git a/Changes b/Changes
index 2964115..086d789 100644
--- a/Changes
+++ b/Changes
@@ -1,4 +1,7 @@

+0.10 2014-03-10 08:32:33+0900
+    - Add --version option to md2inao.pl
+
 0.09 2014-03-09 23:33:52+0900
     - Resolve #24, #74, #81, #79, #80, #78
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.

3 participants