breautek commented on PR #995: URL: https://github.com/apache/cordova-plugin-inappbrowser/pull/995#issuecomment-1669926867
> I suggest updating the PR code to match the one used in apache/cordova-ios. This will not only improve code consistency within the Cordova ecosystem but also determinate the default value for web on whether it is a debug build or not. In the PR value was always set to YES, which allowed inspection for all cases. I agree with this. The rationale behind the way cordova-ios does it is to maintain older behaviour on older iOS devices, which xcode effectively hard-coded the web inspector as enabled for debug builds, and disabled for release builds and I think this PR should follow suite. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
