Descriptionをmarkdownで書けるようにした kentaro/triglav#34

[glidenote]

#34 に対応しました…

[kentaro]

簡単なものでいいのでhelperのテストかいてほしー。

# Title

- foo
-bar

hoge

が

<h1>Title</h1>
<ul>
  <li>foo</li>
  <li>bar</li>
</ul>
<p>hoge</p>

になるぐらいかな。あとはscript通さないようにするとかそのあたり。

[kentaro]

あとhtml_safeしてるとscriptとか通っちゃいそうですね。以下とかみて、適切なオプションを有効にするとかかなー。

https://github.com/ged/bluecloth/blob/master/lib/bluecloth.rb#L65

[glidenote]

なるほど。ちょっと勉強します…

optionは対応しました。

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb                                                                                                                              
index be4601f..98df4dd 100644                                                                                                                                                                                   
--- a/app/helpers/application_helper.rb                                                                                                                                                                         
+++ b/app/helpers/application_helper.rb                                                                                                                                                                         
@@ -28,6 +28,7 @@ module ApplicationHelper                                                                                                                                                                      
   end                                                                               

   def markdown(text)                                                                                       
-    BlueCloth.new(text).to_html.html_safe                                                                                                                                                                      
+    options = [:escape_html => true, :auto_links => true,]                                                                                                                                                     
+    BlueCloth.new(text, *options).to_html.html_safe                                                                                                                                                            
   end                                                           
 end

[glidenote]

指摘の内容を修正しました…

[kentaro]

expect(helper.markdown(markdown_text)).to be == html_text.chomp にしてください。

[kentaro]

👍

[glidenote]

指摘もらった部分を修正するとコメントごと消えるんですね…

[glidenote]

app/helpers/application_helper.rbも修正済みしました

[kentaro]

Great! Thanks!!1

[kyanny]

@glidenote

http://192.168.25.37/gyazo/data/7bef06f87e472f8ae5e4f8a766f37709.png
show outdated diff をクリックすると過去のレビューコメントが表示できます。

[kentaro]

社内LANの貼らないようにｗ

[glidenote]

@kyanny
おお、ありがとうございます!!
知らなかった

@kentaro
社外からも見れるように画像転載しました

[kyanny]

oops public repo でした失礼