Closed Bug 602883 Opened 13 years ago Closed 13 years ago

Remove "Save Link" context-menu option on awesomescreen

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: madhava, Assigned: mfinkle)

Details

Attachments

(1 file)

patch
1.22 KB, patch
mbrubeck
: review+
Details | Diff | Splinter Review 
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 patch — — Splinter 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: 13 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.