[GitHub] [cordova-plugin-inappbrowser] jcesarmobile commented on pull request #828: (ios): add statusbar 'darkcontent'

[GitBox <gi...@apache.org>]

jcesarmobile commented on pull request #828:
URL: https://github.com/apache/cordova-plugin-inappbrowser/pull/828#issuecomment-741905577


   We should be careful with the `@available` usage, the code inside it is not understand if using an older SDK to build than the version on the `@available`, which means, this code doesn't compile on Xcode 10 or older.
   
   For this PR is not a big deal since we are planning a major release and can bump the engines to cordova-ios 6 instead of 5.1.0 as on my pending PR, but then noticed version 4.1.0 is already using `@available` with iOS 13 and fails to build.
   
   I've reported [the issue](https://github.com/apache/cordova-plugin-inappbrowser/issues/831)
   
   


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org