Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some bugs corrected #5

Closed
wants to merge 3 commits into from
Closed

Some bugs corrected #5

wants to merge 3 commits into from

Conversation

javray
Copy link
Contributor

@javray javray commented Oct 2, 2013

Correct some bugs in Android and iOS versions.

java.lang.RuntimeException: java.lang.Throwable: Warning: A WebView method was called on thread 'WebViewCoreThread'. All WebView methods must be called on the UI thread. Future versions of WebView may not support use on other threads. FIXED
Remove buggy code
Some chars like 'º' don't escape well with function window.escape, is better use encodeURIComponent to prevent errors on decode.
@agrieve
Copy link
Contributor

agrieve commented Dec 6, 2013

Thanks for the fixes! Sorry to not have seen them sooner.

The Android one has already been fixed in the meantime, but I've reported the iOS one as:
https://issues.apache.org/jira/browse/CB-5591

And merged it in:
http://git-wip-us.apache.org/repos/asf/cordova-plugin-inappbrowser/commit/f448ce88

Please close the pull request.

@agrieve
Copy link
Contributor

agrieve commented Feb 10, 2014

@javray Please close this PR

@javray
Copy link
Contributor Author

javray commented Feb 10, 2014

Sorry, I forgot to close it.

@javray javray closed this Feb 10, 2014
JohnCashmore pushed a commit to shapingcloud/cordova-plugin-inappbrowser that referenced this pull request Feb 20, 2015
growthengineering pushed a commit to growthengineering/cordova-plugin-inappbrowser that referenced this pull request Jul 10, 2020
[ TKA-320 ] Fix topnotch ios bar

Approved-by: Eric Nyangezi <eric.nyangezi@growthengineering.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants