aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon McVittie <smcv@debian.org>2017-05-14 18:16:53 +0100
committerSimon McVittie <smcv@debian.org>2017-05-14 18:16:53 +0100
commit4db4e589e4c73f076b666a77b86743695454a3ce (patch)
tree14436dc88eb75dc528f3ee27f6f5abb9a5634478
parent81c3258269fa53684043ce5b9fd701bb6a381017 (diff)
downloadikiwiki-4db4e589e4c73f076b666a77b86743695454a3ce.tar
ikiwiki-4db4e589e4c73f076b666a77b86743695454a3ce.tar.gz
mdwn: Enable footnotes by default when using Discount
A new mdwn_footnotes option can be used to disable footnotes in MultiMarkdown and Discount.
-rw-r--r--IkiWiki/Plugin/mdwn.pm24
-rw-r--r--debian/changelog3
-rw-r--r--doc/bugs/footnotes-look-weird.mdwn4
3 files changed, 30 insertions, 1 deletions
diff --git a/IkiWiki/Plugin/mdwn.pm b/IkiWiki/Plugin/mdwn.pm
index aeb3a3fd7..e142fec46 100644
--- a/IkiWiki/Plugin/mdwn.pm
+++ b/IkiWiki/Plugin/mdwn.pm
@@ -7,6 +7,7 @@ use strict;
use IkiWiki 3.00;
sub import {
+ hook(type => "checkconfig", id => "mdwn", call => \&checkconfig);
hook(type => "getsetup", id => "mdwn", call => \&getsetup);
hook(type => "htmlize", id => "mdwn", call => \&htmlize, longname => "Markdown");
hook(type => "htmlize", id => "md", call => \&htmlize, longname => "Markdown (popular file extension)", nocreate => 1);
@@ -33,6 +34,17 @@ sub getsetup () {
safe => 1,
rebuild => 1,
},
+ mdwn_footnotes => {
+ type => "boolean",
+ example => 1,
+ description => "enable footnotes in Markdown (where supported)?",
+ safe => 1,
+ rebuild => 1,
+ },
+}
+
+sub checkconfig () {
+ $config{mdwn_footnotes} = 1 unless defined $config{mdwn_footnotes};
}
my $markdown_sub;
@@ -54,7 +66,13 @@ sub htmlize (@) {
}
else {
$markdown_sub=sub {
- Text::MultiMarkdown::markdown(shift, {use_metadata => 0});
+ my %flags=( use_metadata => 0 );
+
+ if ($config{mdwn_footnotes}) {
+ $flags{disable_footnotes}=1;
+ }
+
+ Text::MultiMarkdown::markdown(shift, \%flags);
}
}
}
@@ -79,6 +97,10 @@ sub htmlize (@) {
# Use the typography plugin instead
$flags |= Text::Markdown::Discount::MKD_NOPANTS();
+ if ($config{mdwn_footnotes}) {
+ $flags |= Text::Markdown::Discount::MKD_EXTRA_FOOTNOTE();
+ }
+
# Workaround for discount's eliding
# of <style> blocks.
# https://rt.cpan.org/Ticket/Display.html?id=74016
diff --git a/debian/changelog b/debian/changelog
index 08e320f30..e2a861c3c 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -8,6 +8,9 @@ ikiwiki (3.20170112) UNRELEASED; urgency=medium
a set-but-empty name as if they were not signed in.
* When the CGI fails, print the error to stderr, not "Died"
* mdwn: Don't mangle <style> into <elyts> under some circumstances
+ * mdwn: Enable footnotes by default when using the default Discount
+ implementation. A new mdwn_footnotes option can be used to disable
+ footnotes in MultiMarkdown and Discount.
-- Simon McVittie <smcv@debian.org> Sun, 14 May 2017 15:34:52 +0100
diff --git a/doc/bugs/footnotes-look-weird.mdwn b/doc/bugs/footnotes-look-weird.mdwn
index be1e5d562..a821ab43c 100644
--- a/doc/bugs/footnotes-look-weird.mdwn
+++ b/doc/bugs/footnotes-look-weird.mdwn
@@ -92,6 +92,10 @@ screen readers), as detailed in [this Stack Overflow discussion][].
>>> --[[smcv]]
>>>
>>>> Makes perfect sense to me. --[[anarcat]]
+>>>>
+>>>>> I have now enabled footnotes in Discount by default, with a new
+>>>>> `mdwn_footnotes` option that can switch them off if they become
+>>>>> problematic. --[[smcv]]
>>>
>> For example, to enable footnotes, one needs to call Discount like this:
>>