1531712 - Update readability.js etc. from git (commit 9009f64f9ce8b7d593c1ef90864843f72e193cba )

Status: RESOLVED FIXED

(Toolkit :: Reader Mode, enhancement, P3)

Description

[:Gijs (he/him)]

There are some nice fixes in here for table parsing, GDPR headers and interactive graphs on NPR / NYT and other news sites.

Comment 1

[:Gijs (he/him)]

Contains fixes:

• Avoid gdpr modals interfering with reader mode
• Improve byline data on the New York Times
• Fix tables being removed and fix table headers missing from readability output
• Exclude CSS-styled DOM-based graphs using ai2html from output
• Fix embedded videos getting removed
  Note: this will not in itself fix videos in Firefox due to security restrictions in reader mode. We need to fix bug 1204818 in order for that to be feasible.
• Avoid removing "sharing" content that has significant amounts of text

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/817f8ac60713
update readability to 9009f64f9ce8b7d593c1ef90864843f72e193cba from github, rs=already-reviewed

Comment 3

[:Gijs (he/him)]

(try was: https://treeherder.mozilla.org/#/jobs?repo=try&revision=13f9b18c19fc25d47650ae3bed68d2b9906afe92 )

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/817f8ac60713