Opened 7 years ago

Closed 3 years ago

#6182 closed New feature (wontfix)

python-markdown2 support in markup

Reported by: John Downey <jdowney@…> Owned by: nobody
Component: contrib.markup Version: master
Severity: Normal Keywords: markdown markup
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description (last modified by mtredinnick)

This is a patch to allow the usage of the python-markdown2 library in place of the original markdown library. They claim better performance but I just believe it should be an option.

Attachments (1)

markdown2.patch (1.0 KB) - added by John Downey <jdowney@…> 7 years ago.
patch to allow use of python-markdown2 in place of python-markdown

Download all attachments as: .zip

Change History (9)

Changed 7 years ago by John Downey <jdowney@…>

patch to allow use of python-markdown2 in place of python-markdown

comment:1 Changed 7 years ago by Simon Greenhill <dev@…>

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Design decision needed

comment:2 Changed 7 years ago by mtredinnick

  • Description modified (diff)
  • Patch needs improvement set
  • Triage Stage changed from Design decision needed to Accepted

For backwards compatibility purposes, it should probably behave exactly as it does now by default. That is, uses the markdown module if it is installed. There's no guarantee that markdown2 and markdown will produce exactly the same output in all cases (in fact, they don't).

Given how trivial it is to write a filter like this, it's not completely clear we need to include this, but it's not too harmful, providing it's the second option chosen.

comment:3 Changed 7 years ago by mtredinnick

  • Description modified (diff)

Somehow I screwed up the description. Fixing. :-(

comment:4 Changed 4 years ago by gabrielhurley

  • Component changed from Contrib apps to contrib.markup

comment:5 Changed 4 years ago by gabrielhurley

  • Severity set to Normal
  • Type set to New feature

comment:6 Changed 3 years ago by aaugustin

  • UI/UX unset

Change UI/UX from NULL to False.

comment:7 Changed 3 years ago by aaugustin

  • Easy pickings unset

Change Easy pickings from NULL to False.

comment:8 Changed 3 years ago by claudep

  • Resolution set to wontfix
  • Status changed from new to closed

contrib.markup has been deprecated (#18504) and will be removed in Django 1.6. Closing as obsolete.

Note: See TracTickets for help on using tickets.
Back to Top