Closed
Bug 1150229
Opened 9 years ago
Closed 9 years ago
Page fails to load in Reader View on entering Reader View, works on 2nd attempt
Categories
(Toolkit :: Reader Mode, defect)
Tracking
()
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
firefox38 | --- | verified |
People
(Reporter: designakt, Unassigned)
References
Details
Attachments
(1 file)
1.48 MB,
video/mp4
|
Details |
(see video)
Comment 1•9 years ago
|
||
I thought the medium bug 1146666 would have fixed this. Are you on the latest Nightly?
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Bryan Clark (Firefox PM) [:clarkbw] from comment #1) > I thought the medium bug 1146666 would have fixed this. Are you on the > latest Nightly? Still fails to load on the first attempt with Nightly 40.0a1 (2015-04-06)
Comment 3•9 years ago
|
||
That's weird that clicking the button a second time makes this work... the logic *should* be the same whenever you click the button (but it's different if you hit the refresh button on a failed about:reader page). What's the URL you were using to test this?
Blocks: desktop-reader
Updated•9 years ago
|
Flags: needinfo?(mjaritz)
Reporter | ||
Comment 4•9 years ago
|
||
Works for every article accessed from the medium.com frontpage. Refresh also succeeds to display the article.
Flags: needinfo?(mjaritz)
Comment 5•9 years ago
|
||
Yeah, this is clearly not completely fixed yet... :-\ I'll try to look into this later today.
Flags: needinfo?(gijskruitbosch+bugs)
Comment 6•9 years ago
|
||
This is because some url shuffling is going on and so article.url === url fails, and so we show an error. We just need to nuke that check from orbit, but there's an existing bug on that...
Flags: needinfo?(gijskruitbosch+bugs)
Comment 7•9 years ago
|
||
Should be fixed by bug 1147337 now.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 8•9 years ago
|
||
I'm no longer seeing this issue on Beta 38.0b6-build1 (20150420134330). Closing.
You need to log in
before you can comment on or make changes to this bug.
Description
•