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

SmartyPants: even occurrences of double quotes are not recognized #183

Closed
AlekseyMartynov opened this issue Dec 13, 2017 · 1 comment · Fixed by #360
Closed

SmartyPants: even occurrences of double quotes are not recognized #183

AlekseyMartynov opened this issue Dec 13, 2017 · 1 comment · Fixed by #360

Comments

@AlekseyMartynov
Copy link

0.14.8

var p = new MarkdownPipelineBuilder()
    .UseSmartyPants()
    .Build();

var html = Markdown.ToHtml("\"aaa\" - \"bbb\" - \"ccc\" - \"ddd\"", p);

html is <p>&ldquo;aaa&rdquo; - &quot;bbb&quot; - &ldquo;ccc&rdquo; - &quot;ddd&quot;</p>

Note the alternation of ldquo-rdquo and quot-quot

@xoofx xoofx added the bug label Dec 14, 2017
@deanebarker
Copy link

I've found this too, but weirdly, it's alternating "sets" of quotes:

You spend a lot of time saying things like, “Where in
the world did all of this come from?” and "I don’t even
remember this" and “Who put this here?”

Three quoted phrases. The middle one ("I don't even remember this") retains straight quotes, while the other two have their quotes replaced.

MihaZupan added a commit to MihaZupan/markdig that referenced this issue Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants