Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WKWebView built-in support status #534

Closed
guylando opened this issue Feb 16, 2019 · 15 comments
Closed

WKWebView built-in support status #534

guylando opened this issue Feb 16, 2019 · 15 comments
Milestone

Comments

@guylando
Copy link

guylando commented Feb 16, 2019

It was said here: https://cordova.apache.org/news/2018/08/01/future-cordova-ios-webview.html that cordova-ios 5 will add easy transition to wkwebview however I do not see any mentioning of wkwebview in cordova-ios 5 release: https://cordova.apache.org/announcements/2019/02/09/cordova-ios-release-5.0.0.html

Where can we found a roadmap or status on WKWebView support (and status of its limitations which gradually are fixed)?

Basically looking for an updated https://cordova.apache.org/news/2018/08/01/future-cordova-ios-webview.html post since 6 months passed and this is a hot topic now that UIWebview is deprecated and ios 13 is going to be released in 4-5 months https://www.macworld.co.uk/news/iosapps/ios-13-release-date-3678394/.

Also what is the chance that apple will remove UIWebView from ios13? it will break thousands of apps so maybe it will not happen in ios 13? Anyone has some inside information on this? Since ios 13 will be released in 4-5 months as the link above says then if it can be confirmed that UIWebView will be removed it will influence the priority given for upgrading to WKWebView. Maybe they will not touch existing apps and just prevent uploading new apps/updates which contain UIWebView?

The main problems of upgrading to WKWebview are:

  1. Iframes support/bugs
  2. Cookies support/bugs
  3. CORS support/bugs
  4. WKWebView does not seems stable enough such as the keyboard scroll bug and other bugs. Short search of people who attempted to upgrade to WKWebView shows hundreds of complaints and problems.
@janpio
Copy link
Member

janpio commented Feb 16, 2019

Great question(s).

Some context from the comments to the 5.0.0 release blog post:

jcesar:
When the post was written we didn't know Apple will require Xcode 10 in March, so considering the resources we have, we could't make the WKWebView integration on time.
So better have a 5.0 compatible with Xcode 10 before Apple's deadline than still being on 4.x, not properly working on Xcode 10, because we were waiting for the WKWebView integration.

https://cordova.apache.org/announcements/2019/02/09/cordova-ios-release-5.0.0.html#comment-4338739841

That explains why 5.0.0 does not have "promised" WKWebView support. Might have been smarter so say "some future release", and not Cordova 5 in https://cordova.apache.org/news/2018/08/01/future-cordova-ios-webview.html ;)

@guylando
Copy link
Author

guylando commented Feb 16, 2019

@janpio Actually I saw that comment however that answer does not clarify the timelines/roadmaps and the other things I asked. My main concern is for our app not to be "banned" because of the UIWebView and since ios 13 comes our in 4-5 months it raises concerns related to exact timeline/roadmap related to WKWebView and related to if apple are really going to "ban" UIWebView in ios 13.

Seems that upgrade to WKWebview is not trivial and thus adding that into the tasks of a startup (which uses cordova for its app) is a major decision because for a startup each day is important.

Thus some clarifications to those questions can help with such decision.

@janpio
Copy link
Member

janpio commented Feb 16, 2019

Actually I saw that comment however that answer does not clarify the timelines/roadmaps and the other things I asked.

I know that, but I can't give you those answers as I personally just don't know. I could give you feedback and context on your first paragraph, so that's what I did.

Your response sounds a bit confrontative (instead of just "Thanks for that info" or anything else appreciative), which makes me regret a bit that I replied at all.

We are just a group of volunteers here, working on this mostly in our free time, not a company you are paying to deliver you solutions to your problem (although I am sure some contributors would be happy to work on Cordova via a contract!). We are more than happy to accept any help - Join us on our Cordova Slack in #dev if you want to contribute.


That being said, your concerns are of course valid and I completely understand - and actually share them!

I can tell you that we are doing everything we can to get any necessary changes in. Right now this means working on the big Cordova CLI 9 release, which Cordova iOS 5 and Cordova Android 8 were part of (and that includes almost all our other many packages). After that, someone will probably focus on WKWebView stuff again.


Also what is the chance that apple will remove UIWebView from ios13? it will break thousands of apps so maybe it will not happen in ios 13? Anyone has some inside information on this? Since ios 13 will be released in 4-5 months as the link above says then if it can be confirmed that UIWebView will be removed it will influence the priority given for upgrading to WKWebView. Maybe they will not touch existing apps and just prevent uploading new apps/updates which contain UIWebView?

I personally can't imagine that they really will remove UIWebView. But who knows, sometimes they take drastic measure. Just yesterday I couldn't take part in a video conference via my laptop because my earbuds have a different plug than my computer 🤷‍♀️

But this personal opinion/hope doesn't change the fact that WKWebView is better in many ways (and unfortunately much worse in others) - and the future that Apple wants for us. So this will definitely happen.

@guylando
Copy link
Author

guylando commented Feb 16, 2019

Thanks for the info. The goal of this issue was to get some "internal" info from the developers or hopefully from some apple insider in light of the big change coming soon. My last reply goal was to clarify that I was aware of that comment in that link in order to prevent someone from closing this issue because of thinking that it answered the questions I asked.

By "that answer" I referred to the answer inside the comments in the link and not your answer.

@guylando
Copy link
Author

guylando commented Feb 16, 2019

Also from what I been reading it seems that part of the changes required to make the transition to WKWebView smooth is on the apple side and not on cordova side so regarding that of course I dont expect anybody from the cordova developers to know a timeline but maybe someone is aware of the latest progress from the apple side

and if it is even real to expect things like iframe to ever work in WKWebview (since WKWebView tightened the security measures maybe some features are permanently removed without a possibility for a workaround)
Important usecases for iframes:

  1. Stocks charts
  2. Twitter widget
  3. Iframe money withdrawal/deposit form required for easier regulation by PCI-DSS

@mgatto
Copy link

mgatto commented Feb 17, 2019

For iframes, could InAppBrowser be a useful replacement in these use cases?

@dpogue dpogue added this to the 6.0.0 milestone Mar 7, 2019
@jcesarmobile
Copy link
Member

Apple is not going to ban the UIWebView. But as it’s deprecated they can just remove it in a future release.
FYI, UIWebView is still on iOS 13 beta.

@guylando
Copy link
Author

@jcesarmobile just to clarify what you wrote: if they don't ban it but remove it, you mean that it will not work on new ios versions and work only on old ones? basically that would make UIWebView undesirable to choose to developer with.

@jcesarmobile
Copy link
Member

I mean they won’t ban apps because they use it.
But yeah, once they remove it, apps won’t work on new iOS version. For now iOS 13 seems ok, but of course people should move ASAP as we can’t know when that will happen.

@Phil15634280
Copy link

FYI: https://forums.developer.apple.com/thread/122114
Although Apple will not "ban" an app, it does seem as though they will stop accepting new apps in the future (no mention of exactly when).

@jacobg
Copy link

jacobg commented Sep 2, 2019

It sounds like it means submission of any app for review, including updates on existing apps.

@janpio
Copy link
Member

janpio commented Sep 2, 2019

Big current discussion about this here: #661

@guylando
Copy link
Author

guylando commented Sep 2, 2019

Seems like #661 is a duplicate of this issue.

@jcesarmobile
Copy link
Member

Not really, this is about the integration, the other one was about removing the UIWebView entirely. It already has a PR that allows to do it in compile time with a preference, which allows to strip it without making it a breaking change and without the need of integrating the WKWebView into cordova-ios (as long as there is a WKWebView plugin installed). So the other is more short term issue for next minor release and this one more long term for next major release.

braginxv pushed a commit to braginxv/cordova-ios that referenced this issue Nov 6, 2019
braginxv pushed a commit to braginxv/cordova-ios that referenced this issue Nov 6, 2019
braginxv pushed a commit to braginxv/cordova-ios that referenced this issue Nov 6, 2019
braginxv pushed a commit to braginxv/cordova-ios that referenced this issue Nov 8, 2019
braginxv pushed a commit to braginxv/cordova-ios that referenced this issue Nov 8, 2019
braginxv pushed a commit to braginxv/cordova-ios that referenced this issue Nov 11, 2019
@erisu
Copy link
Member

erisu commented Feb 14, 2020

Closing as UIWebView code has been removed from master and WKWebView has been added to core. #773

@erisu erisu closed this as completed Feb 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants