Closed
Bug 1531712
Opened 4 years ago
Closed 4 years ago
Update readability.js etc. from git (commit 9009f64f9ce8b7d593c1ef90864843f72e193cba )
Categories
(Toolkit :: Reader Mode, enhancement, P3)
Toolkit
Reader Mode
Tracking
()
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: Gijs, Assigned: Gijs)
References
Details
There are some nice fixes in here for table parsing, GDPR headers and interactive graphs on NPR / NYT and other news sites.
Assignee | ||
Updated•4 years ago
|
Priority: -- → P3
Assignee | ||
Updated•4 years ago
|
Summary: Update readability.js etc. from git (commit 27ee1e947ec0594c2b86d1b1ce89274612a390df ) → Update readability.js etc. from git (commit 9009f64f9ce8b7d593c1ef90864843f72e193cba )
Assignee | ||
Comment 1•4 years ago
|
||
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
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/817f8ac60713 update readability to 9009f64f9ce8b7d593c1ef90864843f72e193cba from github, rs=already-reviewed
Assignee | ||
Comment 3•4 years ago
|
||
Comment 4•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•