Closed Bug 1206998 Opened 6 years ago Closed 6 years ago

Long-press link menu should offer Send Tab and/or Share

Categories

(Firefox for iOS :: Browser, defect)

All
Unspecified
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: rnewman, Assigned: johannesmarbach, Mentored)

Details

Attachments

(1 file)

48 bytes, text/x-github-pull-request
sleroux
: review+
tecgirl
: ui-review+
Details | Review
Right now the only way to send a page to another device is to load it. Given that there are usually only a couple of things in the menu, Send seems like a good addition.
This seems like a great idea. Do we want only 'Send Tab' to start?
I'd love the iOS share sheet to be in there; quick access to Pocket/Whatsapp/etc.

That might even be enough as a first step, given that Send Tab will be in the share sheet.
Attached file Pull request
This adds a "Share Link" option to the long press action sheet.
Attachment #8687692 - Flags: review?(rnewman)
Assignee: nobody → johannesmarbach
Status: NEW → ASSIGNED
Attachment #8687692 - Flags: ui-review?(randersen)
Attachment #8687692 - Flags: review?(sleroux)
Attachment #8687692 - Flags: review?(rnewman)
Has this been tested on iPad too?
(In reply to Stefan Arentz [:st3fan] from comment #4)
> Has this been tested on iPad too?

Good call, Stefan! I just tested on the iPad simulator and found that the second popover (after tapping "Share Link") was not positioned correctly because the tap location was queried too late from the gesture recognizer. I rebased the PR on master and added the fix.
Attachment #8687692 - Flags: ui-review?(randersen) → ui-review+
Comment on attachment 8687692 [details] [review]
Pull request

Code looks good - just need one more rebase and Ill give it a test run and merge it in.
(In reply to Stephan Leroux [:sleroux] from comment #7)
> Code looks good - just need one more rebase and Ill give it a test run and
> merge it in.

Thanks for reviewing, Stephan. I just rebased the PR again.
Thanks for all the rebasing.

master ecbd4ec8490a0fc6bd523da0c71d6257fb3ff9fa
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Attachment #8687692 - Flags: review?(sleroux) → review+
You need to log in before you can comment on or make changes to this bug.