From: http://smcv.pseudorandom.co.uk/ <smcv@web>
Date: Sat, 4 Jun 2011 14:36:31 +0000 (+0000)
Subject: this is meant to work like Jon expected it would
X-Git-Tag: 3.20110608~26
X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/commitdiff_plain/d5a3335e15cc2203dd357e6e55fa9d15bc39a181?ds=sidebyside

this is meant to work like Jon expected it would
---

diff --git a/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn b/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
index 23ddc01b6..a656a33a0 100644
--- a/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
+++ b/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
@@ -11,6 +11,11 @@ The presentation of the resulting comments is not sorted by this date, which I w
 > Yes, comments are displayed via an inline, and usual [[pagespec/sorting]]
 > (eg, default of when the file was first seen) is used. The comment
 > date only affects the date displayed.
+>
+> > That's not what I intended - it's meant to be more or less just
+> > syntactic sugar for `\[[!meta date=foo]]`, setting the `%pagectime`.
+> > The code looks as though it ought to work, but perhaps it's buggy?
+> > --[[smcv]]
 > 
 > The only time I've seen this be much problem personally is when moving
 > a page, which means moving its comments directory, which tends to
@@ -34,3 +39,10 @@ The presentation of the resulting comments is not sorted by this date, which I w
 >>
 >> In my current situation, I could live with by-filename ordering. By-title
 >> ordering would also be workable. — [[Jon]]
+
+>>> I agree with Jon's reasons for embedding an explicit date in the file.
+>>> As I said, this is *meant* to work, but it might not.
+>>>
+>>> Sorting by filename would only be useful with
+>>> [[!cpan Sort::Naturally]], since normal `cmp` ordering would break pages
+>>> with more than 9 comments. --s