dpa99c commented on pull request #700: URL: https://github.com/apache/cordova-plugin-inappbrowser/pull/700#issuecomment-684958777
This PR removes several existing features such as the postMessage API and `beforeload` event with no justification or replacement functionality. There are also some questionable additions such as hard-coded strings in a handful of languages. My feeling that in its current state this PR cannot be merged without breaking existing plugin functionality and the additional code is of questionable suitability for generic use by plugin users. ---------------------------------------------------------------- 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]
