diff options
author | http://smcv.pseudorandom.co.uk/ <http://smcv.pseudorandom.co.uk/@web> | 2010-04-03 00:12:59 +0000 |
---|---|---|
committer | Joey Hess <joey@finch.kitenet.net> | 2010-04-03 00:12:59 +0000 |
commit | aec7dec2795aedfe1f13cff9a888bed83ee760df (patch) | |
tree | e77cddc37ff0c868e335b09328437772fc38db86 | |
parent | 011fe920d162924876170d167be11dc64cf8be2f (diff) | |
download | ikiwiki-aec7dec2795aedfe1f13cff9a888bed83ee760df.tar ikiwiki-aec7dec2795aedfe1f13cff9a888bed83ee760df.tar.gz |
make questions to Joey more explicit
-rw-r--r-- | doc/todo/allow_plugins_to_add_sorting_methods.mdwn | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/doc/todo/allow_plugins_to_add_sorting_methods.mdwn b/doc/todo/allow_plugins_to_add_sorting_methods.mdwn index 156678da7..36c134a59 100644 --- a/doc/todo/allow_plugins_to_add_sorting_methods.mdwn +++ b/doc/todo/allow_plugins_to_add_sorting_methods.mdwn @@ -36,9 +36,9 @@ That earlier version of the branch is also available for comparison: >>>>> introducing "sort spec" in the API. I took a cue from >>>>> [[ikiwiki/pagespec/sorting]] being a subpage of >>>>> [[ikiwiki/pagespec]], and decided that yes, sorting is ->>>>> a bit like a pagespec :-) --s +>>>>> a bit like a pagespec :-) Which name would you prefer? --s ->>>> I would be inclined to drop the `check_` stuff. +>>>> I would be inclined to drop the `check_` stuff. --J >>>>> It basically exists to support `title_natural`, to avoid >>>>> firing up the whole import mechanism on every `cmp` @@ -50,10 +50,10 @@ That earlier version of the branch is also available for comparison: >>>>> >>>>> The former function could be achieved at a small >>>>> compatibility cost by putting `title_natural` in a new ->>>>> sortnatural plugin (that fails to load if you don't +>>>>> `sortnatural` plugin (that fails to load if you don't >>>>> have `title_natural`), if you'd prefer - that's what would >>>>> have happened if `title_natural` was written after this ->>>>> code had been merged, I suspect. --s +>>>>> code had been merged, I suspect. Would you prefer this? --s >>>> Wouldn't it make sense to have `meta(title)` instead >>>> of `meta_title`? --J @@ -97,7 +97,7 @@ That earlier version of the branch is also available for comparison: >>>> "ascending age" both seem useful to be able to explicitly specify. >>>> --[[Joey]] ->>>>> Perhaps. I do like the simplicity of [[KathyrnAndersen]]'s syntax +>>>>> Perhaps. I do like the simplicity of [[KathrynAndersen]]'s syntax >>>>> from [[plugins/contrib/report]] (which I copied verbatim, except for >>>>> turning sort-by-`field` into a parameterized spec), and I can't really >>>>> think of any sensible way to combine sort specs other than "sort by a, @@ -130,7 +130,8 @@ That earlier version of the branch is also available for comparison: >>>>> voltage, with more or less recently updated pages being said to have >>>>> more or less updateage. I don't know whether that's good or bad :-) >>>>> ->>>>> I'm sure there's a much better word, but I can't see it. --s +>>>>> I'm sure there's a much better word, but I can't see it. Do you have +>>>>> a better idea? --s ## Documentation from sort-package branch |