Closed Bug 1132674 Opened 7 years ago Closed 6 years ago
Reader mode toolbar button is slow to appear
MozReview Request: Bug 1132674 - Show reader toolbar button on DOMContentLoaded instead of pageshow. r=Gijs
39 bytes, text/x-review-board-request
This is an issue madhava complained about, but I didn't see a bug go by yet. Right now we wait until pageshow to start parsing the content: http://mxr.mozilla.org/mozilla-central/source/browser/base/content/content.js#509 We could probably try doing this on DOMContentLoaded instead. This is an interesting issue because the UX is different than Android, where the reader mode icon replaces the stop button. On Android, if we know about the reader-mode-ability of a page before it's done loading, we still wouldn't show something to the user. But on desktop, we should try to make that reader mode button appear ASAP.
With the change to avoid parse-on-load (bug 1139678), we could definitely show the button earlier. With my patch in bug 1139678, I'm only using the URL to decide whether or not to show the button, so we could even show it on location change, not pageshow.
Assignee: nobody → margaret.leibovic
/r/5637 - Bug 1132674 - Show reader toolbar button on DOMContentLoaded instead of pageshow. r=Gijs Pull down this commit: hg pull review -r da77ba76b2e0284da630bb1dfec51b24ab9b8e68
Attachment #8579460 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8579460 [details] MozReview Request: bz://1132674/margaret https://reviewboard.mozilla.org/r/5635/#review4617 Ship It!
Attachment #8579460 - Flags: review?(gijskruitbosch+bugs) → review+
Is the DOMContentLoaded event fired when a page is reloaded from the bfcache?
You need to log in before you can comment on or make changes to this bug.