Closed Bug 1246232 Opened 4 years ago Closed 4 years ago

Remove "Add to Reading List" from context menus when long-pressing on link

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- verified

People

(Reporter: antlam, Assigned: ahunt)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.
Assignee: nobody → ahunt
Comment on attachment 8727653 [details]
MozReview Request: Bug 1246232 - Remove "add to reading list" from link-long-press context menu r?liuche

https://reviewboard.mozilla.org/r/38581/#review35193

::: mobile/android/chrome/content/browser.js
(Diff revision 1)
> -            type: "Reader:AddToList",

Do we send this Reader:AddToList message anywhere else? If so, remove that from any Android listeners (and any strings, toasts, etc that might get popped up to confirm). I feel like there might be another message that gets sent back...but I guess it depends on if we are keeping the "+readinglist" icon in the menu.

::: mobile/android/locales/en-US/chrome/browser.properties
(Diff revision 1)
> -contextmenu.addToReadingList=Add to Reading List

Nice. If this needs to be uplifted, you might need to do this in a separate patch (so we don't break the "no string changes rule" which includes removing strings I guess? You could double check that with someone).
Attachment #8727653 - Flags: review?(liuche) → review+
(In reply to Chenxia Liu [:liuche] from comment #2)

> ::: mobile/android/locales/en-US/chrome/browser.properties
> (Diff revision 1)
> > -contextmenu.addToReadingList=Add to Reading List
> 
> Nice. If this needs to be uplifted, you might need to do this in a separate
> patch (so we don't break the "no string changes rule" which includes
> removing strings I guess? You could double check that with someone).

Let's not uplift any of this work. The reading list migration work should all happen in 48.
https://hg.mozilla.org/integration/fx-team/rev/64e9f80e3be5ca554e3983f12025add4ac220241
Bug 1246232 - Remove "add to reading list" from link-long-press context menu r=liuche
https://hg.mozilla.org/mozilla-central/rev/64e9f80e3be5
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
"Add to Reading List" is not an option anymore in context menus when long-pressing on a link

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)
You need to log in before you can comment on or make changes to this bug.