]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/todo/comments.mdwn
typos
[git.ikiwiki.info.git] / doc / todo / comments.mdwn
index 6a4b403325917ea5b4aabca43a548d41306d6ea3..aa6b130dc737449782e08997b33318ea5f6c73df 100644 (file)
@@ -13,6 +13,8 @@ Known issues with the [[plugins/comments]] plugin:
 
   > Done in my comments git branch --[[smcv]]
 
 
   > Done in my comments git branch --[[smcv]]
 
+  > Not seeing it there, which branch? --[[Joey]] 
+
 * The default template should have a (?) icon next to unauthenticated users (with the IP address
   as title) and an OpenID icon next to OpenIDs
 
 * The default template should have a (?) icon next to unauthenticated users (with the IP address
   as title) and an OpenID icon next to OpenIDs
 
@@ -44,20 +46,22 @@ Known issues with the [[plugins/comments]] plugin:
   > a good feature to have, though. --[[smcv]]
 
 * One can use inline to set up a feed of all comments posted to any page.
   > a good feature to have, though. --[[smcv]]
 
 * One can use inline to set up a feed of all comments posted to any page.
-  Using template=comments_display they are displayed right. Only problem
+  Using template=comment they are displayed right. Only problem
   is there is no indication in that template of what page each comment in the
   feed is a comment on. So, if a comment is inlined into a different page,
   I think it should show a link back to the page commented on.
   (BTW, the rss feed in this situation seems ok; there the link element
   points back to the parent page.
 
   is there is no indication in that template of what page each comment in the
   feed is a comment on. So, if a comment is inlined into a different page,
   I think it should show a link back to the page commented on.
   (BTW, the rss feed in this situation seems ok; there the link element
   points back to the parent page.
 
+  > done --[[Joey]] 
+
 * It would be useful to have a pagespec that always matches all comments on 
   pages matching a glob. Something like `comment(blog/*)`.
   Perhaps postcomment could also be folded into this? Then the pagespec
   would match both existing comments, as well as new comments that are
   being posted.
 
 * It would be useful to have a pagespec that always matches all comments on 
   pages matching a glob. Something like `comment(blog/*)`.
   Perhaps postcomment could also be folded into this? Then the pagespec
   would match both existing comments, as well as new comments that are
   being posted.
 
-  > Please see [[plugins/comment/discussion]]. If I've convinced you that
+  > Please see [[plugins/comments/discussion]]. If I've convinced you that
   > internal pages are the way forward, then sure, we can do that, because
   > people who can comment still won't be able to edit others' comments
   > (one of my goals is that commenters can't put words into each other's
   > internal pages are the way forward, then sure, we can do that, because
   > people who can comment still won't be able to edit others' comments
   > (one of my goals is that commenters can't put words into each other's
@@ -69,11 +73,17 @@ Known issues with the [[plugins/comments]] plugin:
   > would also let X edit/delete comments on blog pages (including those
   > written by others) in arbitrary ways, which doesn't seem good. --[[smcv]]
 
   > would also let X edit/delete comments on blog pages (including those
   > written by others) in arbitrary ways, which doesn't seem good. --[[smcv]]
 
+  > I had a look at implementing comment() and fell afoul of 
+  > some optimisations that assume only internal() will be used to match
+  > internal pages. So probably this isn't worth doing. --[[Joey]] 
+
 * One of Joey's commit messages says "Not ideal, it would be nicer to jump to
   the actual comment posted, but no anchor is available". In fact there is
   an anchor - the `\[[_comment]]` preprocessing wraps the comment in a <div>
   with id="comment_123" or something. I'll fix this, unless Joey gets there
   first. --[[smcv]]
 
 * One of Joey's commit messages says "Not ideal, it would be nicer to jump to
   the actual comment posted, but no anchor is available". In fact there is
   an anchor - the `\[[_comment]]` preprocessing wraps the comment in a <div>
   with id="comment_123" or something. I'll fix this, unless Joey gets there
   first. --[[smcv]]
 
+  > done --[[Joey]] 
+
 * Now that inline has some comments-specific functionality anyway, it would
 * Now that inline has some comments-specific functionality anyway, it would
-  be good to output <link rel="comments"> in Atom and the equivalent in RSS.
+  be good to output '<link rel="comments">' in Atom and the equivalent in RSS.