Github user shazron commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
@reidab @jcesarmobile filed https://issues.apache.org/jira/browse/CB-12457
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jcesarmobile commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
@reidab Can you create a new issue for that on issues.cordova.io?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user reidab commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Following this change I'm seeing my IAB windows overlapping the status bar
in all presentation modes. Dropping the `windowLevel` of the `tmpWindow` to
`UIWindowLevelNormal` seems
Github user shazron commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Merging and lightly testing now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have th
Github user nerdmed commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
@shazron @manucorporat, i know its christmas time but guys lets forward.
This i hanging around long enough and people are working with forks to run in
production, this really aff
Github user Glenndejaeger commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Any idea when this PR will be merged into the official plugin's code base?
Don't want to use a forked version for a long time
---
If your project is set up for it, you can
Github user jlchereau commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
I have experienced the defect described here with oAuth flows but
considering [Google deprecation of webviews for oAuth
flows](https://developers.googleblog.com/2016/08/moderni
Github user jcesarmobile commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
I agree with @jonathanli2, as this way is a bit hackish, I think it's
better doing it were it's strictly necessary to fix the bug
---
If your project is set up for it, you
Github user jonathanli2 commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Would it be better to add an if condition check, and only apply the change
if WKWebView is used? so that the change will not affect UIWebView's old
behavior by any chance.
Github user shazron commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
LGTM ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user dylanvdmerwe commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Any updates with this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user nerdmed commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
@shazron @manucorporat any progress here? Would be great to see this PR
getting merged!
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user cordova-qa commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Cordova CI Build has completed successfully.
**Commit** -
[Link](https://github.com/apache/cordova-plugin-inappbrowser/pull/187/commits/f3a8fbe1c0737138d4b0e1b358b
Github user manucorporat commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
@shazron done!
renamed to tmpWindow and tmpController
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user ephemer commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
@manucorporat the term is reference counted and deinited / cleaned up.
"Garbage Collection" sounds like heresy to my ears, but the principles are
somewhat similar, so you're forg
Github user manucorporat commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
@ephemer sure! I can rename it! it was just a funny name (bad clean code
practice haha)
> does the window clean itself up if its rootViewController dismisses
itself, or
Github user shazron commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
LGTM ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user dotNetkow commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Developer here that originally requested this change. To lend more credence
to @manucorporat's changes: I tested them on an iPhone 6 running iOS 10 - works
now as expected. Th
Github user cordova-qa commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Cordova CI Build has completed successfully.
**Commit** -
[Link](https://github.com/apache/cordova-plugin-inappbrowser/pull/187/commits/f8773088d7c6e26b689931389a3
Github user cordova-qa commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
Cordova CI Build has completed successfully.
**Commit** -
[Link](https://github.com/apache/cordova-plugin-inappbrowser/pull/187/commits/ce33cf5ac733854bba8d1b578c2
Github user manucorporat commented on the issue:
https://github.com/apache/cordova-plugin-inappbrowser/pull/187
ping @ephemer @jcesarsh @shazron
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not hav
21 matches
Mail list logo