timbru31 commented on a change in pull request #1112:
URL: https://github.com/apache/cordova-docs/pull/1112#discussion_r468135278



##########
File path: www/docs/en/dev/config_ref/index.md
##########
@@ -299,8 +299,8 @@ android-targetSdkVersion(integer) <br/> ==Android== | 
*Default: Dependent on cor
 AppendUserAgent(string) <br/> ==Android== ==iOS== | If set, the value will 
append to the end of old UserAgent of webview. When using with 
OverrideUserAgent, this value will be ignored.
 BackgroundColor(string) <br/> ==Android== ==iOS== ==Windows== | Sets the app's 
background color. Supports a four-byte hex value, with the first byte 
representing the alpha channel, and standard RGB values for the following three 
bytes. <br/> For Windows, the alpha channel is ignored. <br/> __Note__: 
`transparent` value will set the application tile background to the accent 
color on Windows.
 BackupWebStorage(string) <br/> ==iOS== | *Default: cloud* <br/> Allowed 
values: none, local, cloud. <br/>   Set to cloud to allow web storage data to 
backup via iCloud. Set to local to allow only local backups via iTunes sync. 
Set to none prevent web storage backups.
-CordovaWebViewEngine(string) <br/> ==iOS== | *Default: CDVUIWebViewEngine* 
<br/>  This sets the WebView engine plugin to be used to render the host app. 
The plugin must conform to the CDVWebViewEngineProtocol protocol. The 'value' 
here should match the 'feature' name of the WebView engine plugin that is 
installed. This preference usually would be set by the WebView engine plugin 
that is installed, automatically.
-CordovaDefaultWebViewEngine(string) <br/> ==iOS== | *Default: 
CDVUIWebViewEngine* <br/> As the previous setting, CordovaWebViewEngine, this 
enables you to override the default fallback WebView with a different plugin. 
The default WebView uses a private plugin, CDVUIWebViewEngine. You can override 
this if you need to use a public plugin instead. A use case for this setting is 
where you need to use something other than the default CDVUIWebViewEngine to 
render the host app. If CordovaDefaultWebViewEngine and CordovaWebViewEngine is 
set, CordovaWebViewEngine will be chosen to render the host app for devices 
running iOS9 and above, while CordovaDefaultWebViewEngine will be the fallback 
for iOS8. The plugin must conform to the CDVWebViewEngineProtocol protocol. The 
'value' here should match the 'feature' name of the WebView engine plugin that 
is installed.
+CordovaWebViewEngine(string) <br/> ==iOS== | *Default: CDVWKWebViewEngine* 
<br/>  This sets the WebView engine plugin to be used to render the host app. 
The plugin must conform to the CDVWebViewEngineProtocol protocol. The 'value' 
here should match the 'feature' name of the WebView engine plugin that is 
installed. This preference usually would be set by the WebView engine plugin 
that is installed, automatically.

Review comment:
       ```suggestion
   CordovaWebViewEngine(string) <br/> ==iOS== | *Default: CDVWebViewEngine* 
<br/>  This sets the WebView engine plugin to be used to render the host app. 
The plugin must conform to the CDVWebViewEngineProtocol protocol. The 'value' 
here should match the 'feature' name of the WebView engine plugin that is 
installed. This preference usually would be set by the WebView engine plugin 
that is installed, automatically.
   ```
   
   This is actually not right: 
https://github.com/apache/cordova-ios/commit/240a177d9dd5966768c20ab14920b708af3baa0b#diff-3d962b8f101cacedea7abce2ca246eacR504

##########
File path: www/docs/en/dev/config_ref/index.md
##########
@@ -380,8 +380,8 @@ Examples:
 <preference name="AppendUserAgent" value="My Browser" />
 <preference name="target-device" value="universal" />
 <preference name="deployment-target" value="7.0" />
-<preference name="CordovaWebViewEngine" value="CDVUIWebViewEngine" />
-<preference name="CordovaDefaultWebViewEngine" value="CDVUIWebViewEngine" />
+<preference name="CordovaWebViewEngine" value="CDVWKWebViewEngine" />
+<preference name="CordovaDefaultWebViewEngine" value="CDVWKWebViewEngine" />

Review comment:
       ```suggestion
   <preference name="CordovaDefaultWebViewEngine" value="CDVWebViewEngine" />
   ```

##########
File path: www/docs/en/dev/config_ref/index.md
##########
@@ -380,8 +380,8 @@ Examples:
 <preference name="AppendUserAgent" value="My Browser" />
 <preference name="target-device" value="universal" />
 <preference name="deployment-target" value="7.0" />
-<preference name="CordovaWebViewEngine" value="CDVUIWebViewEngine" />
-<preference name="CordovaDefaultWebViewEngine" value="CDVUIWebViewEngine" />
+<preference name="CordovaWebViewEngine" value="CDVWKWebViewEngine" />

Review comment:
       ```suggestion
   <preference name="CordovaWebViewEngine" value="CDVWebViewEngine" />
   ```

##########
File path: www/docs/en/dev/config_ref/index.md
##########
@@ -299,8 +299,8 @@ android-targetSdkVersion(integer) <br/> ==Android== | 
*Default: Dependent on cor
 AppendUserAgent(string) <br/> ==Android== ==iOS== | If set, the value will 
append to the end of old UserAgent of webview. When using with 
OverrideUserAgent, this value will be ignored.
 BackgroundColor(string) <br/> ==Android== ==iOS== ==Windows== | Sets the app's 
background color. Supports a four-byte hex value, with the first byte 
representing the alpha channel, and standard RGB values for the following three 
bytes. <br/> For Windows, the alpha channel is ignored. <br/> __Note__: 
`transparent` value will set the application tile background to the accent 
color on Windows.
 BackupWebStorage(string) <br/> ==iOS== | *Default: cloud* <br/> Allowed 
values: none, local, cloud. <br/>   Set to cloud to allow web storage data to 
backup via iCloud. Set to local to allow only local backups via iTunes sync. 
Set to none prevent web storage backups.
-CordovaWebViewEngine(string) <br/> ==iOS== | *Default: CDVUIWebViewEngine* 
<br/>  This sets the WebView engine plugin to be used to render the host app. 
The plugin must conform to the CDVWebViewEngineProtocol protocol. The 'value' 
here should match the 'feature' name of the WebView engine plugin that is 
installed. This preference usually would be set by the WebView engine plugin 
that is installed, automatically.
-CordovaDefaultWebViewEngine(string) <br/> ==iOS== | *Default: 
CDVUIWebViewEngine* <br/> As the previous setting, CordovaWebViewEngine, this 
enables you to override the default fallback WebView with a different plugin. 
The default WebView uses a private plugin, CDVUIWebViewEngine. You can override 
this if you need to use a public plugin instead. A use case for this setting is 
where you need to use something other than the default CDVUIWebViewEngine to 
render the host app. If CordovaDefaultWebViewEngine and CordovaWebViewEngine is 
set, CordovaWebViewEngine will be chosen to render the host app for devices 
running iOS9 and above, while CordovaDefaultWebViewEngine will be the fallback 
for iOS8. The plugin must conform to the CDVWebViewEngineProtocol protocol. The 
'value' here should match the 'feature' name of the WebView engine plugin that 
is installed.
+CordovaWebViewEngine(string) <br/> ==iOS== | *Default: CDVWKWebViewEngine* 
<br/>  This sets the WebView engine plugin to be used to render the host app. 
The plugin must conform to the CDVWebViewEngineProtocol protocol. The 'value' 
here should match the 'feature' name of the WebView engine plugin that is 
installed. This preference usually would be set by the WebView engine plugin 
that is installed, automatically.
+CordovaDefaultWebViewEngine(string) <br/> ==iOS== | *Default: 
CDVWKWebViewEngine* <br/> As the previous setting, CordovaWebViewEngine, this 
enables you to override the default fallback WebView with a different plugin. 
The default WebView uses a private plugin, CDVWKWebViewEngine. You can override 
this if you need to use a public plugin instead. A use case for this setting is 
where you need to use something other than the default CDVWKWebViewEngine to 
render the host app. If CordovaDefaultWebViewEngine and CordovaWebViewEngine is 
set, CordovaWebViewEngine will be chosen to render the host app for devices 
running iOS9 and above, while CordovaDefaultWebViewEngine will be the fallback 
for iOS8. The plugin must conform to the CDVWebViewEngineProtocol protocol. The 
'value' here should match the 'feature' name of the WebView engine plugin that 
is installed.

Review comment:
       ```suggestion
   CordovaDefaultWebViewEngine(string) <br/> ==iOS== | *Default: 
CDVWebViewEngine* <br/> As the previous setting, CordovaWebViewEngine, this 
enables you to override the default fallback WebView with a different plugin. 
The default WebView uses a private plugin, CDVWKWebViewEngine. You can override 
this if you need to use a public plugin instead. A use case for this setting is 
where you need to use something other than the default CDVWKWebViewEngine to 
render the host app. If CordovaDefaultWebViewEngine and CordovaWebViewEngine is 
set, CordovaWebViewEngine will be chosen to render the host app for devices 
running iOS9 and above, while CordovaDefaultWebViewEngine will be the fallback 
for iOS8. The plugin must conform to the CDVWebViewEngineProtocol protocol. The 
'value' here should match the 'feature' name of the WebView engine plugin that 
is installed.
   ```




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to