]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/bugs/inline_sort-by-title_issues.mdwn
optimize for common case where list is not changed
[git.ikiwiki.info.git] / doc / bugs / inline_sort-by-title_issues.mdwn
index 32b505e5acaaa4fd05c07af2ee5ca718b8524dc8..ff45550673ba99cb2d41c4354b62adb614747e68 100644 (file)
@@ -15,7 +15,7 @@ which appears to list `unixauth` before `creole`.
 
 I'm not sure what the best fix is.  One fix would be to add another sort option, `sort="path"`, that would use the current (broken) sort by title.  Then add a true `sort="title"` that actually sorts on the title.  It might also be interesting to modify the sort=path to actually list the full path in the links - that way it would be obvious how it is sorted.  Or you could ignore the idea for `sort="path"`, and tell people to use [[plugins/map]] for that.
 
---Will
+--[[users/Will]]
 
 And here is a [[patch]] for this.  It makes `sort=title` actually sort on the title, and adds `sort=path` if you really want to sort on the path.  `sort=path` still only displays titles.  Just use map if you want more.
 
@@ -23,7 +23,7 @@ And here is a [[patch]] for this.  It makes `sort=title` actually sort on the ti
     index 9c336e7..99f6de3 100644
     --- a/IkiWiki/Plugin/inline.pm
     +++ b/IkiWiki/Plugin/inline.pm
-    @@ -185,9 +185,12 @@ sub preprocess_inline (@) { #{{{
+    @@ -185,9 +185,12 @@ sub preprocess_inline (@) {
                }
        }
      
@@ -51,3 +51,7 @@ And here is a [[patch]] for this.  It makes `sort=title` actually sort on the ti
      * `reverse` - If set to "yes", causes the sort order to be reversed.
      * `feedshow` - Specify the maximum number of matching pages to include in
        the rss/atom feeds. The default is the same as the `show` value above.
+
+> Thanks for the patch. [[done]], but I left off the sort=path. Also left
+> off the lc (if you ask your locale to sort case-sensatively, it should, I
+> think). --[[Joey]]