-
Notifications
You must be signed in to change notification settings - Fork 210
Switch to WKWebView instead of UIWebView #260
Comments
Mattt Thompson has a nice breakdown of it here: http://nshipster.com/wkwebkit/ |
I think this is worthwhile as there's a good chance we'll drop iOS 7 support. |
This is a status message posted by @SergioEstevao in an initial/tentative PR. I'm posting this here for context: Trying to upgrade the uses of UIWebView to WKWevView, here a list of main issues to solve:
|
Custom fonts are now loading in the web view. |
I'll be implementing native callbacks in a separate branch named |
When integrating this into WPiOS we have these issues:
|
@SergioEstevao - Changing back and forth from source mode to regular mode should be working fine now. I just applied a fix for the issues you were seeing. |
Fixed the remaining issues we'd found. Will keep testing before creating a PR. |
Fixed an issue with device rotation by removing an old hack that seems not to be necessary anymore. Ping @bummytime to confirm. |
@rachelmcr, @bummytime: Added several fixes for the focusing issues and recovery. Also added a fix for the input accessory view issues when using an iPad. More specifically:
|
Due to limitations in In particular we're not able to present images from private blogs, as there's no mechanism to add authentication cookies to those requests. We'll keep this issue open and branch More information: |
I'm dissociating myself from this task for the time being, and until cookies-adding support for |
Good work on this @diegoreymendez! We will keep our fingers crossed for real cookie support soon... |
Just leaving a note that the webkit bug was resolved a couple weeks ago, so hopefully it'll land on iOS 11 |
|
Title has it. This is an iOS 8 only api, but we could consider regressing to UIWebView for iOS 7 users.
Marketing blurb:
The text was updated successfully, but these errors were encountered: