]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/todo/Option_linktext_for_pagestats_directive.mdwn
Feeling out how to present patch for review
[git.ikiwiki.info.git] / doc / todo / Option_linktext_for_pagestats_directive.mdwn
index 16fc3c5121cb6821e8568757db126d45976a1cac..ab5eb229b0eae766826b9ac50c2c1a5034c5a0c9 100644 (file)
@@ -182,3 +182,22 @@ 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]]
+
+>> [[!template id=gitbranch branch=smcv/pagestats-show
+author="[[Louis|spalax]], [[smcv]]"
+browse=http://git.pseudorandom.co.uk/smcv/ikiwiki.git/shortlog/refs/heads/pagestats-show]]
+>> Here's a branch. It depends on my `ready/limit` branch
+>> from [[todo/pick a new canonical name for equivalent of SQL limit]].
+>> --[[smcv]]