343749 - Feed Preview should show HTML content

Status: VERIFIED DUPLICATE of bug 340554

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

Description

[Dão Gottwald [:dao]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060706 Minefield/3.0a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060706 Minefield/3.0a1

If you take a look at the example feed, you'll notice that most of the summaries don't show up, namely those with type="html".

Reproducible: Always

Steps to Reproduce:
1. create / open an Atom feed with an entry that has a HTML summary
Actual Results:  
The summary is ignored.

Expected Results:  
HTML summaries should be recognized and embedded as HTML. The content would have to be nested within <div class="feedEntryContent">, because <p class="feedEntryContent"> would break if it contained <p>.

Comment 1

[Dão Gottwald [:dao]]

Attachment: sample Atom feed

Comment 2

[Robert Sayre]

The summaries that are elided contain only img tags and no text. Since a3 and b1 strip tags, they don't show up.

Comment 3

[Robert Sayre]

Note: We will turn on HTML content in a future beta

Comment 4

[Dão Gottwald [:dao]]

I didn't meant to target Firefox 2. Furthermore it was my intention to say that tags shouldn't be stripped, so simply stating that they are stripped ATM can't be the reason for not fixing this.

Also note that the sample feed contains text within the alt attribute. If removing HTML is the way to go, it could be more intelligent.

Comment 5

[Dão Gottwald [:dao]]

Just discovered bug 302749. Maybe it should be reopened. Then this would be a duplicate of it.

Comment 6

[Dão Gottwald [:dao]]

(In reply to comment #3)
> Note: We will turn on HTML content in a future beta

Crap. I misread that as "in a future version". Never mind then.

Comment 7

[Marco Bonardo [:mak]]

you have my vote on this, this strip_tags behaviour kills bittorrent rss feeds usability, for example this feed:
http://www.tntvillage.org/tntforum/ssi.php?a=releases&cat=13&show=10&type=rss
does not link down .torrent files while previous version did.
wait for reintroduction of html tags

Comment 8

[Robert Sayre]

I'm going to mark this a dupe of bug 340554. There's a patch there that turns on HTML.

*** This bug has been marked as a duplicate of 340554 ***

Comment 9

[Tanner M. Young [:tmyoung]]

Verified Duplicate.