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

Making footnotes compatible with picky CMSes #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

workwithnano
Copy link

Hi fletcher,

This is edge case, yes, but here's what's prompted me to patch MMdown. I use Notational Velocity (nvAlt) to write blog posts. I use MMdown w/ footnotes. I view the source and copy/paste it into posterous's html editor, and it takes the liberty of stripping out "id" attributes from the "li" footnote elements, and on top of that, chokes on the colons in the footnote ids.

This patch replaces colons with underscores. It also adds "a name=fn_footnoteid"-style anchors — which posterous accepts — in order to give the footnotes a proper link.

I absolutely love MMdown, thanks for all the hard work.

All best,
Nano

Posterous (and possibly other CMS systems) does not like colons in the "id" or "name" attributes of html elements. Also, it strips out "id" attributes for most elements (including the "li" element of the MMdown footnote). So this commit adds "a name=fn_footnoteid" elements to the footnote "li" elements, as well as changing the footnote name/id to "fn_footnoteid" rather than "fn:footnoteid".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant