`\` dosen't work well on "_" #214

Closed
jjyr opened this Issue Mar 20, 2013 · 5 comments

Comments

Projects
None yet
4 participants

jjyr commented Mar 20, 2013

render.render("\\_dd_")
=> "<p>_dd_</p>\n"
#is ok

but

render.render("_dd\\_dd_")
=> "<p><em>dd\\</em>dd_</p>\n"
#unexpect, it should be
#<p><em>dd_dd</em></p>\n
Collaborator

mattr- commented Apr 4, 2013

What options are you using when you parsing the markdown document?

jjyr commented Apr 5, 2013

@mattr-
:autolink => true, :space_after_headers => false, strikethrough: true, :fenced_code_blocks => true

Collaborator

mattr- commented Apr 14, 2013

Awesome. Thanks! A bit behind on looking at issues at the moment, but I'll
get to this one as soon as I can.

On Fri, Apr 5, 2013 at 4:13 AM, jjy notifications@github.com wrote:

@mattr- https://github.com/mattr-
:autolink => true, :space_after_headers => false, strikethrough: true,
:fenced_code_blocks => true


Reply to this email directly or view it on GitHubhttps://github.com/vmg/redcarpet/issues/214#issuecomment-15945729
.

mattr- was assigned Jun 2, 2013

egonSchiele referenced this issue in egonSchiele/mdpress Aug 17, 2013

Closed

\ Escaping broken #25

Contributor

jcheatham commented Aug 20, 2013

Piggy-backing another broken evaluation case onto this issue (as they're superficially related at least):

renderer = Redcarpet::Render::HTML.new
markdown = Redcarpet::Markdown.new(renderer)
markdown.render("blah \\\\server\\path blah")
=> "<p>blah \\server\\path blah</p>\n"

mattr- removed their assignment Apr 15, 2014

@jcheatham jcheatham added a commit to jcheatham/redcarpet that referenced this issue May 28, 2014

@jcheatham jcheatham Fix emphasis escape sequence detection
Fixes issues #214 and #369
fe79ef9

@jcheatham jcheatham added a commit to jcheatham/redcarpet that referenced this issue May 28, 2014

@jcheatham jcheatham Fix emphasis escape sequence detection
Fixes issues #214 and #369
708cde3

@jcheatham jcheatham added a commit to jcheatham/redcarpet that referenced this issue May 28, 2014

@jcheatham jcheatham Fix emphasis escape sequence detection
Fixes issues #214 and #369
dc82a20
Collaborator

robin850 commented May 29, 2014

This issue has been addressed in #380 thanks to @jcheatham so we can give it a close. Thank you everyone! :-)

robin850 closed this May 29, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment