]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
review
authorhttp://smcv.pseudorandom.co.uk/ <smcv@web>
Sat, 22 Feb 2014 23:10:14 +0000 (19:10 -0400)
committeradmin <admin@branchable.com>
Sat, 22 Feb 2014 23:10:14 +0000 (19:10 -0400)
doc/todo/Option_linktext_for_pagestats_directive.mdwn

index 16fc3c5121cb6821e8568757db126d45976a1cac..8bbb7c2cf912bbb7cd26d0dc0ed9eef747fbbe72 100644 (file)
@@ -182,3 +182,15 @@ Here is a [[patch]] that adds an option `disp` for the [[ikiwiki/directive/pages
 
 Regards,  
 -- [[Louis|spalax]]
+
+> Saved to my git repository as `contrib/spalax/pagestats-disp`.
+> I'd rather find a better name than `disp` for the parameter.
+> I think `display` would be an improvement, but that doesn't solve the
+> problem of "it's a synonym for show, and non-obvious which is which".
+> Maybe `linktext`?
+> 
+> It's unfortunate that `map` and `pagestats` have different meanings for
+> the `show` parameter. I'm tempted to propose a patch that adds something
+> like `limit` (by analogy with SQL) or `max` as the canonical name for the
+> "number of things to match" parameter, at which point a non-numeric
+> `show` could mean this thing. --[[smcv]]