support the 'feed' link type in HTML

RESOLVED FIXED in Firefox 3 alpha2

Status

()

Firefox
RSS Discovery and Preview
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: Robert Sayre, Assigned: Robert Sayre)

Tracking

Trunk
Firefox 3 alpha2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

11 years ago
A no-brainer.
(Assignee)

Updated

11 years ago
Flags: blocking-firefox3?
OS: Mac OS X 10.3 → All
Hardware: PC → All
(Assignee)

Comment 1

11 years ago
Use the test case from bug 362239

https://bugzilla.mozilla.org/attachment.cgi?id=247042
(Assignee)

Comment 2

11 years ago
Created attachment 247561 [details] [diff] [review]
look for the feed relation

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+
(Assignee)

Comment 4

11 years ago
Created attachment 247854 [details] [diff] [review]
fix gavin's comments
Attachment #247561 - Attachment is obsolete: true
(Assignee)

Comment 5

11 years ago
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
Last Resolved: 11 years ago
Resolution: --- → FIXED

Comment 6

11 years ago
(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

Updated

10 years ago
Flags: blocking-firefox3?

Updated

10 years ago
Depends on: 377611
You need to log in before you can comment on or make changes to this bug.