Closed
Bug 1240728
Opened 9 years ago
Closed 9 years ago
Remove add to reading list button from share overlay
Categories
(Firefox for Android Graveyard :: Reading List, defect)
Firefox for Android Graveyard
Reading List
Tracking
(firefox48 verified)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | verified |
People
(Reporter: Margaret, Assigned: ahunt)
References
Details
Attachments
(4 files)
One big problem with the current reading list is that not all articles are necessarily available offline, so it's inconsistent.
We should stop letting users add things to the reading list that are reader-view-able. To do this, we need to stop letting them save arbitrary URLs to their reading list.
Flags: needinfo?(alam)
Comment 1•9 years ago
|
||
Agree, let's remove it from the share overlay.
FYI, we are almost about to add "Open in Firefox" in this same menu too (in bug 1155860). But I don't think that should affect this work.
Flags: needinfo?(alam)
Comment 2•9 years ago
|
||
Attaching visual of what the share overlay would look like, once bug 1155860 is done. But, we aren't going to block on that bug (bug 1155860).
This is just to show what we're talking about.
Assignee | ||
Comment 3•9 years ago
|
||
String resources are shared with the main app menu, and will therefore be remove
in Bug 1234319 which aims to remove those items.
Review commit: https://reviewboard.mozilla.org/r/38593/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38593/
Attachment #8727664 -
Flags: review?(michael.l.comella)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → ahunt
Assignee | ||
Comment 4•9 years ago
|
||
Comment on attachment 8727664 [details]
MozReview Request: Bug 1240728 - Remove add to reading list from the share overlay r?mcomella
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38593/diff/1-2/
Assignee | ||
Comment 5•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/38711/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38711/
Attachment #8727971 -
Flags: review?(michael.l.comella)
Updated•9 years ago
|
Attachment #8727664 -
Flags: review?(michael.l.comella) → review+
Comment 6•9 years ago
|
||
Comment on attachment 8727664 [details]
MozReview Request: Bug 1240728 - Remove add to reading list from the share overlay r?mcomella
https://reviewboard.mozilla.org/r/38593/#review38617
Comment 7•9 years ago
|
||
Comment on attachment 8727971 [details]
MozReview Request: Bug 1240728 - Post: cleanup / remove AddToReadingList from OverlayActionService r?mcomella
https://reviewboard.mozilla.org/r/38711/#review38623
::: mobile/android/base/java/org/mozilla/gecko/overlays/service/OverlayActionService.java
(Diff revision 1)
> @Override
> public void run() {
> shareTypes.clear();
>
> shareTypes.put(ShareMethod.Type.ADD_BOOKMARK, new AddBookmark(context));
> - shareTypes.put(ShareMethod.Type.ADD_TO_READING_LIST, new AddToReadingList(context));
`ShareMethod.Type.ADD_TO_READING_LIST` is unused -- let's remove it.
Attachment #8727971 -
Flags: review?(michael.l.comella) → review+
Assignee | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/3c4f41b6babdb8a5339635495c228c0ad9fd53a0
Bug 1240728 - Remove add to reading list from the share overlay r=mcomella
https://hg.mozilla.org/integration/fx-team/rev/eeea0cd4c9f24f4e43ad18584e528f7bff1d1cc4
Bug 1240728 - Post: cleanup / remove AddToReadingList from OverlayActionService r=mcomella
Comment 9•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3c4f41b6babd
https://hg.mozilla.org/mozilla-central/rev/eeea0cd4c9f2
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Comment 10•9 years ago
|
||
Verified as fixed using:
Device: Nexus 6 (Android 6.0) and Nexus 7 (Android 5.1)
Build: Firefox for Android 48.0a1 (2016-04-10)
Updated•9 years ago
|
Updated•6 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•