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

With Markdown enabled double tilde strikethrough does not work in v.4 #1078

Open
IuliiSe opened this issue Jun 28, 2017 · 3 comments
Open

With Markdown enabled double tilde strikethrough does not work in v.4 #1078

IuliiSe opened this issue Jun 28, 2017 · 3 comments

Comments

@IuliiSe
Copy link

IuliiSe commented Jun 28, 2017

Instead of getting a strikethrough text the pair of tildes are rendered as text. It work while text is being edited, but fails when text is in preview mode.

@dbiere
Copy link

dbiere commented Sep 12, 2017

The issue is that SimpleMDE (used during editing) enables strikethrough by default, while the Laravel-Markdown CommonMark wrapper library (used for viewing the doc) does not support strikethrough.

From the SimpleMDE config guide...

strikethrough: If set to false, will not process GFM strikethrough syntax. Defaults to true.

The Markdown and CommonMark specs do not mention strikethrough/strikeout at all.

So it seems our options are:

  1. Disable SimpleMDE strikethrough, or
  2. Enable Laravel-Markdown strikethrough via a CommonMark extension such as commonmark-ext

I'd be happy to work on this but would appreciate guidance from the group. Which option makes the most sense? Should Madison stick closer to the Markdown spec or support common extensions like strikethrough?

@dbiere
Copy link

dbiere commented Sep 15, 2017

Do you have any thoughts on this @sethetter?

@sethetter
Copy link
Collaborator

@dbiere apologies for the delay! I think it's sufficient to simply disable the SimpleMDE strike through. We haven't seen too many attempted uses of the feature and can always add to it later.

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

No branches or pull requests

3 participants