[web_view]Expose the allowsLinkPreview property in WKWebView for iOS
Based off of https://github.com/flutter/plugins/pull/5110 So most of the credit goes to @b099l3 Also exposed it through the webview_controller
This PR fixes this issue:
- https://github.com/flutter/flutter/issues/69748
- (duplicate) https://github.com/flutter/flutter/issues/100783
Pre-launch Checklist
- [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
- [x] I read and followed the [relevant style guides] and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use
dart format.) - [x] I signed the [CLA].
- [x] The title of the PR starts with the name of the package surrounded by square brackets, e.g.
[shared_preferences] - [x] I listed at least one issue that this PR fixes in the description above.
- [x] I updated
pubspec.yamlwith an appropriate new version according to the [pub versioning philosophy], or this PR is [exempt from version changes]. - [x] I updated
CHANGELOG.mdto add a description of the change, [following repository CHANGELOG style]. - [x] I updated/added relevant documentation (doc comments with
///). - [x] I added new tests to check the change I am making, or this PR is [test-exempt].
- [x] All existing and new tests are passing.
@cyanglaz @bparrishMines I saw that you reviewed https://github.com/flutter/plugins/pull/5110 the first time. Sorry to make you re-review it... but I would love if you could review this. I did my best to port it over from that pr. Thank you for your help with this! Note, I checked the Linux and Mac checks, and saw that they had failed on previous pulls that went in, so I am assuming they failing on the main branch? Let me know though if there is something I need to do @b099l3 I figured you would want to be aware that I have ported your patch over to the new repo
cc @bparrishMines
@camfrandsen No worries, I'm glad it was of some use.
I had a guilt in the back of my mind to come back to it at some point and get it merged. So I can sleep easily now. 😅
@camfrandsen Please see https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#changing-federated-plugins for how to structure this PR so that it will pass tests. Please let us know if you have any issues setting that up!
I am splitting this into two pulls since I touched the webview_flutter_platform_interface: https://github.com/flutter/packages/pull/5292 https://github.com/flutter/packages/pull/5293
I am splitting this into two pulls since I touched the webview_flutter_platform_interface
That's step 3 of the process, which should only be done after the previous steps (which include approvals of the combined PR) are complete.
@stuartmorgan Ah, sorry about that. I looked at the failures of the checks, and it said that it needed to be broken up... I will re-open this one and close the other two.
Update from triage: still waiting on the wrapper generator, which is currently under very active development.
Sounds like this is blocked by other work. I am converting this to draft. feel free to request my review when it's ready.
Any updates on this?
Update from triage: the conversion to the new wrapper automation that this is blocked on is likely to be done sometime around August.
Update from triage: still likely on track for August.
(PR Triage): Hey @stuartmorgan is this still planned to move forward? Is it blocked on something?
https://github.com/flutter/packages/pull/6602 is the blocking issue; it's just slipped slightly.
From triage: This should be unblocked soon; https://github.com/flutter/packages/pull/7794 updates Android, and a similar PR for iOS will follow shortly.
Sorry for the longer-than-expected delay here. https://github.com/flutter/packages/pull/8311 will unblock this, and is going through final review now.
From triage: @camfrandsen Sorry for the long delay on the blocking PR. Are you still planning on updating this for the new API?
I went ahead and updated the code to the new ProxyApi wrapper. There doesn't seem to be a comparable feature on Android to control preview links.
Sounds like this should be ready for review then.