If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Reader Mode: Update reader popup to match state of current url

RESOLVED FIXED in Firefox 16

Status

()

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

People

(Reporter: lucasr, Unassigned)

Tracking

unspecified
Firefox 16
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

5 years ago
For instance we should probably disable or hide the "Add to Reading List" option if the current URL is already in the reading list. Madhava, Ian, any input here?
(Reporter)

Updated

5 years ago
OS: Linux → Android
Hardware: x86 → All
To be honest, having played with your test build for a while I think we should try putting "Add to Reading List" somewhere else altogether, possibly in the overflow menu. Having to choose every time between 'Add to Reading List' and 'Reader Mode' is pretty irritating. 

So this would mean that tapping the book icon would always just switch to reader mode.

--

As an aside, I realize that we can't just keep stuffing things into our already huge menu like this, so I'm going to start thinking about how we can manage down that list a bit.
(In reply to Ian Barlow (:ibarlow) from comment #1)
> To be honest, having played with your test build for a while I think we
> should try putting "Add to Reading List" somewhere else altogether, possibly
> in the overflow menu. Having to choose every time between 'Add to Reading
> List' and 'Reader Mode' is pretty irritating. 
> 
> So this would mean that tapping the book icon would always just switch to
> reader mode.
> 
> --
> 
> As an aside, I realize that we can't just keep stuffing things into our
> already huge menu like this, so I'm going to start thinking about how we can
> manage down that list a bit.

Instead of the menu, why not add it to the reader itself? I guess we would lose the ability to add a page to the reader list without viewing first.

Note that if we added "Add to Reading List" to the menu, we would need to enable/disable it based on the ability to show the page in reader mode. I dislike having menu items that are disabled for many pages.
I dislike that too, a lot. 

Just thinking out loud, it might be worth considering rolling any "Add page to..." action into a single submenu. So instead of Share, Bookmark, and Add to Reading list, we might have a single menu item called Save or Add to... or something, that opens a secondary menu with sharing, reading list and bookmarking options.
(Reporter)

Comment 4

5 years ago
Created attachment 634948 [details] [diff] [review]
(1/4) Change reader button to go straight to Reader Mode
Attachment #634948 - Flags: review?(mark.finkle)
(Reporter)

Comment 5

5 years ago
Created attachment 634949 [details] [diff] [review]
(2/4) Add needed BrowserDB api to manage reading list
Attachment #634949 - Flags: review?(mark.finkle)
(Reporter)

Comment 6

5 years ago
Created attachment 634950 [details] [diff] [review]
(3/4) Update bookmarks icons to show yellow star when active
Attachment #634950 - Flags: review?(mark.finkle)
(Reporter)

Comment 7

5 years ago
Created attachment 634951 [details] [diff] [review]
(4/4) Add reading list app menu item
Attachment #634951 - Flags: review?(mark.finkle)
Attachment #634948 - Flags: review?(mark.finkle) → review+
Attachment #634949 - Flags: review?(mark.finkle) → review+
Attachment #634950 - Flags: review?(mark.finkle) → review+
Attachment #634951 - Flags: review?(mark.finkle) → review+
(Reporter)

Comment 8

5 years ago
Pushed:
http://hg.mozilla.org/integration/mozilla-inbound/rev/bca300e4459c
http://hg.mozilla.org/integration/mozilla-inbound/rev/7e01c868ffdd
http://hg.mozilla.org/integration/mozilla-inbound/rev/1f5b0566d955
http://hg.mozilla.org/integration/mozilla-inbound/rev/32e5d236fd28

Comment 9

5 years ago
https://hg.mozilla.org/mozilla-central/rev/bca300e4459c
https://hg.mozilla.org/mozilla-central/rev/7e01c868ffdd
https://hg.mozilla.org/mozilla-central/rev/1f5b0566d955
https://hg.mozilla.org/mozilla-central/rev/32e5d236fd28
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 16
You need to log in before you can comment on or make changes to this bug.