From db38cad40f9614ce1c37c818415d430db07813b8 Mon Sep 17 00:00:00 2001 From: Joey Hess Date: Tue, 22 Feb 2011 17:32:01 -0400 Subject: [PATCH] reviewing next patch --- doc/todo/feed_enhancements_for_inline_pages.mdwn | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/doc/todo/feed_enhancements_for_inline_pages.mdwn b/doc/todo/feed_enhancements_for_inline_pages.mdwn index 02b4e9d36..4e80fdeb8 100644 --- a/doc/todo/feed_enhancements_for_inline_pages.mdwn +++ b/doc/todo/feed_enhancements_for_inline_pages.mdwn @@ -36,6 +36,13 @@ description is provided by either the directive parameter or the page metadata, we use a user-configurable default based on both the page title and wiki name rather than hard-coding the wiki name as description. +> Reviewing, this seems ok, but I don't like that +> `feed_desc_fmt` is "safe => 0". And I question if that needs +> to be configurable at all. I say, drop that configurable, and +> only use the page meta description (or wikiname for index). +> +> Oh, and could you indent your `elsif` the same as I? --[[Joey]] + The third patch, ‘inline: allow assigning an id to postform/feedlink’, does just that. I don't currently use it, but it can be particularly useful in the postform case for example for scriptable management of -- 2.32.0.93.g670b81a890