Open Bug 1403149 Opened 7 years ago Updated 2 years ago

Table Header Omitted in Reader Mode

Categories

(Toolkit :: Reader Mode, defect, P3)

Product:
Toolkit
Component:
Reader Mode
Version:
55 Branch
Type:
defect

Tracking

()

People

(Reporter: funTomas, Unassigned)

Details

(Whiteboard: [reader-mode-readability-algorithm]) 

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:55.0) Gecko/20100101 Firefox/55.0
Build ID: 20170824053622

Steps to reproduce:

Visited page at https://www.patria.cz/zpravodajstvi/3635797/studie-cesko-je-nejnezdravejsi-zemi-sveta.html and switched to Reader Mode.


Actual results:

Both tables in the content miss first rows.


Expected results:

Tables featured in content should be displayed in Reader Mode completely, incl. a row with "header" style.
Status: UNCONFIRMED → NEW
Component: Untriaged → Reader Mode
Ever confirmed: true
Product: Firefox → Toolkit
The issue is that the headers have a "header" class, instead of using either <thead> or <th> as appropriate. We filter out things with the "header" class because generally speaking, we don't want items with that type of class in the output (i.e. toplevel headers at the top of the page). It's possible we can make reader mode more sophisticated here, but it may be tricky to do without regressing things, especially considering there's a type of page that (still, in 2017) uses tables for layout.
Priority: -- → P3
Whiteboard: [reader-mode-readability-algorithm]
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.