Skip to content
This repository has been archived by the owner on May 19, 2021. It is now read-only.

App exit on backbutton, if ad banner overlay=false (on ionic) #19

Closed
warcry2000 opened this issue Oct 24, 2016 · 7 comments
Closed

App exit on backbutton, if ad banner overlay=false (on ionic) #19

warcry2000 opened this issue Oct 24, 2016 · 7 comments

Comments

@warcry2000
Copy link
Collaborator

Whenever the app was at main page or other page, when i click onHardwareBack it will direct exit the apps without going back previous state or prompt toast message.

If overlay=true work perfectly.

When you load the banner, it is as if the focus to stay somewhere.

@warcry2000
Copy link
Collaborator Author

hello,
I just solve, the problem is that the 'webview' does not have focus to display the banner. Only in android.

in

if (bannerAtTop) {
  parentView.addView(adView, 0);
} else {
  parentView.addView(adView);
}
parentView.bringToFront();
parentView.requestLayout();

add
parentView.requestFocus();

getting

if (bannerAtTop) {
  parentView.addView(adView, 0);
} else {
  parentView.addView(adView);
}
parentView.bringToFront();
parentView.requestLayout();
parentView.requestFocus();

@ratson
Copy link
Owner

ratson commented Oct 25, 2016

@raulpopi Thanks, would you put this into a PR?
If not, I could include it myself later.

@warcry2000
Copy link
Collaborator Author

Hello, I've never done a PR ... I have to study how it's done.
For now if you want to include you when you can, as is faster.
Thanks

@ratson ratson closed this as completed in 30aee84 Oct 26, 2016
@ratson
Copy link
Owner

ratson commented Oct 26, 2016

@raulpopi Released the fix. For creating pull request, you may refer to https://help.github.com/articles/creating-a-pull-request/

@erperejildo
Copy link

erperejildo commented Feb 18, 2018

@ratson I'm facing the same issue right now. Is this fix integrated to the master branch?

@shafimcs
Copy link

@erperejildo is this fixed?

@erperejildo
Copy link

I don't know @shafimcs I'm not using this anymore

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants