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

Tapping on nodes that are part of a way does not present the Quest UI #61

Open
tordans opened this issue Mar 8, 2020 · 3 comments
Open
Labels
bug Something isn't working

Comments

@tordans
Copy link

tordans commented Mar 8, 2020

Node: https://www.openstreetmap.org/node/433727998

This is about the bus shelter challenge.

  • I have it active
  • I tab the given node
  • Nothing happens
  • To double check, I tab a different challenge (handrail) - which works

1 challenge (the one that is inactive here, but active for the test)
2 tab no action
3 tab on other challenges does show action

@wtimme
Copy link
Owner

wtimme commented Mar 8, 2020

Thanks for the report and the screenshots, @tordans! It was very helpful that you provided the link to the node. That way, I was able to take a look myself.

Indeed, at first it seems as if that particular bus stop was not selectable. However, if you long-tap, you'll see that there are quite some objects that can be selected:

moeckernbruecke-long-tap

I had a quick look at iD (to see if there was an issue with the data), but it looks fine to me. I think the culprit here is the way that the app figures out what it think you tapped. If you select the last option in the list that is presented upon long-tap, the quest dialog shows up as expected:

moeckernbruecke-quest-dialog

Maybe it makes sense to prioritize quest-enabled objects for the tap. What do you think?

@wtimme
Copy link
Owner

wtimme commented Mar 8, 2020

After a bit more digging, I realized that the issue is the lack of support for Quest-related nodes that are part of a way. I was not able to figure out a quick solution for this without introducing a lot of messy code.

@wtimme wtimme changed the title Quest answers don't show after tab Tapping on nodes that are part of a way does not present the Quest UI Mar 8, 2020
@bryceco
Copy link
Contributor

bryceco commented Mar 8, 2020

Yes, if you want to directly select a node that is part of a way without first selecting the way then it’s going to be messy. You can probably update the hit-testing code to do what you want, but you you’ll want to ensure you select the (correct) way it belongs to simultaneously. If you don’t then you could easily break assumptions made in other parts of the code.

@wtimme wtimme added the bug Something isn't working label Mar 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants