diff options
author | Joey Hess <joey@gnu.kitenet.net> | 2010-04-02 17:26:32 -0400 |
---|---|---|
committer | Joey Hess <joey@gnu.kitenet.net> | 2010-04-02 17:26:32 -0400 |
commit | 9e7dcefd7ed9424de20706f63c7bab5182c5df78 (patch) | |
tree | 7cbb8b1d0fcfa1d88446d95bfdcc6a9c36bade5f | |
parent | 1b0d4e8d885ded59ca0ad1b4d1ca1315585cce06 (diff) |
comments
-rw-r--r-- | doc/todo/allow_plugins_to_add_sorting_methods.mdwn | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/doc/todo/allow_plugins_to_add_sorting_methods.mdwn b/doc/todo/allow_plugins_to_add_sorting_methods.mdwn index e4e1829dc..67d85f6f8 100644 --- a/doc/todo/allow_plugins_to_add_sorting_methods.mdwn +++ b/doc/todo/allow_plugins_to_add_sorting_methods.mdwn @@ -50,6 +50,21 @@ NIH'd sorting mechanisms: >>> share the `IkiWiki::PageSpec` package, but with a `cmp_` prefix. Gitweb: >>> <http://git.pseudorandom.co.uk/smcv/ikiwiki.git?a=shortlog;h=refs/heads/sort-package> +>>>> I agree it seems more elegant, so I have focused on it. +>>>> +>>>> I don't know about reusing `IkiWiki::PageSpec` for this. +>>>> +>>>> I would be inclined to drop the `check_` stuff. +>>>> +>>>> Wouldn't it make sense to have `meta(title)` instead +>>>> of `meta_title`? +>>>> +>>>> As I read the regexp in `cmpspec_translate`, the "command" +>>>> is required to have params. They should be optional, +>>>> to match the documentation and because most sort methods +>>>> do not need parameters. +>>>> --[[Joey]] + ## Documentation from sort-hooks branch ### sort hook (added to [[plugins/write]]) |