1200658 - Can't make the 'add to Reading List' icon appear

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect)

Description

[Karen Rudnitski [:kar]]

Running build 43 on iOS 8 on my iPhone 6

Go to wikipedia or something and look at an article

I see the 'Reader View' icon in the URL bar. I tap on that and I see the article in Reader View.

However for the life of me, I can't make the 'add to Reading List' icon appear. I had it flicker at some point, but don't ask me how. In fact, there is *no* Reader View icon either, which means I can't get 'out' of Reader View.

Therefore the bug description may need to change depending on the problem, but there are definitely two problems here.

(Then there will be a navigation point, ie when I press 'back' I go back a page instead of out of Reader View which is how we manage the navigation on Android, but we can cross that bridge later.)

Comment 1

[Richard Newman [:rnewman]]

I see issues with https://mail.mozilla.org/pipermail/mobile-firefox-dev/2015-August/001451.html

Comment 2

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Karen, can you please add a few links to this bug where this happens?

Comment 3

[Karen Rudnitski [:kar]]

https://en.m.wikipedia.org/wiki/Little_Bunny_Foo_Foo
www.bbc.co.uk/news/world-europe-34113471

(although when trying to find a link to make this work, I stumbled on www.dkrisel.com/en/blog/2013/0802_xerox-workcentres_are_switching_written_numbers_when_scanning and that worked)

Comment 4

[Aaron Train [:aaronmt]]

(In reply to Karen Rudnitski [:kar] from comment #3)
> https://en.m.wikipedia.org/wiki/Little_Bunny_Foo_Foo
> www.bbc.co.uk/news/world-europe-34113471

I can see the Add to Reading List button in our toolbar fine. Tested on master.

Comment 5

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

All sites that Karen mentioned should just work. I think there is something bigger happening, possibly related to our internal web server. Trying to produce a good STR.

Comment 6

[Karen Rudnitski [:kar]]

Attachment: screenshot referenced in comment 6

Comment 7

[Karen Rudnitski [:kar]]

Ok, I have another example here that may provide more clues.

I went to techcrunch.com and scrolled down to the 'latest' and tapped on the 'Mozilla Drops First Public Preview Of Firefox For iOS' article.

It loaded, but the URL bar still states 'techcrunch.com' and hasn't loaded the exact article page (see attached screenshot as comment 6).

when I tap the RV icon, I see the progress bar, but then no change except that any icon in the URL bar disappears. 

If it also helps, I'm on 8.4.1.

Comment 8

[Aaron Train [:aaronmt]]

(In reply to Karen Rudnitski [:kar] from comment #7)
> Ok, I have another example here that may provide more clues.
> 
> I went to techcrunch.com and scrolled down to the 'latest' and tapped on the
> 'Mozilla Drops First Public Preview Of Firefox For iOS' article.
> 
> It loaded, but the URL bar still states 'techcrunch.com' and hasn't loaded
> the exact article page (see attached screenshot as comment 6).
> 
> when I tap the RV icon, I see the progress bar, but then no change except
> that any icon in the URL bar disappears. 
> 
> If it also helps, I'm on 8.4.1.

Now that I read this, I'm interpreting this bug differently and it's sounding like bug https://bugzilla.mozilla.org/show_bug.cgi?id=1183075

Comment 9

[Stephan Leroux [:sleroux]]

Now that I read this, it's starting to sound like https://bugzilla.mozilla.org/show_bug.cgi?id=1201592 :)

Comment 10

[Karen Rudnitski [:kar]]

Steph - that refers to the app being backgrounded, no? Which in my case it wouldn't be? (asking, because I don't know and/or you're thinking that bug may resolve a larger issue)

Comment 11

[Stephan Leroux [:sleroux]]

It could be the case that you were previously using the, put the phone down for the bit and reopened it and saw it not working after that. This could also explain the infrequency of the bug. For example, if you navigate to a site that does work for reader view, then background the app or lock the phone then reopen it, it won't work the second time on the same site.

Comment 12

[Karen Rudnitski [:kar]]

that would be a false assumption.

This is me taking my phone in hand, navigating to the site, letting it load, pressing the RV icon, seeing the page 'load' but then nothing happen from that point on. Device still in hand.

Have done the exact same thing just navigating to wikipedia, tapping on the 'article of the day' (hemmema), and this time the full URL loads, yet not the article in RV (and the icon disappears).

Any logs that I could take? ... rnewman is in London next week and I could hand him my phone unless you want to tell me what to do in xcode.

Comment 13

[Stephan Leroux [:sleroux]]

I was just able to produce this on another tech crunch article: http://techcrunch.com/2015/05/21/firefox-for-ios-begins-its-beta-testing-program/#.ib0hwl:wYYx but only some of the time. I haven't been able to get a steady STR but I think I can reproduce it enough to investigate. :st3fan, mind if I take this one unless you have better STR?

Comment 14

[Stephan Leroux [:sleroux]]

Interestingly I'm able to reproduce this 100% of the time when following the same STR as this bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1194501. It looks like what's happening is that because the web server no longer accepts connects, it's not displaying the reader version of the page. The reason the icon still disappears is because that code lives in the user script which runs on the current page. I'm just about to land the patch that will fix that bug so we should do a test flight next week and see if we can still reproduce this after that fix as has landed.

Comment 15

[Richard Newman [:rnewman]]

(In reply to Karen Rudnitski [:kar] from comment #12)
> that would be a false assumption.
> 
> This is me taking my phone in hand, navigating to the site, letting it load,
> pressing the RV icon, seeing the page 'load' but then nothing happen from
> that point on. Device still in hand.

That's the behavior I see in Comment 1. It's exactly as if we load the same page again, but with no reader controls and no reader icon. If that's what you mean by this bug, Karen, then I guess I can repro weirdness myself without having to shoulder-surf :D

If you mean something else, then let's split bugs!

Comment 16

[Stephan Leroux [:sleroux]]

Marking https://bugzilla.mozilla.org/show_bug.cgi?id=1194501 as a dependent issue to see if that fixes this issue. I'll keep this bug open in case that it doesn't and we can track the work to resolve any additional fixes here.

Comment 17

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

I'm pretty confident that https://bugzilla.mozilla.org/show_bug.cgi?id=1201592 will fix this.