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

Document Kramdown/Maruku problems WRT options #135

Closed
gioele opened this issue Feb 16, 2012 · 3 comments
Closed

Document Kramdown/Maruku problems WRT options #135

gioele opened this issue Feb 16, 2012 · 3 comments
Labels
Milestone

Comments

@gioele
Copy link

gioele commented Feb 16, 2012

I read in the TEMPLATES.md

Every implementation of Markdown should support these options, but there are some known problems with the Kramdown and Maruku engines.

Could you please document these issues? The source code for Kramdown and Maruku is very small and clean, no problems are highlighted in the comments and the issues on GH do not show any problem with options (aside :smart_quotes, but that is handled just fine by the current code). Maybe that paragraph is a leftover from previous versions of Tilt?

@judofyr
Copy link
Collaborator

judofyr commented May 1, 2013

This might have been a leftover from a previous version. I'll have a look at it.

@judofyr
Copy link
Collaborator

judofyr commented May 2, 2013

The Markdown tests actually show this very clearly:

begin
  require 'kramdown'

  class MarkdownKramdownTest < Test::Unit::TestCase
    include MarkdownTests
    template Tilt::KramdownTemplate
    # Doesn't support escaping
    undef test_escape_html_true
    # Smarty Pants is *always* on, but doesn't support it fully
    undef test_smarty_pants
    undef test_smarty_pants_false
    undef test_smarty_pants_true
  end
rescue LoadError => boom
  # It should already be warned in the main tests
end


begin
  require 'maruku'

  class MarkdownMarukuTest < Test::Unit::TestCase
    include MarkdownTests
    template Tilt::MarukuTemplate
    # Doesn't support escaping
    undef test_escape_html_true
    # Doesn't support Smarty Pants, and even fails on ``Foobar''
    undef test_smarty_pants
    undef test_smarty_pants_false
    undef test_smarty_pants_true
    # Smart Quotes is always on
    undef test_smart_quotes
    undef test_smart_quotes_false
  end
rescue LoadError => boom
  # It should already be warned in the main tests
end

@judofyr
Copy link
Collaborator

judofyr commented May 12, 2013

I'm moving this under #189.

@judofyr judofyr closed this as completed May 12, 2013
Announcement added a commit to Announcement/tilt that referenced this issue Jun 25, 2016
@Announcement Announcement mentioned this issue Jun 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants