From 3b3a85f941f4c041303c78a982ab13c4fc60d319 Mon Sep 17 00:00:00 2001
From: "http://smcv.pseudorandom.co.uk/" <smcv@web>
Date: Thu, 6 Jan 2011 00:31:30 +0000
Subject: updated branch

---
 doc/todo/autoindex_should_use_add__95__autofile.mdwn | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

(limited to 'doc/todo')

diff --git a/doc/todo/autoindex_should_use_add__95__autofile.mdwn b/doc/todo/autoindex_should_use_add__95__autofile.mdwn
index 9f3fd71f1..23f4af379 100644
--- a/doc/todo/autoindex_should_use_add__95__autofile.mdwn
+++ b/doc/todo/autoindex_should_use_add__95__autofile.mdwn
@@ -3,7 +3,7 @@ so the latter should probably use the former. --[[smcv]]
 
 ----
 
-[[!template id=gitbranch branch=smcv/autoindex-autofile author="[[smcv]]"]]
+[[!template id=gitbranch branch=smcv/ready/autoindex-autofile author="[[smcv]]"]]
 
 I'm having trouble fixing this:
 
@@ -69,6 +69,8 @@ manually-created version, but that doesn't necessarily mean we don't want
 > when a page is deleted, nothing will ever re-create it behind ones back.
 > --[[Joey]]
 
+>> Fair enough, I'll make autoindex do that. --s
+
 ## autoindex
 
 The autoindex machinery records a more complex set. Items are added to the
@@ -110,3 +112,6 @@ and
 
 > I doubt there is any good reason for this behavior. These are probably
 > bugs. --[[Joey]] 
+
+>> OK, I believe my updated branch gives `autoindex` the same behaviour
+>> as auto-creation of tags. --s
-- 
cgit v1.2.3


From ea713f002515c3c60c28fc5eb0d70d421093af83 Mon Sep 17 00:00:00 2001
From: "http://smcv.pseudorandom.co.uk/" <smcv@web>
Date: Thu, 6 Jan 2011 01:09:13 +0000
Subject: tested and seems to work

---
 doc/todo/autoindex_should_use_add__95__autofile.mdwn | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'doc/todo')

diff --git a/doc/todo/autoindex_should_use_add__95__autofile.mdwn b/doc/todo/autoindex_should_use_add__95__autofile.mdwn
index 23f4af379..908c85cb4 100644
--- a/doc/todo/autoindex_should_use_add__95__autofile.mdwn
+++ b/doc/todo/autoindex_should_use_add__95__autofile.mdwn
@@ -114,4 +114,5 @@ and
 > bugs. --[[Joey]] 
 
 >> OK, I believe my updated branch gives `autoindex` the same behaviour
->> as auto-creation of tags. --s
+>> as auto-creation of tags. The `auto-del-create-del` and
+>> `create-del-auto` use cases work the same as for tags on my demo wiki. --s
-- 
cgit v1.2.3


From 65486cd75144b8400870604c6f809d7f6b6fd615 Mon Sep 17 00:00:00 2001
From: Joey Hess <joey@kitenet.net>
Date: Thu, 6 Jan 2011 14:49:28 -0400
Subject: feedback

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 30 ++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index 5a7b1f839..04099ec42 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -3,6 +3,36 @@
 A few patches to clean up and improve feed management for inline pages.
 
 * the first patch simply replaces the id attribute in the default template for feedlinks with a class attribute by the same name. This is necessary in pages with multiple inlines to guarantee correctness
+
+  > Ok, but blogform.tmpl has the same problem. And either change can need
+  > CSS changes. (blogform in particular is used in style.css as an id.)
+  > So this needs more documentation and associated work. --[[Joey]]
+
 * the second patch tries to define the default description for a feed based not only on the wiki name, but also on the current page name. The actual way this is built might not be the optimal one, so I'm open to suggestions
+
+  > I don't really like using "wikiname/page" as the name of the feed. It's
+  > a bit too mechanical. I'd be ok with using just the page name,
+  > with a fallback to wikiname for the toplevel index. Or maybe
+  > something like "$wikiname's $page".
+  > 
+  > Also, shouldn't `pagetitle` be run on the page name? (Haven't checked.)
+  > --[[Joey]] 
+
 * the third patch passes the feed titles to the templates, changing the default templates to use these as title attributes for the links. a rel="alternate" attribute is also included
+
+  > Seems reasonable. Cherry-picked. Note that the title attribute
+  > will be shown by browsers as a tooltip. So I made it say 
+  > "$name (RSS feed)"
+
 * the fourth patch introduces a feedlinks parameter to the inline directive, to allow for the specifications of the locations where the feed links should appear. Currently, two options are allowed (head and body), plus both and none with obvious significance
+
+  > Hmm. This doesn't affect the feed links in the blogform.tmpl. Anyway,
+  > this is not something I see a real benefit of making configurable above
+  > the template editing level. I don't see any point whatsoever of
+  > allowing to turn off the feed links in the `<head>` -- they are not
+  > user-visible, and IIRC that is the recommended and most portable way
+  > to encode the information for feed discovery agents (rather than
+  > putting it in the body). And the sorry state of "modern" browsers, 
+  > such as chromium's support for RSS means that it still makes sense to 
+  > have user-visible feed buttons. If that changed, it would make sense to
+  > modify ikiwiki to globally remove them. --[[Joey]]
-- 
cgit v1.2.3


From a4b1fbb858ec6b8571ac700339e345c9018f55fc Mon Sep 17 00:00:00 2001
From: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date: Thu, 6 Jan 2011 22:21:49 +0100
Subject: inline stuff: reply, new branch pushed to my repo

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 26 ++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index 04099ec42..9fb0867b4 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -8,6 +8,15 @@ A few patches to clean up and improve feed management for inline pages.
   > CSS changes. (blogform in particular is used in style.css as an id.)
   > So this needs more documentation and associated work. --[[Joey]]
 
+  >> I didn't include blogform in the change because the case of two
+  >> blog post forms in the same page is probably extremely rare. But
+  >> then again I remember doing having them in one of my ikiwiki
+  >> draftings, so I rewrote the patch to include blogform. I had
+  >> checked the distributed CSS for #feedlinks references, without
+  >> finding any. The new patch does include CSS changes for the
+  >> #blogform -> .blogform change. I have no idea on where to document
+  >> this change though.
+
 * the second patch tries to define the default description for a feed based not only on the wiki name, but also on the current page name. The actual way this is built might not be the optimal one, so I'm open to suggestions
 
   > I don't really like using "wikiname/page" as the name of the feed. It's
@@ -18,12 +27,19 @@ A few patches to clean up and improve feed management for inline pages.
   > Also, shouldn't `pagetitle` be run on the page name? (Haven't checked.)
   > --[[Joey]] 
 
+  >> The rewritten patch now uses the page title, obtained from meta if
+  >> possible. I introduce a new configuration key to customize
+  >> the actual automatic description allowing combinations of the wiki
+  >> name and page title.
+
 * the third patch passes the feed titles to the templates, changing the default templates to use these as title attributes for the links. a rel="alternate" attribute is also included
 
   > Seems reasonable. Cherry-picked. Note that the title attribute
   > will be shown by browsers as a tooltip. So I made it say 
   > "$name (RSS feed)"
 
+  >> Good, thanks.
+
 * the fourth patch introduces a feedlinks parameter to the inline directive, to allow for the specifications of the locations where the feed links should appear. Currently, two options are allowed (head and body), plus both and none with obvious significance
 
   > Hmm. This doesn't affect the feed links in the blogform.tmpl. Anyway,
@@ -36,3 +52,13 @@ A few patches to clean up and improve feed management for inline pages.
   > such as chromium's support for RSS means that it still makes sense to 
   > have user-visible feed buttons. If that changed, it would make sense to
   > modify ikiwiki to globally remove them. --[[Joey]]
+
+  >> I was actually quite surprised myself by the lack of automatic feed
+  >> discovery in chromium (although I noticed there's a sort-of
+  >> official plugin to do it). Overall, I believe your critique is
+  >> well-founded, I'll scratch this patch.
+
+Instead, a new patch has been added, that fixes the url in the feeds to
+point to included page rather than the including one. This is actually
+a forgotten piece from the patch that based the feed paths on the
+included rather than the including page.
-- 
cgit v1.2.3


From be5c588ab7600935efeca46645975f4031f4ba02 Mon Sep 17 00:00:00 2001
From: "http://oblomov.myopenid.com/" <oblomov@web>
Date: Thu, 6 Jan 2011 21:23:48 +0000
Subject: layout fix

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index 9fb0867b4..8935aae59 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -14,7 +14,7 @@ A few patches to clean up and improve feed management for inline pages.
   >> draftings, so I rewrote the patch to include blogform. I had
   >> checked the distributed CSS for #feedlinks references, without
   >> finding any. The new patch does include CSS changes for the
-  >> #blogform -> .blogform change. I have no idea on where to document
+  >> \#blogform -> .blogform change. I have no idea on where to document
   >> this change though.
 
 * the second patch tries to define the default description for a feed based not only on the wiki name, but also on the current page name. The actual way this is built might not be the optimal one, so I'm open to suggestions
-- 
cgit v1.2.3


From 8adc1d50fae5c4c110fdf3b76e23967c87eeeb9e Mon Sep 17 00:00:00 2001
From: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date: Fri, 7 Jan 2011 09:45:57 +0100
Subject: further patches

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 27 +++++++++++++++---------
 1 file changed, 17 insertions(+), 10 deletions(-)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index 8935aae59..290326d9a 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -27,12 +27,12 @@ A few patches to clean up and improve feed management for inline pages.
   > Also, shouldn't `pagetitle` be run on the page name? (Haven't checked.)
   > --[[Joey]] 
 
-  >> The rewritten patch now uses the page title, obtained from meta if
-  >> possible. I introduce a new configuration key to customize
-  >> the actual automatic description allowing combinations of the wiki
-  >> name and page title.
+  >> The rewritten patch now sets the feed title using the page title, and the feed description
+  >> using the page _description_, both obtained from meta if possible. If there is no page
+  >> description, then we use the page title combined with the wiki name. I introduce a new
+  >> configuration key to customize the actual automatic description.
 
-* the third patch passes the feed titles to the templates, changing the default templates to use these as title attributes for the links. a rel="alternate" attribute is also included
+* the (former) third patch passes the feed titles to the templates, changing the default templates to use these as title attributes for the links. a rel="alternate" attribute is also included
 
   > Seems reasonable. Cherry-picked. Note that the title attribute
   > will be shown by browsers as a tooltip. So I made it say 
@@ -40,7 +40,11 @@ A few patches to clean up and improve feed management for inline pages.
 
   >> Good, thanks.
 
-* the fourth patch introduces a feedlinks parameter to the inline directive, to allow for the specifications of the locations where the feed links should appear. Currently, two options are allowed (head and body), plus both and none with obvious significance
+* the (new) third patch passes uses the included rather than the including page for the URL. This is
+  actually a forgotten piece from my previous patch (now upstream) to base the feed name on the
+  included rather than the including page, and it's only relevant for nested inline pages.
+
+* the (former) fourth patch introduces a feedlinks parameter to the inline directive, to allow for the specifications of the locations where the feed links should appear. Currently, two options are allowed (head and body), plus both and none with obvious significance
 
   > Hmm. This doesn't affect the feed links in the blogform.tmpl. Anyway,
   > this is not something I see a real benefit of making configurable above
@@ -58,7 +62,10 @@ A few patches to clean up and improve feed management for inline pages.
   >> official plugin to do it). Overall, I believe your critique is
   >> well-founded, I'll scratch this patch.
 
-Instead, a new patch has been added, that fixes the url in the feeds to
-point to included page rather than the including one. This is actually
-a forgotten piece from the patch that based the feed paths on the
-included rather than the including page.
+* the (new) fourth patch introduces a `feedtitle` parameter to override the feed title. I opted for
+  not squashing it with the second patch to allow you to scrap this but sitll get the other, in case
+  you're not too happy about having a plethora of parameters
+
+* a fifth patch introduces an `id` parameter to allow setting the HTML id attribute in the
+  blogpost/feedlinks template. Since we replace their id with a class (first patch), this brings
+  back the possibility for direct CSS customization and JavaScript manipulation based on id.
-- 
cgit v1.2.3


From 4b21b29dae141330563a64dec7d686d514243c82 Mon Sep 17 00:00:00 2001
From: "http://smcv.pseudorandom.co.uk/" <smcv@web>
Date: Fri, 7 Jan 2011 20:37:54 +0000
Subject: updated (many) branches

---
 doc/todo/transient_pages.mdwn | 22 +++++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

(limited to 'doc/todo')

diff --git a/doc/todo/transient_pages.mdwn b/doc/todo/transient_pages.mdwn
index 1a35dddb1..f86857e6a 100644
--- a/doc/todo/transient_pages.mdwn
+++ b/doc/todo/transient_pages.mdwn
@@ -23,7 +23,8 @@ It could also be used for an [[todo/alias_directive]].
 
 [[!template id=gitbranch branch=smcv/ready/transient author="[[smcv]]"]]
 [[!template id=gitbranch branch=smcv/ready/transient-recentchanges author="[[smcv]]"]]
-[[!template id=gitbranch branch=smcv/ready/transient-tag author="[[smcv]]"]]
+[[!template id=gitbranch branch=smcv/ready/tag-test author="[[smcv]]"]]
+[[!template id=gitbranch branch=smcv/transient-tag author="[[smcv]]"]]
 [[!tag patch]]
 
 I think this branch is now enough to be useful. It adds the following:
@@ -197,6 +198,22 @@ Not done yet (in that branch, at least):
 >>   >> it for those simple cases then? (demand-calculate wikistatedir)
 >>   >> --[[Joey]] 
 
+>>   >>> The reason that transientdir needs to be absolute is that it's
+>>   >>> added as an underlay.
+>>   >>>
+>>   >>> We could avoid using `Cwd` by taking the extra commit from either
+>>   >>> `smcv/transient-relative` or `smcv/transient-relative-api`;
+>>   >>> your choice. I'd personally go for the latter.
+>>   >>>
+>>   >>> According to git grep, [[plugins/po]] already wants to look at
+>>   >>> the underlaydirs in its checkconfig hook, so I don't think
+>>   >>> delaying calculation of the underlaydir is viable. (I also noticed
+>>   >>> a bug,
+>>   >>> [[bugs/po:_might_not_add_translated_versions_of_all_underlays]].)
+>>   >>>
+>>   >>> `underlaydirs` certainly needs to have been calculated by the
+>>   >>> time `refresh` hooks finish, so `find_src_files` can use it. --s
+
 >> * Unsure about the use of `default_pageext` in the `change`
 >>   hook. Is everything in the transientdir really going
 >>   to use that pageext? Would it be better to look up the
@@ -217,6 +234,9 @@ Not done yet (in that branch, at least):
 >>   >> transient page has the same extension as its replacement?
 >>   >> --[[Joey]]
 
+>>   >>> Good idea, that'll be true for web edits at least.
+>>   >>> Commit added. --s
+
 --------------------------
 
 ## An earlier version
-- 
cgit v1.2.3


From 9e2cb94960fd0a314378ebad3a072186079bac12 Mon Sep 17 00:00:00 2001
From: "http://smcv.pseudorandom.co.uk/" <smcv@web>
Date: Fri, 7 Jan 2011 21:42:53 +0000
Subject: aggregation works, too

---
 doc/todo/transient_pages.mdwn | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

(limited to 'doc/todo')

diff --git a/doc/todo/transient_pages.mdwn b/doc/todo/transient_pages.mdwn
index f86857e6a..44613b7ef 100644
--- a/doc/todo/transient_pages.mdwn
+++ b/doc/todo/transient_pages.mdwn
@@ -22,11 +22,20 @@ It could also be used for an [[todo/alias_directive]].
 --------------------------
 
 [[!template id=gitbranch branch=smcv/ready/transient author="[[smcv]]"]]
-[[!template id=gitbranch branch=smcv/ready/transient-recentchanges author="[[smcv]]"]]
-[[!template id=gitbranch branch=smcv/ready/tag-test author="[[smcv]]"]]
-[[!template id=gitbranch branch=smcv/transient-tag author="[[smcv]]"]]
 [[!tag patch]]
 
+Related branches:
+
+* `ready/tag-test`: an extra regression test for tags
+* either `transient-relative` or `transient-relative-api`: avoid using `Cwd`
+  on initialization
+* `ready/transient-aggregate`: use for aggregate
+* `ready/transient-autoindex`: optionally use for autoindex
+  (includes autoindex-autofile from
+  [[todo/autoindex should use add__95__autofile]])
+* `ready/transient-recentchanges`: use for recentchanges
+* `ready/transient-tag`: optionally use for tag (includes tag-test)
+
 I think this branch is now enough to be useful. It adds the following:
 
 If the `transient` plugin is loaded, `$srcdir/.ikiwiki/transient` is added
@@ -149,6 +158,9 @@ Not done yet (in that branch, at least):
   transient underlay too (they'll naturally migrate over time). I haven't
   tested this yet, it's just a proof-of-concept.
 
+  > Now renamed to `ready/transient-aggregate`; it does seem to work fine.
+  > --s
+
 > I can confirm that the behavior of autoindex, at least, is excellent.
 > Haven't tried tag. Joey, can you merge transient and autoindex? --JoeRayhawk
 
-- 
cgit v1.2.3


From 76f925bc8e4f8f4059be13e469e6f63d7bbc3a2f Mon Sep 17 00:00:00 2001
From: "http://smcv.pseudorandom.co.uk/" <smcv@web>
Date: Fri, 7 Jan 2011 22:26:02 +0000
Subject: fixed

---
 doc/todo/Tags_list_in_page_footer_uses_basename.mdwn | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'doc/todo')

diff --git a/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn b/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn
index e2221bb84..888814977 100644
--- a/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn
+++ b/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn
@@ -6,3 +6,6 @@ I think the tag list should always contain the full path to the tag, with the ta
 
 > What if tagbase is not used? I know this would clutter up the display of
 > my tags on several wikis, including this one. --[[Joey]]
+
+>> Since Giuseppe's patches to fix [[!tag bugs/tag_behavior_changes_introduced_by_typed_link_feature]],
+>> the tag list has what Josh requested, but only if a tagbase is used. [[done]] --[[smcv]]
-- 
cgit v1.2.3


From e4fd984e3b78b8553aa685f538a84db9598310d1 Mon Sep 17 00:00:00 2001
From: "http://smcv.pseudorandom.co.uk/" <smcv@web>
Date: Fri, 7 Jan 2011 22:26:42 +0000
Subject: oops, fix syntax

---
 doc/todo/Tags_list_in_page_footer_uses_basename.mdwn | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'doc/todo')

diff --git a/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn b/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn
index 888814977..603e82b20 100644
--- a/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn
+++ b/doc/todo/Tags_list_in_page_footer_uses_basename.mdwn
@@ -7,5 +7,5 @@ I think the tag list should always contain the full path to the tag, with the ta
 > What if tagbase is not used? I know this would clutter up the display of
 > my tags on several wikis, including this one. --[[Joey]]
 
->> Since Giuseppe's patches to fix [[!tag bugs/tag_behavior_changes_introduced_by_typed_link_feature]],
+>> Since Giuseppe's patches to fix [[bugs/tag_behavior_changes_introduced_by_typed_link_feature]],
 >> the tag list has what Josh requested, but only if a tagbase is used. [[done]] --[[smcv]]
-- 
cgit v1.2.3


From 3b2d6cca0f9456814cc25f3eba214d5df2993475 Mon Sep 17 00:00:00 2001
From: 
 "https://www.google.com/accounts/o8/id?id=AItOawna1dYW4AIhTgNOMGflUf8PlpHlugZn5cA"
 <William@web>
Date: Sat, 8 Jan 2011 15:33:58 +0000
Subject: point to rebased patches containling only raw-inline code.

---
 doc/todo/inline_raw_files.mdwn | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

(limited to 'doc/todo')

diff --git a/doc/todo/inline_raw_files.mdwn b/doc/todo/inline_raw_files.mdwn
index 100c07288..58920cc12 100644
--- a/doc/todo/inline_raw_files.mdwn
+++ b/doc/todo/inline_raw_files.mdwn
@@ -1,4 +1,4 @@
-[[!template id=gitbranch branch=wtk/master author="[[wtk]]"]]
+[[!template id=gitbranch branch=wtk/raw_inline author="[[wtk]]"]]
 
 summary
 =======
@@ -59,3 +59,8 @@ usage
 >>
 >> --[[wtk]]
 
+>>> I haven't heard anything in a while, so I've reorganized my version
+>>> history and rebased it on the current ikiwiki head.  Perhaps now it
+>>> will be easier to merge or reject.  Note the new branch name:
+>>> `raw_inline`. I'll open separate todo items for items mentioned in my
+>>> previous comment. --[[wtk]]
-- 
cgit v1.2.3


From 0d7f2c643493481a1f649bb43d0dea0b7d2714f2 Mon Sep 17 00:00:00 2001
From: "W. Trevor King" <wking@drexel.edu>
Date: Sat, 8 Jan 2011 11:21:16 -0500
Subject: Add mdwn_itex todo item.

---
 doc/todo/latex.mdwn     |  2 +-
 doc/todo/mdwn_itex.mdwn | 22 ++++++++++++++++++++++
 2 files changed, 23 insertions(+), 1 deletion(-)
 create mode 100644 doc/todo/mdwn_itex.mdwn

(limited to 'doc/todo')

diff --git a/doc/todo/latex.mdwn b/doc/todo/latex.mdwn
index 76bb69c9e..5a8bdcad2 100644
--- a/doc/todo/latex.mdwn
+++ b/doc/todo/latex.mdwn
@@ -9,7 +9,7 @@ of the ikiwiki [[/logo]].
 
 > [[users/JasonBlevins]] has also a plugin for including [[LaTeX]] expressions (by means of `itex2MML`) -- [[plugins/mdwn_itex]] (look at his page for the link). --Ivan Z.
 
->> I've [updated](http://www.physics.drexel.edu/~wking/unfolding-disasters/posts/mdwn_itex/) Jason's plugin for ikiwiki 3.x.  --W. Trevor King
+>> I've [[updated|mdwn_itex]] Jason's plugin for ikiwiki 3.x.  --[[wtk]]
 
 ----
 
diff --git a/doc/todo/mdwn_itex.mdwn b/doc/todo/mdwn_itex.mdwn
new file mode 100644
index 000000000..a5add9b58
--- /dev/null
+++ b/doc/todo/mdwn_itex.mdwn
@@ -0,0 +1,22 @@
+[[!template id=gitbranch branch=wtk/mdwn_itex author="[[wtk]]"]]
+
+summary
+=======
+
+Extend the [[plugins/mdwn]] plugin to support [itex][] using Jacques
+Distler's [itex2MML][].
+
+notes
+=====
+
+This is an updated form of [[users/JasonBlevin]]'s plugin.  You can
+see the plugin [in action][example] on my blog.  The blog post lists a
+few additional changes you may need to make to use the plugin,
+including changing your page template to a MathML-friendly doctype and
+disabling plugins like [[plugins/htmlscrubber]] and
+[[plugins/htmltidy]] which would otherwise strip out the generated
+MathML.
+
+[itex]: http://golem.ph.utexas.edu/~distler/blog/itex2MMLcommands.html
+[itex2MML]: http://golem.ph.utexas.edu/~distler/blog/itex2MML.html
+[example]: http://www.physics.drexel.edu/~wking/unfolding-disasters/posts/mdwn_itex/
-- 
cgit v1.2.3


From f4dac8813db9e20b3b4f6536fb379d816e197c0e Mon Sep 17 00:00:00 2001
From: "W. Trevor King" <wking@drexel.edu>
Date: Sat, 8 Jan 2011 11:22:49 -0500
Subject: Fix link from todo/mdwn_itex to users/jasonblevins.

---
 doc/todo/mdwn_itex.mdwn | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'doc/todo')

diff --git a/doc/todo/mdwn_itex.mdwn b/doc/todo/mdwn_itex.mdwn
index a5add9b58..3e304fa76 100644
--- a/doc/todo/mdwn_itex.mdwn
+++ b/doc/todo/mdwn_itex.mdwn
@@ -9,7 +9,7 @@ Distler's [itex2MML][].
 notes
 =====
 
-This is an updated form of [[users/JasonBlevin]]'s plugin.  You can
+This is an updated form of [[users/JasonBlevins]]' plugin.  You can
 see the plugin [in action][example] on my blog.  The blog post lists a
 few additional changes you may need to make to use the plugin,
 including changing your page template to a MathML-friendly doctype and
-- 
cgit v1.2.3


From 9b475c00cb2082f3f548118c95ca9191e8eb0cb5 Mon Sep 17 00:00:00 2001
From: "W. Trevor King" <wking@drexel.edu>
Date: Sat, 8 Jan 2011 11:45:19 -0500
Subject: Add org-mode todo item with git branch.

---
 doc/todo/org_mode.mdwn | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)
 create mode 100644 doc/todo/org_mode.mdwn

(limited to 'doc/todo')

diff --git a/doc/todo/org_mode.mdwn b/doc/todo/org_mode.mdwn
new file mode 100644
index 000000000..3e9d95376
--- /dev/null
+++ b/doc/todo/org_mode.mdwn
@@ -0,0 +1,24 @@
+[[!template id=gitbranch branch=wtk/org author="[[wtk]]"]]
+
+summary
+=======
+
+Add a plugin for handling files written in [org-mode][].
+
+notes
+=====
+
+This is an updated form of [Manoj Srivastava's plugin][MS].  You can
+see the plugin [in action][example] on my blog.
+
+For reasons discussed in the [[reStructuredText plugin|plugins/rst]],
+wikilinks and other ikiwiki markup that inserts raw HTML can cause
+problems.  Org-mode provides a [means for processing raw HTML][raw],
+but Ikiwiki currently (as far as I know) lacks a method to escape
+inserted HTML depending on which plugins will be used during the
+[[htmlize phase|plugins/write#index11h3]].
+
+[org-mode]: http://orgmode.org/
+[MS]: http://www.golden-gryphon.com/blog/manoj/blog/2008/06/08/Using_org-mode_with_Ikiwiki/
+[example]: http://www.physics.drexel.edu/~wking/unfolding-disasters/posts/Git/notes/
+[raw]: http://orgmode.org/manual/Quoting-HTML-tags.html
-- 
cgit v1.2.3


From cfaab6d01e78036b57a156d91008379705684307 Mon Sep 17 00:00:00 2001
From: "W. Trevor King" <wking@drexel.edu>
Date: Sat, 8 Jan 2011 11:56:33 -0500
Subject: Add configurable markdown path todo item.

---
 doc/todo/configurable_markdown_path.mdwn | 10 ++++++++++
 1 file changed, 10 insertions(+)
 create mode 100644 doc/todo/configurable_markdown_path.mdwn

(limited to 'doc/todo')

diff --git a/doc/todo/configurable_markdown_path.mdwn b/doc/todo/configurable_markdown_path.mdwn
new file mode 100644
index 000000000..931f98025
--- /dev/null
+++ b/doc/todo/configurable_markdown_path.mdwn
@@ -0,0 +1,10 @@
+[[!template id=gitbranch branch=wtk/mdwn author="[[wtk]]"]]
+
+summary
+=======
+
+Make it easy to configure the Markdown implementation used by the
+[[plugins/mdwn]] plugin.  With this patch, you can set the path to an
+external Markdown executable in your ikiwiki config file.  If you do
+not set a path, the plugin will use the usual config options to
+determine which Perl module to use.
-- 
cgit v1.2.3


From 9b6e3331705badfe0f40265261043b9cdd51820e Mon Sep 17 00:00:00 2001
From: Joey Hess <joey@kitenet.net>
Date: Fri, 14 Jan 2011 14:37:43 -0400
Subject: picked

---
 debian/NEWS                                      | 7 +++++++
 debian/changelog                                 | 2 ++
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 2 ++
 3 files changed, 11 insertions(+)

(limited to 'doc/todo')

diff --git a/debian/NEWS b/debian/NEWS
index f73ac9c3d..c53dbc068 100644
--- a/debian/NEWS
+++ b/debian/NEWS
@@ -1,3 +1,10 @@
+ikiwiki (3.20110114) UNRELEASED; urgency=low
+
+  If you have custom CSS that uses "#feedlinks" or "#blogform", you will
+  need to change it to instead use ".feedlinks" and ".blogform"
+
+ -- Joey Hess <joeyh@debian.org>  Fri, 14 Jan 2011 14:34:54 -0400
+
 ikiwiki (3.20100515) unstable; urgency=low
 
   There are two significant changes to the page.tmpl template in this version.
diff --git a/debian/changelog b/debian/changelog
index 53241e10d..36e4a9576 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,8 @@ ikiwiki (3.20110106) UNRELEASED; urgency=low
 
   * inline: Pass feed titles to templates and add title and rel attributes
     to feed links. (Giuseppe Bilotta)
+  * inline: Use class rather than id for feedlinks and blogform.
+    (Giuseppe Bilotta)
 
  -- Joey Hess <joeyh@debian.org>  Thu, 06 Jan 2011 14:41:34 -0400
 
diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index 290326d9a..f82c30482 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -17,6 +17,8 @@ A few patches to clean up and improve feed management for inline pages.
   >> \#blogform -> .blogform change. I have no idea on where to document
   >> this change though.
 
+  >>> Picked. NEWSed. --[[Joey]]
+
 * the second patch tries to define the default description for a feed based not only on the wiki name, but also on the current page name. The actual way this is built might not be the optimal one, so I'm open to suggestions
 
   > I don't really like using "wikiname/page" as the name of the feed. It's
-- 
cgit v1.2.3


From 807ef1714af64659ce226a2ba2d49960cd9a0033 Mon Sep 17 00:00:00 2001
From: Joey Hess <joey@kitenet.net>
Date: Fri, 14 Jan 2011 14:49:16 -0400
Subject: title change redundant?

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index f82c30482..22a7d8171 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -34,6 +34,10 @@ A few patches to clean up and improve feed management for inline pages.
   >> description, then we use the page title combined with the wiki name. I introduce a new
   >> configuration key to customize the actual automatic description.
 
+  >>> The feed title part of this seems unnecessary. As far as I can see,
+  >>> ikiwiki already uses the page title as the feed title; TITLE in the
+  >>> rsspage.tmpl is handled the same as TITLE in page.tmpl. --[[Joey]]
+
 * the (former) third patch passes the feed titles to the templates, changing the default templates to use these as title attributes for the links. a rel="alternate" attribute is also included
 
   > Seems reasonable. Cherry-picked. Note that the title attribute
-- 
cgit v1.2.3


From b9c45e84aa9b162e62af324f5cdb94db984f7358 Mon Sep 17 00:00:00 2001
From: Joey Hess <joey@kitenet.net>
Date: Fri, 14 Jan 2011 14:55:32 -0400
Subject: doubtfil about patch #3

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 13 +++++++++++++
 1 file changed, 13 insertions(+)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index 22a7d8171..3ca613609 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -50,6 +50,19 @@ A few patches to clean up and improve feed management for inline pages.
   actually a forgotten piece from my previous patch (now upstream) to base the feed name on the
   included rather than the including page, and it's only relevant for nested inline pages.
 
+  > I have a vague memory of considering doing this before, and not,
+  > because there is actually no guarantee that the inlined page (that
+  > itself contains an inline) will generate an url. It could be excluded;
+  > it could be an internal page; it could use a conditional to omit the
+  > inline when not inlined.
+  > 
+  > Also, I think that `destpage` gets set wrong. And I think that
+  > `get_inline_content` is called with the source page, rather than the
+  > destpage, and so could generate urls that don't work on the destpage.
+  >
+  > All in all, this is an edge case, and currently seems to work ok, so
+  > why change it? --[[Joey]] 
+
 * the (former) fourth patch introduces a feedlinks parameter to the inline directive, to allow for the specifications of the locations where the feed links should appear. Currently, two options are allowed (head and body), plus both and none with obvious significance
 
   > Hmm. This doesn't affect the feed links in the blogform.tmpl. Anyway,
-- 
cgit v1.2.3


From ec5c5b8f7ab9e6e263dc85d933f9732dcfc8f9d3 Mon Sep 17 00:00:00 2001
From: Joey Hess <joey@kitenet.net>
Date: Fri, 14 Jan 2011 15:05:19 -0400
Subject: review remaining patches

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index 3ca613609..ee28ffa6e 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -85,6 +85,27 @@ A few patches to clean up and improve feed management for inline pages.
   not squashing it with the second patch to allow you to scrap this but sitll get the other, in case
   you're not too happy about having a plethora of parameters
 
+  > This seems clearly a good idea, since there is already a "description"
+  > parameter. But, by analogy with that parameter, it should just be
+  > called "title". --[[Joey]] 
+
 * a fifth patch introduces an `id` parameter to allow setting the HTML id attribute in the
   blogpost/feedlinks template. Since we replace their id with a class (first patch), this brings
   back the possibility for direct CSS customization and JavaScript manipulation based on id.
+
+  > That sort of makes sense, but it somehow seems wrong that "id" should
+  > apply to only cruft at the top of the inline, and not the entire div
+  > generated for it. --[[Joey]] 
+
+* 30a4de2aa3ab29dd9397c2edd91676e80bc06feb "urlto: prevent // when {url} ends with /"
+
+  > The `url` in the setup file should not end in a slash. Probably more
+  > things get ugly doubled slashes if someone does that. --[[Joey]] 
+
+* 57a9b5c4affda9e855f09a64747e5225d6254079 "inline: use urlto instead of manually building the RSS url"
+
+  > Well, that seems ok. 3 parameter urlto should give us an absolute url.
+  > 
+  > But we have to be careful and verify that it will always produce
+  > exactly the same url as before. Changing the feed url unnecessarily
+  > can probably flood aggregators or something... --[[Joey]]
-- 
cgit v1.2.3


From 5559701767f1b085cb6695accea6e6d3f4b14e84 Mon Sep 17 00:00:00 2001
From: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date: Fri, 14 Jan 2011 22:42:54 +0100
Subject: reply to inlinestuff review

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 111 ++++++++++++++---------
 1 file changed, 69 insertions(+), 42 deletions(-)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index ee28ffa6e..c3405313c 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -2,24 +2,11 @@
 
 A few patches to clean up and improve feed management for inline pages.
 
-* the first patch simply replaces the id attribute in the default template for feedlinks with a class attribute by the same name. This is necessary in pages with multiple inlines to guarantee correctness
+(I moved the picked/scratched stuff at the bottom.)
 
-  > Ok, but blogform.tmpl has the same problem. And either change can need
-  > CSS changes. (blogform in particular is used in style.css as an id.)
-  > So this needs more documentation and associated work. --[[Joey]]
-
-  >> I didn't include blogform in the change because the case of two
-  >> blog post forms in the same page is probably extremely rare. But
-  >> then again I remember doing having them in one of my ikiwiki
-  >> draftings, so I rewrote the patch to include blogform. I had
-  >> checked the distributed CSS for #feedlinks references, without
-  >> finding any. The new patch does include CSS changes for the
-  >> \#blogform -> .blogform change. I have no idea on where to document
-  >> this change though.
-
-  >>> Picked. NEWSed. --[[Joey]]
-
-* the second patch tries to define the default description for a feed based not only on the wiki name, but also on the current page name. The actual way this is built might not be the optimal one, so I'm open to suggestions
+* the second patch tries to define the default description for a feed based not only on the wiki name,
+  but also on the current page name. The actual way this is built might not be the optimal one,
+  so I'm open to suggestions
 
   > I don't really like using "wikiname/page" as the name of the feed. It's
   > a bit too mechanical. I'd be ok with using just the page name,
@@ -38,13 +25,9 @@ A few patches to clean up and improve feed management for inline pages.
   >>> ikiwiki already uses the page title as the feed title; TITLE in the
   >>> rsspage.tmpl is handled the same as TITLE in page.tmpl. --[[Joey]]
 
-* the (former) third patch passes the feed titles to the templates, changing the default templates to use these as title attributes for the links. a rel="alternate" attribute is also included
-
-  > Seems reasonable. Cherry-picked. Note that the title attribute
-  > will be shown by browsers as a tooltip. So I made it say 
-  > "$name (RSS feed)"
-
-  >> Good, thanks.
+  >>>> I'm afraid this is not the case in the ikiwiki I have. It might be the effect of some kind of interaction of
+  >>>> this with the next patch, but apparently I need both to ensure that the proper title is being used (see also
+  >>>> below for further details).
 
 * the (new) third patch passes uses the included rather than the including page for the URL. This is
   actually a forgotten piece from my previous patch (now upstream) to base the feed name on the
@@ -63,32 +46,21 @@ A few patches to clean up and improve feed management for inline pages.
   > All in all, this is an edge case, and currently seems to work ok, so
   > why change it? --[[Joey]] 
 
-* the (former) fourth patch introduces a feedlinks parameter to the inline directive, to allow for the specifications of the locations where the feed links should appear. Currently, two options are allowed (head and body), plus both and none with obvious significance
-
-  > Hmm. This doesn't affect the feed links in the blogform.tmpl. Anyway,
-  > this is not something I see a real benefit of making configurable above
-  > the template editing level. I don't see any point whatsoever of
-  > allowing to turn off the feed links in the `<head>` -- they are not
-  > user-visible, and IIRC that is the recommended and most portable way
-  > to encode the information for feed discovery agents (rather than
-  > putting it in the body). And the sorry state of "modern" browsers, 
-  > such as chromium's support for RSS means that it still makes sense to 
-  > have user-visible feed buttons. If that changed, it would make sense to
-  > modify ikiwiki to globally remove them. --[[Joey]]
-
-  >> I was actually quite surprised myself by the lack of automatic feed
-  >> discovery in chromium (although I noticed there's a sort-of
-  >> official plugin to do it). Overall, I believe your critique is
-  >> well-founded, I'll scratch this patch.
+  >> Because it does not work ok for me. I have a number of directories `dir1/`, `dir2/`, `dir3/`
+  >> each with a corresponding `dir1.mdwn`, `dir2.mdwn`, `dir3.mdwn` etc that is basically just
+  >> an inline instruction. Then my index.mdwn inlines `dir[123]`. Without these two patches, the
+  >> `dir[123]` feeds get the wrong title.
 
 * the (new) fourth patch introduces a `feedtitle` parameter to override the feed title. I opted for
-  not squashing it with the second patch to allow you to scrap this but sitll get the other, in case
+  not squashing it with the second patch to allow you to scrap this but still get the other, in case
   you're not too happy about having a plethora of parameters
 
   > This seems clearly a good idea, since there is already a "description"
   > parameter. But, by analogy with that parameter, it should just be
   > called "title". --[[Joey]] 
 
+  >> I'll rework the patch to that effect.
+
 * a fifth patch introduces an `id` parameter to allow setting the HTML id attribute in the
   blogpost/feedlinks template. Since we replace their id with a class (first patch), this brings
   back the possibility for direct CSS customization and JavaScript manipulation based on id.
@@ -97,11 +69,16 @@ A few patches to clean up and improve feed management for inline pages.
   > apply to only cruft at the top of the inline, and not the entire div
   > generated for it. --[[Joey]] 
 
+  >> Good point. I'll look into a way to move the id to the `inlinepage` div, although I guess
+  >> that falling back to `id`ing the `feedlink` div in the feedonly case would be ok.
+
 * 30a4de2aa3ab29dd9397c2edd91676e80bc06feb "urlto: prevent // when {url} ends with /"
 
   > The `url` in the setup file should not end in a slash. Probably more
   > things get ugly doubled slashes if someone does that. --[[Joey]] 
 
+  >> I was not aware of this. Did I miss it or is it just not documented?
+
 * 57a9b5c4affda9e855f09a64747e5225d6254079 "inline: use urlto instead of manually building the RSS url"
 
   > Well, that seems ok. 3 parameter urlto should give us an absolute url.
@@ -109,3 +86,53 @@ A few patches to clean up and improve feed management for inline pages.
   > But we have to be careful and verify that it will always produce
   > exactly the same url as before. Changing the feed url unnecessarily
   > can probably flood aggregators or something... --[[Joey]]
+
+  >> AFAICS, the feed url would not change (unless the previous patch is also applied and the wiki url ends with a slash)
+
+
+-----
+
+* the first patch simply replaces the id attribute in the default template for feedlinks with a class attribute by the same name. This is necessary in pages with multiple inlines to guarantee correctness
+
+  > Ok, but blogform.tmpl has the same problem. And either change can need
+  > CSS changes. (blogform in particular is used in style.css as an id.)
+  > So this needs more documentation and associated work. --[[Joey]]
+
+  >> I didn't include blogform in the change because the case of two
+  >> blog post forms in the same page is probably extremely rare. But
+  >> then again I remember doing having them in one of my ikiwiki
+  >> draftings, so I rewrote the patch to include blogform. I had
+  >> checked the distributed CSS for #feedlinks references, without
+  >> finding any. The new patch does include CSS changes for the
+  >> \#blogform -> .blogform change. I have no idea on where to document
+  >> this change though.
+
+  >>> Picked. NEWSed. --[[Joey]]
+
+
+* the (former) third patch passes the feed titles to the templates, changing the default templates to use these as title attributes for the links. a rel="alternate" attribute is also included
+
+  > Seems reasonable. Cherry-picked. Note that the title attribute
+  > will be shown by browsers as a tooltip. So I made it say 
+  > "$name (RSS feed)"
+
+  >> Good, thanks.
+
+* the (former) fourth patch introduces a feedlinks parameter to the inline directive, to allow for the specifications of the locations where the feed links should appear. Currently, two options are allowed (head and body), plus both and none with obvious significance
+
+  > Hmm. This doesn't affect the feed links in the blogform.tmpl. Anyway,
+  > this is not something I see a real benefit of making configurable above
+  > the template editing level. I don't see any point whatsoever of
+  > allowing to turn off the feed links in the `<head>` -- they are not
+  > user-visible, and IIRC that is the recommended and most portable way
+  > to encode the information for feed discovery agents (rather than
+  > putting it in the body). And the sorry state of "modern" browsers, 
+  > such as chromium's support for RSS means that it still makes sense to 
+  > have user-visible feed buttons. If that changed, it would make sense to
+  > modify ikiwiki to globally remove them. --[[Joey]]
+
+  >> I was actually quite surprised myself by the lack of automatic feed
+  >> discovery in chromium (although I noticed there's a sort-of
+  >> official plugin to do it). Overall, I believe your critique is
+  >> well-founded, I'll scratch this patch.
+
-- 
cgit v1.2.3


From 12016efb3fdcc78064a9b83c4ae6df44cbf7d9e9 Mon Sep 17 00:00:00 2001
From: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date: Sat, 15 Jan 2011 12:23:43 +0100
Subject: feed enhancements for inline pages: further comments

---
 doc/todo/feed_enhancements_for_inline_pages.mdwn | 52 ++++++++++++++++++++----
 1 file changed, 44 insertions(+), 8 deletions(-)

(limited to 'doc/todo')

diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn
index c3405313c..2a922ec35 100644
--- a/doc/todo/feed_enhancements_for_inline_pages.mdwn
+++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn
@@ -4,7 +4,7 @@ A few patches to clean up and improve feed management for inline pages.
 
 (I moved the picked/scratched stuff at the bottom.)
 
-* the second patch tries to define the default description for a feed based not only on the wiki name,
+* the (now first) patch tries to define the default description for a feed based not only on the wiki name,
   but also on the current page name. The actual way this is built might not be the optimal one,
   so I'm open to suggestions
 
@@ -26,10 +26,18 @@ A few patches to clean up and improve feed management for inline pages.
   >>> rsspage.tmpl is handled the same as TITLE in page.tmpl. --[[Joey]]
 
   >>>> I'm afraid this is not the case in the ikiwiki I have. It might be the effect of some kind of interaction of
-  >>>> this with the next patch, but apparently I need both to ensure that the proper title is being used (see also
-  >>>> below for further details).
-
-* the (new) third patch passes uses the included rather than the including page for the URL. This is
+  >>>> this with the next patch, but apparently I need both to ensure that the proper title is being used.
+
+  >>>> Some further analysis: before my patch, the feed title would be set to
+  >>>> `pagetitle($page)`, or to the wiki name if the pagetitle was index. As
+  >>>> it turns out, in my setup (see below for details) this happens quite
+  >>>> often on my `dirN.mdwn` index pages, where I would like to have `dirN`
+  >>>> as title instead. Plus, unless I'm mistaken, `pagetitle()` doesn't
+  >>>> actually use `meta` information, which my patch does. So I still think
+  >>>> the title part of the patch is worth it. As a bonus, it also allows title
+  >>>> customization by the `title=` parameter as offered in another patch.
+
+* the (now second) patch passes uses the included rather than the including page for the URL. This is
   actually a forgotten piece from my previous patch (now upstream) to base the feed name on the
   included rather than the including page, and it's only relevant for nested inline pages.
 
@@ -38,11 +46,20 @@ A few patches to clean up and improve feed management for inline pages.
   > itself contains an inline) will generate an url. It could be excluded;
   > it could be an internal page; it could use a conditional to omit the
   > inline when not inlined.
-  > 
+
+  >> I would say that in this cases my patch wouldn't change anything because
+  >> either the code would still act as before or it wouldn't be triggered at
+  >> all. --GB
+
   > Also, I think that `destpage` gets set wrong. And I think that
   > `get_inline_content` is called with the source page, rather than the
   > destpage, and so could generate urls that don't work on the destpage.
-  >
+
+  >> `destpage` getting set wrong is probably a bug that should be
+  >> fixed, but I must say I haven't come across it (yet).
+  >> `get_inline_content` is called with both the source and dest page,
+  >> and in my experience the urls have always been generated correctly.
+
   > All in all, this is an edge case, and currently seems to work ok, so
   > why change it? --[[Joey]] 
 
@@ -72,12 +89,29 @@ A few patches to clean up and improve feed management for inline pages.
   >> Good point. I'll look into a way to move the id to the `inlinepage` div, although I guess
   >> that falling back to `id`ing the `feedlink` div in the feedonly case would be ok.
 
+  >> After looking into it, I hit again the same naive error I did while
+  >> working on inline the first time: there is no "outer" div that
+  >> encloses all of the generated content: each inlined page has its
+  >> "inlinepage"-classed div, and the lot of them is prefixed by either
+  >> the feedlinks or postform template output. So the only way to "id"
+  >> a whole block of inlines is by adding a wrapping div that encloses
+  >> the whole product of the inline directive. I can do that if you
+  >> believe it's worth it.
+
 * 30a4de2aa3ab29dd9397c2edd91676e80bc06feb "urlto: prevent // when {url} ends with /"
 
   > The `url` in the setup file should not end in a slash. Probably more
   > things get ugly doubled slashes if someone does that. --[[Joey]] 
 
   >> I was not aware of this. Did I miss it or is it just not documented?
+  >> Also, grepping through the current official code (core and plugins)
+  >> there is only one other place that looks like it could be affected
+  >> by the `url` config ending in slash, and it's the `$local_url`
+  >> stuff in `IkiWiki.pm`, but that code does terminal double-slash
+  >> sanitation itself. So it would seem that my proposed patch would
+  >> lift the restriction about the terminal / (an otherwise unnecessary
+  >> restriction) without affecting much, as long as `url` users rely on
+  >> the core functions to build paths with it (as in the next patch).
 
 * 57a9b5c4affda9e855f09a64747e5225d6254079 "inline: use urlto instead of manually building the RSS url"
 
@@ -87,7 +121,9 @@ A few patches to clean up and improve feed management for inline pages.
   > exactly the same url as before. Changing the feed url unnecessarily
   > can probably flood aggregators or something... --[[Joey]]
 
-  >> AFAICS, the feed url would not change (unless the previous patch is also applied and the wiki url ends with a slash)
+  >> AFAICS, the feed url would only change in the case of /-terminating
+  >> `$config{url}`, and even then only if the preceding urlto sanitation patch
+  >> was included too.
 
 
 -----
-- 
cgit v1.2.3


From 2563611ffd7ac320c777b8d9539ad34d3be93680 Mon Sep 17 00:00:00 2001
From: "http://smcv.pseudorandom.co.uk/" <smcv@web>
Date: Thu, 20 Jan 2011 10:59:46 +0000
Subject: link Debian bug

---
 doc/todo/transient_pages.mdwn | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'doc/todo')

diff --git a/doc/todo/transient_pages.mdwn b/doc/todo/transient_pages.mdwn
index 44613b7ef..c08d54228 100644
--- a/doc/todo/transient_pages.mdwn
+++ b/doc/todo/transient_pages.mdwn
@@ -12,8 +12,8 @@ suggests:
 added.
 
 This would also be useful for autoindex, as suggested on
-[[plugins/autoindex/discussion]]. I'd also like to use it for
-[[plugins/contrib/album]].
+[[plugins/autoindex/discussion]] and [[!debbug 544322]]. I'd also like
+to use it for [[plugins/contrib/album]].
 
 It could also be used for an [[todo/alias_directive]].
 
@@ -30,8 +30,8 @@ Related branches:
 * either `transient-relative` or `transient-relative-api`: avoid using `Cwd`
   on initialization
 * `ready/transient-aggregate`: use for aggregate
-* `ready/transient-autoindex`: optionally use for autoindex
-  (includes autoindex-autofile from
+* `ready/transient-autoindex`: optionally use for autoindex,
+  which is [[!debbug 544322]] (includes autoindex-autofile from
   [[todo/autoindex should use add__95__autofile]])
 * `ready/transient-recentchanges`: use for recentchanges
 * `ready/transient-tag`: optionally use for tag (includes tag-test)
-- 
cgit v1.2.3