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

Location Services: Multiple popups on locked phone #136

Closed
SecUpwN opened this issue Sep 30, 2014 · 3 comments
Closed

Location Services: Multiple popups on locked phone #136

SecUpwN opened this issue Sep 30, 2014 · 3 comments
Labels

Comments

@SecUpwN
Copy link
Member

SecUpwN commented Sep 30, 2014

I noticed that the current WIP-Internal v0.1.24-alpha-build-11 shows multiple popups about non-enabled location services when unlocking the phone. These popups are especially annoying because they are being placed on top of each other, thus a user has to decide more than 5 times times on the prompt asking: You cannot track GPS data without enabling Location Services. Enable them now?

Much better would be if our App would only show that prompt once and notice if the user has made a choice or not, thus not displaying another prompt. I wonder why our App is prompting about disabled Location Services when the phone is locked anyhow. If I chose to not enable them, it should remember.

@SecUpwN SecUpwN added the bug label Sep 30, 2014
@E3V3A
Copy link
Contributor

E3V3A commented Oct 4, 2014

I can confirm this issue.

@SecUpwN
Copy link
Member Author

SecUpwN commented Oct 4, 2014

Hm.. not sure if this is easy to fix. Should be. What do you think, @tobykurien?

@tobykurien
Copy link
Contributor

It's because the prompt is coming from the service, rather than from the activity. Should be easy to fix.

SecUpwN added a commit that referenced this issue Oct 4, 2014
@SecUpwN SecUpwN closed this as completed Oct 5, 2014
tobykurien added a commit that referenced this issue Jan 2, 2015
Also removed old map viewer code
SecUpwN added a commit that referenced this issue Jan 2, 2015
SecUpwN pushed a commit that referenced this issue Apr 17, 2015
Also removed old map viewer code
SecUpwN added a commit that referenced this issue Apr 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants