www-infrastructure-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joe Schaefer (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (INFRA-9705) Remove elementid Markdown extension
Date Thu, 02 Jul 2015 01:46:04 GMT

    [ https://issues.apache.org/jira/browse/INFRA-9705?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14611312#comment-14611312
] 

Joe Schaefer commented on INFRA-9705:
-------------------------------------

Wait a second. So @sebb hacked the mdx_elementid.py script to enable the nifty subversion
magic paragraph indicators.  And I can see that at one point the tests actually worked with
that hack enabled.  Otherwise the rest of mdx_elementid.py remains perfectly well-functioning,
and now those hacks @sebb did no longer apply because THAT STUFF IS COMMENTED OUT.  So why
is this sufficient justification to remove this module from the markdown processing entirely?


> Remove elementid Markdown extension
> -----------------------------------
>
>                 Key: INFRA-9705
>                 URL: https://issues.apache.org/jira/browse/INFRA-9705
>             Project: Infrastructure
>          Issue Type: Bug
>          Components: CMS, Website
>         Environment: https://svn.apache.org/repos/infra/websites/cms/build/mdx_elementid.py
> https://svn.apache.org/repos/infra/websites/cms/build/markdownd.py
>            Reporter: Sebb
>         Attachments: INFRA-9705.patch
>
>
> The elementid Markdown extension appears to have been originally intended to handle Special
Attributes as per PHP Markdown Extras [1]. However it was not a complete implementation.
> The elementid extension code was later extended to try to handle permalinks as per Subversion
[2]. However it turned out that this code interfered with TOC processing.
> It would of course be possible to fix both of these issues, however there seems little
point as the Python Markdown module includes atttribute lists [3] as well as permalinks [4].
> I think it therefore makes sense to disable elementid by removing it from markdownd.py:EXTENSIONS.
> Attribute lists and permalinks can then be enabled separately; see related JIRAs.
> [1] https://michelf.ca/projects/php-markdown/extra/#spe-attr
> [2] http://subversion.apache.org/style/site.css
> [3] https://pythonhosted.org/Markdown/extensions/attr_list.html
> [4] https://pythonhosted.org/Markdown/extensions/toc.html#usage



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message