1149633 - clicking Reading List items displays page in Reader View

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: Awesomescreen, defect)

Description

[Edwin Wong [:edwong]]

1. add items to RL
2. click on a article in the RL sidebar

actual: always displays in Reader View mode. 

expected: it should display in the mode that it was added or match the desktop experience(displays in the mode that you're currently in)

Comment 1

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

AFAIK, it should open in the mode you added it in. Since we do not have a "side bar" as per desktop. We do not have this "mode" that you stay in.

CC'ing margaret. Didn't we used to open the link in Reader View or not based on how you added to RL (in Reader View vs. not)?

Comment 2

[:Margaret Leibovic]

(In reply to Anthony Lam (:antlam) from comment #1)
> AFAIK, it should open in the mode you added it in. Since we do not have a
> "side bar" as per desktop. We do not have this "mode" that you stay in.
> 
> CC'ing margaret. Didn't we used to open the link in Reader View or not based
> on how you added to RL (in Reader View vs. not)?

No, we have always attempted to open the page in reader view, and we just fall back to the original URL if that fails.

I understand this behavior is inconsistent with desktop, but there isn't a straightforward way to do this with our current implementation, so I think changing this behavior is out of the scope of what we can do for Fx38.

Comment 3

[Blake Winton (:bwinton) (:☕️)]

In that case, would it make sense to change Desktop to match Mobile?
(I'll be honest, I expected Desktop to behave like Mobile, and was surprised when it didn't.)

Comment 4

[Edwin Wong [:edwong]]

I might be less opinionated about this - but the fact that toggling Reader View back to normal view causes a refresh - creates a user cost.

Desktop has the correct behavior: opens the URL in the current mode you're in (i.e. currently in RV clicking displays in RV, currently in normal clicking displays in normal)

As for mobile - this seems like central to the experience, opening a item from RL, and will put undue pressure on RV rendering properly.  It would be much better to default the other way around... Normal mode as default and you must tap the RV book icon to switch it.  Can we at least flip that around?

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

The current Mobile behavior has been in use for over a year. I don't think we're going to change it as a knee-jerk reaction to a choice made on Desktop.

Let's get some thoughtful UX on this. No need to push it for Fx38.

Comment 6

[:Margaret Leibovic]

Another thing to consider is that for mobile, we currently cache the article content for offline use. If we changed this behavior, we wouldn't automatically have offline support.

Also, this is similar to what services like Pocket do - show a readerized version of the content, but default to the original content if we couldn't parse it as an article.

Comment 7

[Edwin Wong [:edwong]]

got mixed up with desktop - this isn't a regression, this is current behavior. I'm fine to defer to future release.

Comment 8

[Firefox Bug Husbandry Bot]

Bulk move: https://bugzilla.mozilla.org/show_bug.cgi?id=1399539

Comment 9

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.