WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 6 years ago

#7341 closed defect (bug) (wontfix)

Default theme doesn't advertise Atom 1.0 feed

Reported by: markjaquith Owned by: Nicholas91
Milestone: Priority: high
Severity: normal Version: 2.6
Component: Template Keywords: dev-feedback, atom, autodiscover, feed, has-patch
Focuses: Cc:

Description

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

Attachments (1)

wp-content--themes--default--header.php.diff (726 bytes) - added by josephscott 7 years ago.

Download all attachments as: .zip

Change History (17)

comment:1 @josephscott7 years ago

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

comment:2 @thee177 years ago

  • Keywords has-patch added

comment:3 @westi7 years ago

  • Resolution set to fixed
  • Status changed from new to closed

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

comment:4 @westi7 years ago

  • Milestone changed from 2.7 to 2.6.1
  • Resolution fixed deleted
  • Status changed from closed to reopened

Re-open for 2.6.1

comment:5 @westi7 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

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

comment:6 @lloydbudd7 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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.

comment:7 @mdawaffe7 years ago

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.

comment:8 @mdawaffe7 years ago

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.

comment:9 @lloydbudd7 years ago

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.

comment:10 @scribu7 years ago

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

comment:11 @ryan7 years ago

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

comment:12 @westi7 years ago

  • Milestone changed from 2.6.1 to 2.7

Moving to 2.7 for now.

comment:13 @ryan7 years ago

  • Milestone changed from 2.7 to 2.8

comment:14 @Nicholas916 years ago

  • Cc dragos.nicholas@… added
  • Keywords dev-feedback added; head link removed
  • Owner changed from anonymous to Nicholas91
  • Status changed from reopened to new

It seems that the feed links are generated by wp_head(); with the feed_links() function in general-template.php. I suggest leaving things as they are. I mean, how many people use Atom instead of RSS?
I'm adding the dev-feedback keyword, so a dev may close this ticket if he decides that the Atom feed link is useless.

comment:15 @thee176 years ago

I also think that there should only be one, And it is not unreasonable to have the code snip-it in the Codex to allow a user to add it if need be to the theme. I think this should be closed.

comment:16 @ryan6 years ago

  • Milestone 2.8 deleted
  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.