1691173 - Reader view shows wrong text

Status: UNCONFIRMED

(Toolkit :: Reader Mode, defect, P3)

Description

[Sean Garagan]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:85.0) Gecko/20100101 Firefox/85.0

Steps to reproduce:

1. Go to the following article link

   • https://www.cnbc.com/2020/09/08/billionaire-warren-buffett-most-overlooked-fact-about-how-he-got-so-rich.html

2. Select reader view from the address bar

Actual results:

Text shown is the cookie notice text, not the text of the article. The headline of the article appears correctly, but the rest is just the cookie legalese

The same thing happens if you go to the print preview screen and select the simplify page option

Expected results:

The article text should have appeared in the reader view form, simplified and easier to read/print

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Toolkit::Reader Mode' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Comment 2

[:Gijs (he/him)]

Unfortunately I cannot reproduce on either Firefox 85 or 86 (which is in beta and has some improvements to reader mode coming), in either private mode or a normal browser window.

Do you see the same thing in a private window and/or a clean Firefox profile ( https://support.mozilla.org/en-US/kb/profile-manager-create-and-remove-firefox-profiles ) ?

If so, can you load the page, open the web console (ctrl-shift-k on Windows), then run:

let serializer = new XMLSerializer();
let serializedDoc = serializer.serializeToString(document);
copy(serializedDoc);

This will copy the contents of the loaded document to the clipboard. Then if you go to https://bugzilla.mozilla.org/attachment.cgi?bugid=1691173&action=enter you should be able to paste it in the top box, give it a name ("Testcase" will do) and then submit the form to attach it to this bug. Hopefully with that I will be able to figure out why it's not working for you.

Comment 3

[Sean Garagan]

Attachment: Testcase

This is the output from the JavaScript posted in the ticket. I tried it in a private window as well as in a new profile, same thing. One thing I did notice though, if I restarted, the proper text showed up in the tab. If I then refreshed, the above text showed up

Comment 4

[:Gijs (he/him)]

(In reply to Sean Garagan from comment #3)

Created attachment 9201855 [details]
Testcase

This is the output from the JavaScript posted in the ticket. I tried it in a private window as well as in a new profile, same thing. One thing I did notice though, if I restarted, the proper text showed up in the tab. If I then refreshed, the above text showed up

I'm sorry, I was not explicit enough - can you do these steps on the page before going into reader mode? Now you've attached the reader-ized markup, but I need the input to reader mode. Sorry for the confusion!

Comment 5

[Sean Garagan]

Attachment: Testcase (before reader mode)

No problem, now you have the before and after :)

Comment 6

[:Gijs (he/him)]

Perfect, thanks. I'll try to have a look at this soon but I have a lot on my plate at the moment, so it might take a little while...