Closed Bug 362156 Opened 13 years ago Closed 13 years ago

support the 'feed' link type in HTML

Categories

(Firefox Graveyard :: RSS Discovery and Preview, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 3 alpha2

People

(Reporter: sayrer, Assigned: sayrer)

References

()

Details

Attachments

(1 file, 1 obsolete file)

A no-brainer.
Flags: blocking-firefox3?
OS: Mac OS X 10.3 → All
Hardware: PC → All
Attached patch look for the feed relation (obsolete) — Splinter Review
This adds support in <link>, but not <a>. Whoever ends up on microformat detail will have to handle that. 

I filed two spinoff bugs-

bug 362864: add support for feed discovery in anchor elements
bug 362863: decide what to do with feed mime types we don't understand
Attachment #247561 - Flags: review?(gavin.sharp)
Comment on attachment 247561 [details] [diff] [review]
look for the feed relation

>RCS file: /cvsroot/mozilla/browser/base/content/browser.js,v

>+    for each (var relValue in erel.split(" ")) { 

Shouldn't this be .split(/\s/)?

Also, please remove any trailing whitespace that this patch adds.
Attachment #247561 - Flags: review?(gavin.sharp) → review+
Attachment #247561 - Attachment is obsolete: true
Checking in browser.js;
/cvsroot/mozilla/browser/base/content/browser.js,v  <--  browser.js
new revision: 1.744; previous revision: 1.743
done
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
(In reply to comment #4)
> Created an attachment (id=247854) [edit]

btw, rels.foo is just as good as rels["foo"].
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20061207 Minefield/3.0a1 ID:2006120714 [cairo]

verified/fixed
Flags: blocking-firefox3?
Depends on: 377611
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.