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

Geofence - enhance dialog and UX #63

Closed
RoadXY opened this issue Dec 3, 2015 · 12 comments
Closed

Geofence - enhance dialog and UX #63

RoadXY opened this issue Dec 3, 2015 · 12 comments

Comments

@RoadXY
Copy link
Contributor

RoadXY commented Dec 3, 2015

I'm currently (still) working on this every night (so please don't change any file associated with it)!
Trying to find how to use onItemClick in listview (onItemLongClick works though).

Also in the add location dialog only a part of the address is shown.
Will continue to work on this tonight.

Screenshots:

Automatically find local address:
screenshot_20151203-093114

Edit radius and a (non-) working manual button for manual input of Latitude and Longitude input:
screenshot_20151203-093203

New Geofence overview:
screenshot_20151203-093213

@galadril
Copy link
Owner

galadril commented Dec 3, 2015

Looks good!

@arjen8
Copy link

arjen8 commented Dec 3, 2015

Additional question (without creating a new issue): After adding a geofence location, the google maps image isn't visible. According to your images, it has to work properly. How is this possible?

screenshot_2015-12-03-09-44-15

@RoadXY
Copy link
Contributor Author

RoadXY commented Dec 3, 2015

You build your own APK with your own debug key.
The release APK, which are signed by Galadril's developer key, are white listed on his Google Developer Console.
I had my debug key white listed on his Developer Console to test this ;-)

See the logcat on your phone.

@arjen8
Copy link

arjen8 commented Dec 3, 2015

Ah thanks for explaining!

@galadril
Copy link
Owner

galadril commented Dec 9, 2015

How are you progressing with this case?
I needed to change the GeoSettings activity because of location issues..

The LocationManager class of Google contained to many bugs, I needed to rewrite it to the New Google API (via GoogleApiClient connection).

@RoadXY
Copy link
Contributor Author

RoadXY commented Dec 9, 2015

Oh okay.
I'll finish my current work and create a pull request

@RoadXY
Copy link
Contributor Author

RoadXY commented Dec 10, 2015

Pull request: #79

@galadril
Copy link
Owner

Nice one, we only need to implement 'Manual' to close this case 👍

@RoadXY
Copy link
Contributor Author

RoadXY commented Dec 11, 2015

Yes, but you needed to implement several changes to the Geo activity so I did it with some rush ;-)

@RoadXY
Copy link
Contributor Author

RoadXY commented Dec 12, 2015

screenshot_20151212-124403
screenshot_20151212-124329

@RoadXY
Copy link
Contributor Author

RoadXY commented Dec 14, 2015

@galadril, could you test the manual (lat/long) implementation?
And the way I implemented switching layout for manual and address?

Thanks!

galadril added a commit that referenced this issue Dec 14, 2015
- #82 Orientation changed to allow landscape
- #86 Fix for Intent leaked after welcome wizard
- #88 Fix for Receiver not registered
- #63 latest version of Geofence dialog (by roadXY)
- #24 Added a helping wizard for new users
- #lots of other fixes!!
galadril added a commit that referenced this issue Dec 14, 2015
- #82 Orientation changed to allow landscape
- #86 Fix for Intent leaked after welcome wizard
- #88 Fix for Receiver not registered
- #63 latest version of Geofence dialog (by roadXY)
- #24 Added a helping wizard for new users
- #lots of other fixes!!
@RoadXY RoadXY closed this as completed Dec 14, 2015
@RoadXY RoadXY mentioned this issue Dec 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants