diff options
| author | Joey Hess <joey@kitenet.net> | 2011-02-22 17:32:01 -0400 |
|---|---|---|
| committer | Joey Hess <joey@kitenet.net> | 2011-02-22 17:32:01 -0400 |
| commit | db38cad40f9614ce1c37c818415d430db07813b8 (patch) | |
| tree | 3209c2d92cbf91c41a31316c94da98e160086da1 /doc/todo | |
| parent | 9836bceaa520919530e4311582b4fee2fd5c4e36 (diff) | |
reviewing next patch
Diffstat (limited to 'doc/todo')
| -rw-r--r-- | doc/todo/feed_enhancements_for_inline_pages.mdwn | 7 |
1 files changed, 7 insertions, 0 deletions
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 |