Closed Bug 602883 Opened 9 years ago Closed 9 years ago

Remove "Save Link" context-menu option on awesomescreen

Categories

(Firefox for Android Graveyard :: General, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: madhava, Assigned: mfinkle)

Details

Attachments

(1 file)

This option doesn't make sense when long-tapping on rows in the awesomescreen or its subsections.  You'll just end up with HTML files downloaded in your download manager, which is not a use case we're interested in (this is what save as pdf is for).
Attached patch patchSplinter Review
Only share links on awesomebar (all pages, bookmarks, history and desktop)

Matt - the API to show/hide the actions really made this easy.
Assignee: nobody → mark.finkle
Attachment #481939 - Flags: review?(mbrubeck)
Attachment #481939 - Flags: review?(mbrubeck) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/b823c73403ba
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified FIXED on builds:

Mozilla/5.0 (Maemo; Linux armv71; rv:2.0b8pre) Gecko/20101011 Namoroka/4.0b8pre Fennec/4.0b2pre

and

Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101011 Namoroka/4.0b8pre Fennec/4.0b2pre

We'll need to remove this entry in our litmus tests.
Status: RESOLVED → VERIFIED
Flags: in-litmus?(tchung)
can't find any instances of Awesomescreen-"Save Link" tests that are active.
Flags: in-litmus?(tchung) → in-litmus-
You need to log in before you can comment on or make changes to this bug.