1120521 - Implement Reader Mode

Status: RESOLVED FIXED

(Firefox for iOS :: Reader View, defect)

Description

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

Implement a Reader Mode that behaves the same way as Reader Mode on Android. Try to reuse the existing style sheets and Readability engine as much as possible so that there is more commonality between the different implementations.

Comment 1

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

Attachment: Pull Request

This is a first rough implementation of Reader Mode. I would like to merge this so that we can get an idea of how this works and start filing followup bugs for both UI and code issues.

I want to iterate rather quickly on this, and it will be nice to get at least some working code in there now to enable that.

Comment 2

[Richard Newman [:rnewman]]

Comment on attachment 8548346 [details] [review]
Pull Request

Flagging me so I remember to look at the data model.

Comment 3

[Richard Newman [:rnewman]]

Comment on attachment 8548346 [details] [review]
Pull Request

Only thing I see that doesn't line up with Android (yet) is excerpt/word count. The latter needs some effort around locale detection etc. (Bug 1106380).

Comment 4

[Brian Nicholson (:bnicholson)]

Comment on attachment 8548346 [details] [review]
Pull Request

Left several questions/comments in the PR. r=me as long as they're answered/addressed, or follow-up bugs are filed to fix them.

Comment 5

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

Attachment: Pull Request

New PR since I messed up git squash. Also removed reader mode button UI code since I have a much better solution for that already.