FreshRSS / FreshRSS

A free, self-hostable news aggregator…
https://freshrss.org
GNU Affero General Public License v3.0
9.36k stars 806 forks source link

[Feature] Replace proprietary AMP tags with web standards for proper rendering #4972

Open bmn001 opened 1 year ago

bmn001 commented 1 year ago

Is your feature request related to a problem? Please describe.

I've found a site that truncates the content in its RSS feed. It lives here: http://feeds.feedburner.com/BritishComedyNews

No problem, I thought. I'lll just tell FreshRSS to pull from its CSS selector, which in this case is ".content-body"

That works, but then FreshRSS only displays the text of the article without any images, video, or other media.

The problem is that the site is using Google Amp to display those assets and FreshRSS is pulling Amp's weirdo proprietary tags from the page's source.

In this one example ( https://www.comedy.co.uk/people/news/7133/rip-john-bird/ ) <img> tags are actually <amp-img> tags. A video uses an <amp-youtube> tag.

FreshRSS doesn't load the scripts that tells a browser how to property load and display those elements.

Describe the solution you’d like

Maybe add a checkbox to FreshRSS's feeds' "Advanced" settings that says something like "Disable Google AMP".

Enabling the checkbox would convert all <amp-img> tags to regular <img> tags.

<amp-youtube> tags would convert to the standard youtube <iframe> embed code.

And it would just generally de-google whatever other unique tags that AMP uses.

Describe alternatives you’ve considered

I can't think of any other alternatives. If there's another way to make this work, I'm listening.

Additional context

Google AMP has been the scourge of Open Web advocates for years. Anything that we can do to work around it would be great.

math-GH commented 1 year ago

Additional context

Google AMP has been the scourge of Open Web advocates for years. Anything that we can do to work around it would be great.

just a small side note: I do not see AMP anymore as very important: https://plausible.io/blog/google-amp

Alkarex commented 1 year ago

I think this is a job for an extension and not in the core of FreshRSS