1477669 - Remove feed preview code

Status: RESOLVED FIXED

(Firefox Graveyard :: RSS Discovery and Preview, enhancement, P1)

Description

[:Gijs (he/him)]

As part of removing feed and live bookmark support, we should remove about:feeds and the "Subscribe..." UI that opens it.

Comment 2

[:Gijs (he/him)]

Attachment: Bug 1477669 - remove feed preview code and associated files/code, r?mak

Comment 3

[nanaya]

Can't this be changed into something like json viewer instead? I usually use feed preview to validate my rss/atom feeds.

Comment 4

[Marco Bonardo [:mak]]

Please see https://validator.w3.org/feed/

Comment 5

[nanaya]

But you can't view what's inside with that. Validate here I mean includes viewing how the content looks like.

Comment 6

[nanaya]

I don't know about others but firefox preview is essential to me for writing rss/atom feeds. I don't know even know other apps for previewing apart of internet explorer. Additionally, having it built into firefox means I get benefit of history and cookies.

Also unlike json, feeds are sometimes html which means it can't be viewed properly through console and anything that can view this means it's got html renderer in it... which browser conveniently already has.

I sometimes use it to preview feeds before actually subscribing to but the development side usage is more important to me.

If you've got other recommendation, I'm all ears though this means one extra hurdle for developers looking to write rss/atom feeds.

Comment 7

[Marco Bonardo [:mak]]

How the content looks like is highly dependent on the feed reader, so I'm not sure there's a globally good tool for that. Any feed reader may be good as far as I can tell, so I'd just look around for some stats about the most widely used ones, and check your feeds with those. In the end it's the same as "is my website displayed correctly?" and unfortunately there's not yet much way out than testing it with the most common browsers.

Comment 8

[nanaya]

Pretty sure most web based feed readers can't read my internal development feeds though :)

Some don't even have preview feature (like tt-rss) or maybe even permanently cache feeds so I can't just update the code and refresh with those.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c8afe4845022
remove feed preview code and associated files/code, r=flod,mak,nika

Comment 10

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/c8afe4845022