I just upgraded s9y to 1.5.1, and in doing so rediscovered some minor changes that I made around the various files over time. Most of them, it seems, should be considered for mainline inclusion.
- I've added a lastBuildDate line to templates/default/feed_2.0.tpl ("<lastBuildDate>{$metadata.lastBuildDate}</lastBuildDate>") and added lastBuildDate to the metadata in rss.php ("'lastBuildDate' => date(DATE_RSS)"). This was a change suggested by Google's automated feed validator, since s9y RSS uses the Slash module of extra feed data.
- I've replaced <content:encoded> with <description>. The complete lack of real RSS standards makes me cry, and doing this is supposed to be wrong, but more RSS readers seem to handle it than content:encoded.
- In templates/bulletproof/index.tpl, I've added '<link rel="icon" href="{$serendipityBaseURL}favicon.ico" />' after the similar line using "shortcut icon." This is because there is a standard way, and an IE way, to specify your favicon, and to work everywhere you have to do both. See http://en.wikipedia.org/wiki/Favicon
- In templates/default/admin/index.tpl, I added favicon links for consistency. This isn't as big a deal and I don't have a strong opinion on whether it merits inclusion in the main s9y branch.