[
https://issues.apache.org/jira/browse/CB-11074?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15266170#comment-15266170
]
ASF GitHub Bot commented on CB-11074:
-------------------------------------
Github user kenichi-fukushima commented on the pull request:
https://github.com/apache/cordova-plugin-wkwebview-engine/pull/7#issuecomment-216118441
A drive-by comment:
In our app, we ended up subclassing CDVWKWebViewEngine and creating another
instance of WKWebView in -pluginInizialize so that we can use configuration
values that we want. I'd suggest factoring out the code to create
WKWebViewConfiguration to another method so that Cordova users can customize
the logic by code rather than static configurations in a file. This means you
can't create a web view instance in the initializer any more. This is OK
because CDVWKWebViewEngine doesn't have to conform to the usual plugin
semantics and you can add a factory method like -webView.
> WKWebView configuration is not considered
> -----------------------------------------
>
> Key: CB-11074
> URL: https://issues.apache.org/jira/browse/CB-11074
> Project: Apache Cordova
> Issue Type: Bug
> Components: Plugin WKWebViewEngine
> Environment: ios
> Reporter: Alexis Kofman
> Priority: Critical
> Labels: ios, triaged
>
> Regarding the code of WKWebView.h, WKWebViewConfiguration seems to be a
> readonly property.
> {code}
> @property (nonatomic, readonly, copy) WKWebViewConfiguration *configuration;
> {code}
> But in this plugin, we are trying to update the WKWebView configuration after
> initialising it and it looks like all this part is not considered :
> https://github.com/apache/cordova-plugin-wkwebview-engine/blob/master/src/ios/CDVWKWebViewEngine.m#L146-L150
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
