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

Grey bar is not visible in GeoPoint widget when user opens widget again with saved location #2857

Closed
mmarciniak90 opened this issue Feb 7, 2019 · 5 comments
Assignees

Comments

@mmarciniak90
Copy link
Contributor

Software and hardware versions

Collect v1.17, older versions were not be verified
Google Maps

Problem description

Grey bar is not visible in GeoPoint widget when user opens widget again with saved location

Steps to reproduce the problem

  1. User opens GeoPoint widget with Google map
  2. User adds location and clicks on save icon
  3. User opens GeoPoint widget again
  4. User sees that grey bar is not available
  5. User clicks on remove icon
  6. Grey bar appears
Google Maps screenshot_2019-02-07-15-37-40 screenshot_2019-02-07-15-37-56
OSM screenshot_2019-02-07-15-38-27 screenshot_2019-02-07-15-38-44

Expected behavior

  • This behavior is visible only for Geopoint widget with Google maps
  • Grey bar is always available

Other information

Discovered while testing #2806

@huangyz0918
Copy link
Contributor

@opendatakit-bot claim

@zestyping
Copy link
Contributor

@mmarciniak90 As I understand it, the disappearance of the grey bar is intentional and by design. The bar says "Long press to place mark...". When there exists a saved location, the user cannot long-press to place a mark, so the instruction is not shown. After the user deletes the saved location, it is then possible again to long-press to place a mark, so the instruction is shown. Can you help me understand why you consider this to be a bug?

@huangyz0918
Copy link
Contributor

The behavior of OSM and the Google Map are inconsistent, the OSM has the bar but the Google Map has not. So I guess this is the reason why @mmarciniak90 consider it as a bug. @zestyping

@zestyping
Copy link
Contributor

zestyping commented Feb 12, 2019

@huangyz0918 Thank you for explaining, I understand.

This issue is obsolete now because the Google Maps activity and the OSM activity have been merged into a single activity after 1.17, so they now have the same behaviour. The change in the user-visible behaviour of this grey bar is noted in the description of #2776, the PR that merges the two activities into one activity, where it says:

  • When the activity is launched with a previously saved point, marker dragging and long-press are initially disabled; the user must click the "trash" button to discard the point before adding/long-pressing/dragging a new marker. Previously, the instructions "Long press to place mark to tap add marker button" would be displayed even though the user cannot do these things; now the instructions are not displayed.

@huangyz0918
Copy link
Contributor

@zestyping Thanks for explaining! So this issue is obsolete and should be closed. @mmarciniak90 , and I shall close #2859 too.

@yanokwa yanokwa closed this as completed Feb 12, 2019
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

5 participants