alternate idea

[ikiwiki.git] / doc / todo / comments.mdwn

diff --git a/doc/todo/comments.mdwn b/doc/todo/comments.mdwn
index ee2ba7ea086631cb3034576f18cae13c976533b4..12aef0bb3968f7f9eaf0a707f4ebbfdb226ae828 100644 (file)
--- a/doc/todo/comments.mdwn
+++ b/doc/todo/comments.mdwn
@@ -13,7 +13,10 @@ Known issues with the [[plugins/comments]] plugin:
 
   > Done in my comments git branch --[[smcv]]
 
-  > Not seeing it there, which branch? --[[Joey]] 
+  > Not seeing it there, which branch? --[[Joey]]
+
+  >> Bah, git push --all is not the default... 'comments' branch now (I've also rebased it).
+  >> Sorry, I'm on mobile Internet at the moment... --[[smcv]]
 
 * The default template should have a (?) icon next to unauthenticated users (with the IP address
   as title) and an OpenID icon next to OpenIDs
@@ -22,6 +25,19 @@ Known issues with the [[plugins/comments]] plugin:
   > {x} and {*} smileys for anonymous, OpenID and login respectively).
   > --[[smcv]]
 
+  >> I've improved this to use independent icons from the wikiicons
+  >> directory (untested!) --[[smcv]]
+
+  >>> The new code produces links like /wikiisons/openid.png, which
+  >>> fail if ikiwiki is not at the root of the web server. --[[Joey]] 
+
+  >>> I got to wondering if the icons are needed. On my comments branch
+  >>> (not master), I've dropped the icons and info can be seen by hovering
+  >>> over the author's name. Idea being that you probably don't care how
+  >>> they authenticated unless something is weird, and in that case you
+  >>> can hover to check. Does that make sense, should I merge it?
+  >>> --[[Joey]] 
+
 * Should the comments be visually set off more from the page above?
   Rather than just a horizontal rule, I'm thinking put the comments
   in a box like is used for inlined pages.
@@ -53,13 +69,15 @@ Known issues with the [[plugins/comments]] plugin:
   (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.
 
-  > 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
@@ -84,4 +102,4 @@ Known issues with the [[plugins/comments]] plugin:
   > done --[[Joey]] 
 
 * 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.