Reader Mode: Implement "read now" action

RESOLVED FIXED in Firefox 16

Status

()

Firefox for Android
General
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: lucasr, Assigned: lucasr)

Tracking

({uiwanted})

unspecified
Firefox 16
All
Android
uiwanted
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

6 years ago
General design involves:
- Showing a "Read now" button on browser toolbar. Initial idea is to show a book-like icon on the right size.
- When "Read now" is tapped, the page is changed into readable style.

See design page for reference:

  https://wiki.mozilla.org/Fennec/NativeUI/UserExperience/ReaderMode

Need assets for the button.
Created attachment 626442 [details]
assets
Attachment #625230 - Attachment is obsolete: true
(Assignee)

Comment 3

6 years ago
Created attachment 631337 [details] [diff] [review]
Add simpler loadUrl method to load page in current tab
Attachment #631337 - Flags: review?(mark.finkle)
(Assignee)

Comment 4

6 years ago
Created attachment 631338 [details] [diff] [review]
Implement "reader mode" action

We might change this to load DOM document from the existing tab directly once we start using the chromeless GeckoApp thingy that Wes is working on. For now, it's saner to just load the article in the same tab from the URL.
Attachment #631338 - Flags: review?(mark.finkle)
Attachment #631338 - Flags: review?(mark.finkle) → review+
Comment on attachment 631337 [details] [diff] [review]
Add simpler loadUrl method to load page in current tab

Do we really need this simplification?
Attachment #631337 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 6

6 years ago
(In reply to Mark Finkle (:mfinkle) from comment #5)
> Comment on attachment 631337 [details] [diff] [review]
> Add simpler loadUrl method to load page in current tab
> 
> Do we really need this simplification?

Strictly speaking, no. But it didn't feel right to use AwesomeBar* bits inside Tab. Maybe Target should really be in GeckoApp instead?
(In reply to Lucas Rocha (:lucasr) from comment #6)
> (In reply to Mark Finkle (:mfinkle) from comment #5)
> > Comment on attachment 631337 [details] [diff] [review]
> > Add simpler loadUrl method to load page in current tab
> > 
> > Do we really need this simplification?
> 
> Strictly speaking, no. But it didn't feel right to use AwesomeBar* bits
> inside Tab. Maybe Target should really be in GeckoApp instead?

Ah, I see what you mean. Yeah, the simplification is OK for not coupling AwesomeBar into more code. Maybe we should move Target into GeckoApp, or somewhere else, to reduce the coupling.
(Assignee)

Comment 9

6 years ago
Comment on attachment 631337 [details] [diff] [review]
Add simpler loadUrl method to load page in current tab

Reader Mode is tentatively planned for Firefox 15. It's still disabled and invisible to users. Need to keep landing it in aurora and central.
Attachment #631337 - Flags: approval-mozilla-aurora?
(Assignee)

Comment 10

6 years ago
Comment on attachment 631338 [details] [diff] [review]
Implement "reader mode" action

Reader Mode is tentatively planned for Firefox 15. It's still disabled and invisible to users. Need to keep landing it in aurora and central.
Attachment #631338 - Flags: approval-mozilla-aurora?
(In reply to Lucas Rocha (:lucasr) from comment #10)
> Comment on attachment 631338 [details] [diff] [review]
> Implement "reader mode" action
> 
> Reader Mode is tentatively planned for Firefox 15. It's still disabled and
> invisible to users. Need to keep landing it in aurora and central.

Has uplifting Reader Mode to FN15 been requested by the product team?
https://hg.mozilla.org/mozilla-central/rev/de917daa32d6
https://hg.mozilla.org/mozilla-central/rev/09ff9a7af93e
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 16
Comment on attachment 631337 [details] [diff] [review]
Add simpler loadUrl method to load page in current tab

[Triage Comment]
Reader mode is not slated for FF15 in the current draft of the product roadmap.
Attachment #631337 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora-

Updated

6 years ago
Attachment #631338 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora-
You need to log in before you can comment on or make changes to this bug.