Open Bug 1179222 Opened 9 years ago Updated 2 years ago

Shouldn't offer reader mode for crash-reports.mozilla.com

Categories

(Toolkit :: Reader Mode, defect, P5)

defect

Tracking

()

Tracking Status
firefox42 --- affected

People

(Reporter: vtamas, Unassigned)

References

(Blocks 1 open bug)

Details

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

Reproducible on: Firefox 38.0.5, Firefox 39 RC,Firefox 40.0a2 and Firefox 42.0a1 on Windows 8.1 32-bit STR 1.Launch Firefox. 2.Navigate to https://crash-stats.mozilla.com/report/index/3078c479-b673-47d9-96a2-f891c2150701 (Crash Reports page) 3.Click on “Reader view” button from location bar. ER The page is correctly displayed in Reader View Mode. AR A piece of code is displayed instead of the text. Additional notes: - This issue is reproducible on Firefox 38.0.5 (20150525141253), Firefox 39 RC build 6 (20150630154324), 40.0a2 (2015-06-29) and Firefox 42.0a1 (2015-06-30) under Windows 8.1 32-bit. - I am attaching a screenshot: http://i.imgur.com/5fANI5t.jpg
I don't think this is a bug. This is in fact the content of the page in the "raw dump" tab. Since this is a multi-tab app style page, I don't think offering reader view for it makes a lot of sense. Perhaps the bug here is that we're offering reader view at all? Who is responsible for making that decision?
Status: NEW → UNCONFIRMED
Ever confirmed: false
Flags: needinfo?(margaret.leibovic)
We use a very basic heuristic to determine whether or not to offer reader view: https://github.com/mozilla/readability/blob/master/Readability.js#L1704 Because this function runs on every page load, we are very concerned about its performance impact, so it is challenging to try to make this perfect. We could try to tweak the algorithm to account for this test case, but we would just also have to be careful to not regress other test cases. Filing an issue on the readability repo would probably be the best way forward here, although I don't think there's anyone actively burning down that issue list right now.
Flags: needinfo?(margaret.leibovic)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P5
Summary: Mozilla Crash Reports Page incorrect displayed in Reader View Mode → Shouldn't offer reader mode for crash-reports.mozilla.com
Whiteboard: [reader-mode-readability-algorithm][reader-mode-isreadable-detection]
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.