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

Unable to select the Sidewalk overlay if there is a Note on it #4529

Closed
mnalis opened this issue Oct 18, 2022 · 6 comments
Closed

Unable to select the Sidewalk overlay if there is a Note on it #4529

mnalis opened this issue Oct 18, 2022 · 6 comments
Labels

Comments

@mnalis
Copy link
Member

mnalis commented Oct 18, 2022

Once upon a time I left a note when answering Sidewalk quest. Now, when I enable Sidewalk overlay and try to click on that way (not on the note!), instead of opening Sidewalk overlay answer window, it opens that Note.

How to Reproduce

  • enable Sidewalk overlay (and disable all quests if wanted)
  • navigate to this Vinkovačka ulica road
  • try to click on the road far away from the note
  • Note window opens
small_SVID_20221019_003737_1.mp4

Expected Behavior
I would expect that instead of the Note window, an Sidewalk overlay chooser would open.

Versions affected
Both SC 47.2 upstream, and my fork with 48.0-beta1+; Android 10

@mnalis mnalis added the bug label Oct 18, 2022
@mnalis
Copy link
Member Author

mnalis commented Oct 18, 2022

It also happens with other way-related overlays, e.g. Street lightning overlay or Street parking overlay when clicking on that way.

@westnordost
Copy link
Member

This is a feature, not a bug.

@westnordost westnordost closed this as not planned Won't fix, can't repro, duplicate, stale Oct 18, 2022
@mnalis
Copy link
Member Author

mnalis commented Oct 19, 2022

Oh, OK. It confused me a lot, but I guess it can be documented in FAQ. But how is one supposed to handle that case with StreetComplete only (i.e. fix the overlay on that street)? Or is that use case (editing roads on which someone has left a Note) not (currently) supported by SC?

Because there does not seem to be a way to actually close the note as far as I can tell. (I wanted to close the note so that I presumably could progress to be able to use Sidewalk Overlay on that street).

Clicking hide does remove the note marker from the map, but clicking on the street in overlay brings up the Note again anyway. If one happens to click hide again, the StreetComplete crashes (I've sent the crash report email about it now, so I guess you have it recorded somewhere so you don't need me to open another issue for that crash?)

Maybe the Overlay editing may be allowed if the SC user has manually hidden the Note?

@westnordost
Copy link
Member

Well that's how the notes work. They block all quests until they are solved, because something is amiss that must be solved first (outside of the app). And hence, also all editing with overlays.

However, thank you for the hint regarding the crash, I'll have a look.

@mnalis
Copy link
Member Author

mnalis commented Oct 19, 2022

(added a FAQ entry)

@westnordost
Copy link
Member

I added a fix now. It will not crash anymore. Instead of "No, hide" if it is already hidden (as a quest), just "No" is displayed, which just closes the form. (Hiding has no effect on it being shown in overlays, only it being shown as a quest.)

92b2734

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