1109666 - Customize behavior when long-pressing a link

Status: RESOLVED FIXED

(Firefox for iOS :: Browser, defect)

Description

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

The WKWebView has a built-in handler for long pressing links. Currently it will show an action sheet with the following options: (Page link), Open, Add to Reading List, Copy, Cancel.

Can we customize this list? Or disable or replace it completely?

Comment 1

[Brian Nicholson (:bnicholson)]

Vivek pinged me and expressed interest in contributing to iOS, so I pointed him here as a good place to start investigating.

Vivek, can you look into replacing the "Add to Reading List" option? This will likely involve 1) hiding the existing reading list option, and 2) adding a new reading list option that we can hook into ourselves.

Comment 2

[codester]

Hi I have done the pull request for this https://github.com/mozilla/firefox-ios/pull/61. Handle the long press on links and shown a custom action sheet like safari.Please let me know your suggestion/feedback.

Comment 3

[Vivek Balakrishnan[:vivek]]

Hi codester, 
Thanks for the contribution. I'll re-assign this bug to you.

Comment 4

[Richard Newman [:rnewman]]

Attachment: Pull request

Comment 5

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

This is great. I think we have to decide what specific items to put in there since there but I think this is a good start that we can merge in as a first iteration.

Comment 6

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Comment on attachment 8540961 [details] [review]
Pull request

Adding brian as a reviewer since he is the mentor for this bug.

Comment 7

[Wesley Johnston (:wesj)]

Comment on attachment 8540961 [details] [review]
Pull request

Comments in the PR. Overall looks pretty good though. Neat to see the JS->Swift interaction. I think we'll want to formalize a bit there eventually. I looked at Chromium and they have some script loaders that load files, and then just call individual methods from their ObjC code. I think little one line function calls like that would be nice. For now, I'd just at least like to see this code not inline in the Swift.

I wrote Fennec's contextmenu code, so I'm a bit biased about how I want that to look eventually, but I'm fine with it not being there now (in fact, I think landing this in incremental pieces like this is good). Also, maybe people have some good ideas on alternatives.

Comment 8

[codester]

Update the pull request.
I have injected the JavaScript.Also returned the element result as JSON. Done handling of links and Images.

Comment 9

[Brian Nicholson (:bnicholson)]

Looking good, but I think this needs another iteration or two before this is ready.

Comment 10

[Brian Nicholson (:bnicholson)]

Comment on attachment 8540961 [details] [review]
Pull request

Looks great now.