Closed Bug 1591316 Opened 3 years ago Closed 10 months ago

Wide tables can cause horizontal scroll in reader mode

Categories

(Toolkit :: Reader Mode, defect, P3)

70 Branch
defect

Tracking

()

VERIFIED FIXED
98 Branch
Tracking Status
firefox98 --- fixed

People

(Reporter: sheelgautam995, Assigned: hujintao)

References

(Blocks 1 open bug, )

Details

(Keywords: testcase, Whiteboard: [about-reader-ui])

Attachments

(1 file)

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:70.0) Gecko/20100101 Firefox/70.0

Steps to reproduce:

Go to this website
https://en.wikipedia.org/wiki/OECD
and open reader mode

Actual results:

Reader mode has a horizontal scrollbar. During scrolling vertically, I accidentally scroll horizontally which messes up the alignment of the text. Sometimes the text gets hidden behind the sidebar of the reader mode.

Expected results:

No horizontal scroll like in Safari.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: Untriaged → Reader Mode
Product: Firefox → Toolkit

This needs some work to make the tables center on the column if they're too wide. Unsure if there's a straightforward way to do that with just CSS...

Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: testcase
Priority: -- → P3
Summary: Reader mode should not scroll horizontally → Wide tables can cause horizontal scroll in reader mode
Whiteboard: [about-reader-ui]
Assignee: nobody → hujintao
Status: NEW → ASSIGNED
Pushed by mtigley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1449eb71de74
add scrollbar to wide tables to prevent horizontal scroll in reader mode. r=mtigley,niklas
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
Flags: qe-verify+

Reproducible on Firefox 70 on macOS. There is a horizontal scroll bar implemented but it doesn't show until the browser is minimized; text gets overlapped as mentioned in STR. Issue fixed on latest beta(98.0b2) and latest Nightly 99.0a1.

Flags: qe-verify+
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.