Closed
Bug 1160577
Opened 9 years ago
Closed 9 years ago
Error message is no longer shown when a page isn't readable
Categories
(Toolkit :: Reader Mode, defect)
Toolkit
Reader Mode
Tracking
()
People
(Reporter: MattN, Assigned: Margaret)
References
()
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
STR: 1) Load http://www.cbc.ca/mrd/episodes/season-4/ 2) Click the Reader View icon in the URL bar Actual result: Where the readable text should appear it stays blank. The title of the page becomes "Failed to load article from page" Expected result: The error message in the page should be visible. The markup is there but it's hidden by CSS. [Tracking Requested - why for this release]: Reader view regression
Flags: firefox-backlog+
Assignee | ||
Comment 1•9 years ago
|
||
This is a regression from bug 1154028, which hasn't been uplifted yet, so you should only see this in Nightly.
Assignee: nobody → margaret.leibovic
Blocks: 1154028
Assignee | ||
Comment 2•9 years ago
|
||
The problem here is that I turned this: <div id="reader-message" class="message"></div> into this: <div class="message"> <div id="reader-message"></div> </div> We have JS that makes the message visible by setting a style on #reader-message, but the default CSS makes .message hidden, so changing the visibility of that child has no effect. Patch coming.
Assignee | ||
Comment 3•9 years ago
|
||
/r/8067 - Bug 1160577 - Set styles on #reader-message div instead of wrapper div. r=MattN Pull down this commit: hg pull -r c7d75d9e4789c632e2b31b7d5f826d6e8cb8a162 https://reviewboard-hg.mozilla.org/gecko/
Attachment #8600512 -
Flags: review?(MattN+bmo)
Reporter | ||
Comment 4•9 years ago
|
||
Comment on attachment 8600512 [details] MozReview Request: bz://1160577/margaret https://reviewboard.mozilla.org/r/8065/#review6809 Ship It!
Attachment #8600512 -
Flags: review?(MattN+bmo) → review+
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/15e2e3b21f74
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Updated•9 years ago
|
Iteration: --- → 40.3 - 11 May
Flags: qe-verify?
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8600512 [details] MozReview Request: bz://1160577/margaret Approval Request Comment [Feature/regressing bug #]: bug 1154028 [User impact if declined]: error message will be invisible in reader view [Describe test coverage new/current, TreeHerder]: landed on Nightly [Risks and why]: low-risk style change restricted to reader view [String/UUID change made/needed]: none
Attachment #8600512 -
Flags: approval-mozilla-beta?
Attachment #8600512 -
Flags: approval-mozilla-aurora?
Comment 8•9 years ago
|
||
Comment on attachment 8600512 [details]
MozReview Request: bz://1160577/margaret
Approved for uplift to aurora. 38.0 has already been built so this can't make it into 38 without being part of a point release.
Attachment #8600512 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
I'm hitting a merge conflict on the uplift to Aurora. Can you look into what's missing, Margaret?
Flags: needinfo?(margaret.leibovic)
Assignee | ||
Comment 10•9 years ago
|
||
(In reply to Wes Kocher (:KWierso) from comment #9) > I'm hitting a merge conflict on the uplift to Aurora. Can you look into > what's missing, Margaret? Oh, the reason is that we didn't uplift bug 1158194 (we don't need to do that for Android, it can just ride the trains). I landed a rebased patch: https://hg.mozilla.org/releases/mozilla-aurora/rev/8051f796431e This should also apply to 38.0.5.
Flags: needinfo?(margaret.leibovic)
Updated•9 years ago
|
Flags: qe-verify? → qe-verify+
QA Contact: andrei.vaida
Comment 11•9 years ago
|
||
Confirmed that the "Failed to load article from page" error message is now shown on Nightly 40.0a1 (2015-05-07), using Windows 7 (x64) with this page [1] since the one from Comment 0 is no longer Reader View-compatible. [1] http://store.makerbot.com/smartextruder
Status: RESOLVED → VERIFIED
status-firefox39:
--- → fixed
Comment 12•9 years ago
|
||
Comment on attachment 8600512 [details]
MozReview Request: bz://1160577/margaret
Should be in 38.0.5 beta 1
Attachment #8600512 -
Flags: approval-mozilla-beta? → approval-mozilla-release+
Comment 13•9 years ago
|
||
The fix didn't reach Firefox 38.0.5 beta 1 build. The issue is fixed on latest Developer Edition 2015-05-10 under all platforms.
Comment 15•9 years ago
|
||
The fix appears to have made it in 38.0.5 Beta 1 build 2.
Comment 16•9 years ago
|
||
Verified as fixed with Firefox 38.0.5 Beta 1 build 2 under Win 7 64-bit and Mac OS X 10.9.5 using the page from comment 11.
Updated•9 years ago
|
Assignee | ||
Comment 17•9 years ago
|
||
Attachment #8600512 -
Attachment is obsolete: true
Attachment #8620218 -
Flags: review+
Assignee | ||
Comment 18•9 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•