Ticket #7341 (reopened defect)

Opened 5 months ago

Last modified 2 months ago

Default theme doesn't advertise Atom 1.0 feed

Reported by: markjaquith Assigned to: anonymous
Priority: high Milestone: 2.8
Component: Template Version: 2.6
Severity: normal Keywords: atom, head, link, autodiscover, feed, has-patch
Cc:

Description

Atom 1.0 is advertised in the Classic theme, but not the Default theme.

Attachments

wp-content--themes--default--header.php.diff (0.7 kB) - added by josephscott on 07/18/08 01:17:56.

Change History

07/18/08 01:17:56 changed by josephscott

  • attachment wp-content--themes--default--header.php.diff added.

07/18/08 01:20:31 changed by josephscott

Added simple diff to add Atom feed to the default theme.

07/21/08 02:49:57 changed by thee17

  • keywords changed from atom, head, link, autodiscover, feed to atom, head, link, autodiscover, feed, has-patch.

07/23/08 06:56:26 changed by westi

  • status changed from new to closed.
  • resolution set to fixed.

(In [8408]) Advertise Atom 1.0 Feed in default theme. Fixes #7341 for trunk props josephscott.

07/23/08 06:56:42 changed by westi

  • status changed from closed to reopened.
  • resolution deleted.
  • milestone changed from 2.7 to 2.6.1.

Re-open for 2.6.1

07/23/08 06:59:48 changed by westi

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [8409]) Advertise Atom 1.0 Feed in default theme. Fixes #7341 for 2.6.x props josephscott.

08/08/08 22:34:26 changed by lloydbudd

  • status changed from closed to reopened.
  • resolution deleted.

Doesn't seem like a compelling reason this went in the maint. stream and seems like the wrong solution. I don't intensely care whether it's RSS or Atom, but only one feed should be listed for auto discovery, otherwise it's just sloppy. The feed included in head should be the /feed/ (once pretty permalinks are enabled).

The better solution seems to be to remove the additional feeds from classic, but when I previously considered this I left it because it's classic (ie ancient).

And if we want to switch the feed to Atom seems like something to debate for 2.7 or a future release.

08/12/08 00:33:40 changed by mdawaffe

Why is listing multiple feeds sloppy? It's just listing the various alternate ways we offer for you to consume the data. To me it seems sloppy to leave any out.

08/12/08 00:55:37 changed by mdawaffe

lloydbudd points out that if multiple feeds are listed and you click the "subscribe to feed" icon in Firefox, say, you have to pick which one you want, which is confusing.

We could title one of them "{Blog title} feed" and the others "Atom", "RSS", etc. to try and make a choice for people.

08/12/08 01:46:05 changed by lloydbudd

The 1st gate should be does this need to go in a maintenance release. So regardless of anything else, I don't think it meets this condition.

Having worked on Flock, I remember them having to create a smart solution for blogs with multiple alternatives of the same content. They had to visit each feed and compare the content of the feeds. Not pretty. It doesn't look like things have chanced so that one can say that two feeds are equiv without visiting.

Better to encourage simple solutions that list true content alternatives like comment feeds for those interested. default is the theme I see themes most often based on it should be as... elegant as possible.

08/14/08 23:02:28 changed by scribu

Shouldn't additional feed links be plugin teritory? (different feed types included)

08/14/08 23:11:30 changed by ryan

(In [8649]) Revert [8409] pending discussion. see #7341

08/16/08 20:50:45 changed by westi

  • milestone changed from 2.6.1 to 2.7.

Moving to 2.7 for now.

10/13/08 22:49:43 changed by ryan

  • milestone changed from 2.7 to 2.8.