[ 
https://issues.apache.org/jira/browse/CB-11074?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15246100#comment-15246100
 ] 

ASF GitHub Bot commented on CB-11074:
-------------------------------------

Github user shazron commented on the pull request:

    
https://github.com/apache/cordova-plugin-wkwebview-engine/pull/7#issuecomment-211493781
  
    It still needs to be able to update it dynamically. I would refactor it to 
not remove the existing code, but create a new `- (WKWebViewConfiguration*) 
createConfigurationFromSettings:(NSDictionary*)settings;`
    
    That way it can be used in both updateSettings and initWithFrame


> 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
>
> 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]

Reply via email to