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

Can't manage apps permissions (overlay issue) #38

Open
licaon-kter opened this issue Aug 17, 2017 · 18 comments
Open

Can't manage apps permissions (overlay issue) #38

licaon-kter opened this issue Aug 17, 2017 · 18 comments

Comments

@licaon-kter
Copy link
Contributor

Steps:

  • have the app installed
  • install some other app (that needs some permissions, like basic Storage, but it's updated to Android 6 and higher so is able to ASK for permissions)
  • see the "Hey this app needs Storage/Camera/whatever permission, do you allow?"
  • press Ok
  • see another dialogue
    denied
  • press the button
  • scroll to tap and turn
  • enter tap and turn
  • disable
  • back
  • give permission
  • use the other app
@gabm
Copy link
Owner

gabm commented Aug 17, 2017

hmm is there any way around? Its how TnT works, by displaying an overlay...

@gabm
Copy link
Owner

gabm commented Aug 17, 2017

I was thinking about this again and i think there is not that much to do. Google decided to not allow any overlays when asking for permissions... it's different than the install dialog for example, because there it's just forbidden to have an overlay over the button element itself. When asking for permission android seems to check whether there is an overlay somewhere...

TnT needs one overlay to be always enabled to enforce a certain orientation. There is no way around this I'm afraid so the workflow will probably be like this ..

The only thing that should work is disabling TnT by pressing the notification. Though i think you disabled it ...

@licaon-kter
Copy link
Contributor Author

pressing the notification

That a good joke ;) of course I hide it (yes, Android thanks for forcing the devs to either have an useless notification or killing the app... really... great... idea)

@gabm
Copy link
Owner

gabm commented Aug 17, 2017

i know u have it disabled. But is important to state here that people who still have it (like me) should be able to temporarily disable TnT using that button... that's by far the fastest way..

Can u name an app that this happened? I'd like to test whether the notification does what it should in this case...

@licaon-kter
Copy link
Contributor Author

Any app (on Android 6) that has some permission (storage, record audio, photo, phone, whatever), eg. take your Gallery app, force close it, delete all data, start it... it should ask for "storage permission"

@gabm
Copy link
Owner

gabm commented Aug 17, 2017

hmm what a pity, on nougat that doesn't happen :/ can't reproduce...

Okay, I'll close here as won't fix.. there is nothing to do.

@gabm gabm closed this as completed Aug 17, 2017
@licaon-kter
Copy link
Contributor Author

I guess you've checked this already?

@gabm
Copy link
Owner

gabm commented Aug 28, 2017

Could u please give me some more information about the version of Android that u use and a clear way of reproducing the issue with 2.7.0. As long as I cannot reproduce it's very difficult :/

@gabm gabm reopened this Aug 28, 2017
@haarp
Copy link

haarp commented Aug 28, 2017

i think there is not that much to do.

Maybe I am missing something, but what about only showing the overlay when the rotate button is to be displayed? It seems the issue is that the overlay is active all the time, even when it isn't even needed.

@gabm
Copy link
Owner

gabm commented Aug 28, 2017

well basically there are two overlays: one is displaying the button and this is only active when necessary...

But the second overlay is basically an empty and invisible view.. this one is requesting the system to have a certain orientation. Therefore it needs to be active all the time...

@licaon-kter
Copy link
Contributor Author

So you are not seeing this say in Android Studio emulating an Android 6 device?

@haarp
Copy link

haarp commented Aug 28, 2017

But the second overlay is basically an empty and invisible view.. this one is requesting the system to have a certain orientation. Therefore it needs to be active all the time...

Hmm, only if you request an orientation that isn't the default. If rotation is disabled by default, then you'll only need an overlay when you want to switch to landscape. Not ideal, but a workaround.

@gabm
Copy link
Owner

gabm commented Aug 28, 2017

So you are not seeing this say in Android Studio emulating an Android 6 device?

good idea, I'll check... on my android 7 i couldn't reproduce ...

@gabm
Copy link
Owner

gabm commented Aug 28, 2017

Hmm, only if you request an orientation that isn't the default.

this I tried in the past but the recognition of the default orientation was very very unreliable. Also i think that the suggested workaround would result in unpredictable behavior... :/

@gkaklas
Copy link

gkaklas commented Sep 13, 2017

Hello

I too had this issue on Android 6. I recently installed Android 7 and saw 9897157 in action. I can't reproduce the problem any more, and after giving the permission to other apps Tap 'n' Turn seems to be working without any issues.

So, if I've understood correctly, this issue can only reproduced in Android 6 and there isn't anything that can be done to fix it there?

@gabm
Copy link
Owner

gabm commented Sep 13, 2017

actually I didn't find time to check in emulator :/ ... thats still open, any help of course highly appreciated

@michaelof
Copy link

michaelof commented Oct 1, 2017

I can confirm this issue for Android 6 on a Lenovo C2.

Ifthis couldn't be fixed because of Android's restrictions, some explicit and clear hint within TapAndTurn how to workaround e.g. "Disable TaT while installing other apps ..." would be very helpful for new users.

Took me a while to find out that this "screen overlay detected" came from TapAndTurn and NOT from Firefox, or the webpage I've visited. First when I've got this issue also in Nextcloud app, I've started to search more widely. And first after finding this page https://www.androidpit.de/display-overlay-erkannt-fehler-beheben-loesung I was able to detect TapAndTurn as the delinquent ;-)

@licaon-kter
Copy link
Contributor Author

licaon-kter commented Oct 1, 2017

Many apps have this overlay, eg. CF.lumen or NavApps, and sometimes it complains about one or another, you don't really know which one. It's a pain. :(

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

No branches or pull requests

5 participants