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

Disable/Remove/Fix MainActivity Menu #43

Open
13rac1 opened this issue Feb 5, 2015 · 3 comments
Open

Disable/Remove/Fix MainActivity Menu #43

13rac1 opened this issue Feb 5, 2015 · 3 comments

Comments

@13rac1
Copy link
Member

13rac1 commented Feb 5, 2015

The three-dot menu on the MainActivity doesn't work right. The Legal Notice does nothing, and the very little of the Preferences actually work. We can't have enabled broken features. I think we could remove it altogether, but maybe it makes sense to have part of it? Thoughts?

@13rac1 13rac1 added the bug label Feb 5, 2015
@13rac1 13rac1 added this to the FAMSF Store submission milestone Feb 5, 2015
@hburgund
Copy link
Member

hburgund commented Feb 5, 2015

The reason for the Legal Notification initially was due to a requirement from Google regarding the use of Google maps, I believe. They make you put this long annoying legal language in the app somewhere and this was the way that made the most sense. But I have no idea if this is still a requirement.

I have never liked that little menu, but it was better than the alternatives way back when. So I agree to remove it if possible.

@13rac1
Copy link
Member Author

13rac1 commented Feb 7, 2015

@hburgund
Copy link
Member

hburgund commented Feb 7, 2015

OK, that's annoying. Any ideas how other apps handle this? I'd love to remove the menu, but not sure where else to put this legal mumbo-jumbo...?

I will add that the preferences have proven to be useful for testing and debugging, though most of them are not to be touched by an actual user, so it wasn't a good idea to include them in the public builds.

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

2 participants