Closed Bug 324588 Opened 19 years ago Closed 18 years ago

In-page RSS discovery and dispatch

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME
Firefox 2

People

(Reporter: bugs, Assigned: bugs)

References

()

Details

Attachments

(1 file, 6 obsolete files)

Per evolving design doc, show in-page discovery and dispatch UI for clicked feeds. Do this the right way without any nasty hacks in the browser chrome.
here's a quick javascript hack which does something like this. We might like to re-do this in C++ and use an actual content sink for parsing the RSS, so that we can decide without having to download and parse all the data whether or not something is a feed.
First couple of thoughts: you'll need CTYPE_APPLICATIONXML too, and unless you've got a check in hand for the product placement, I think the list of things to do would look better as "Tbird, {program registered with the OS for application/(rss|atom)+xml or feed:// handler}, Live Bookmarks."
Why so suspicious? :-P I want to add a generic way for websites to register themselves as the default handler for specific content types... there are many online RSS readers, just like mail readers, pdf/doc/etc viewers and so on. As apps move into the online realm a new kind of helper app is emerging.
Attached patch more progress... (obsolete) (deleted) — Splinter Review
this is further along. it now displays a XUL page with chrome privs for application/rdf+xml content. w00t.
Attachment #209532 - Attachment is obsolete: true
Attached patch patch (obsolete) (deleted) — Splinter Review
moves document analysis into a separate object, implements a feed: protocol handler for consistent user experience clicking on http feed links vs. feed:// links.
Attachment #209758 - Attachment is obsolete: true
... which over time might become generically useful.
Attachment #209782 - Attachment is obsolete: true
Attached patch make text/plain work, make atom work (obsolete) (deleted) — Splinter Review
Attachment #209788 - Attachment is obsolete: true
This relies on some sort of content sniffing so that we can load text/html feeds (sadly, text/html is a fairly common content-type for RSS feeds).
Depends on: 324985
>+ _isFeedRSS10: function() { >+ if (this._de.namespaceURI == "http://www.w3.org/1999/02/22-rdf-syntax-ns#" || >+ this._de.namespaceURI == "http://purl.org/rss/1.0/") That's not RSS 1.0, it's all RDF/XML. Since you also want to catch RSS 0.90, the other RDF RSS, you probably want something more like if (this._de.namespaceURI == "http://www.w3.org/1999/02/22-rdf-syntax-ns#" && (this._de.getElementsByTagNameNS("http://purl.org/rss/1.0/", "channel").length || this._de.getElementsByTagNameNS("http://my.netscape.com/rdf/simple/0.9/", "channel").length))
No longer depends on: 327079
No longer depends on: 333751, 334703
QA Contact: nobody → rss.preview
*** Bug 343671 has been marked as a duplicate of this bug. ***
this is just an idea ... but why not put the RSS preview page out in a sidebar instead... that way, people won't experience that they are browsing away from a page just because they've clicked "Subscribe to this page..." in the location bar... furthermore, the "cleanup" is only to restore the sidebar state to what it was before
We're shipping it, apparently it's pretty much done (elsewhere, mostly), and changes to it should certainly be done elsewhere.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: