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

Last answer shortcut button for only one streetside #4503

Closed
Henry00572 opened this issue Oct 14, 2022 · 6 comments · Fixed by #4524
Closed

Last answer shortcut button for only one streetside #4503

Henry00572 opened this issue Oct 14, 2022 · 6 comments · Fixed by #4524

Comments

@Henry00572
Copy link

Use case
In the quest for sidewalk surfaces (for example) the previous answer is displayed as a shortcut in the bottom left of the quest popup. However, when the quest only asks for one side (e.g. quest for sidewalk:right:smoothness becaue sidewalk=right is already set), this shortcut doesn't show up.

Other examples include the cycleway and shoulder quests on highways with oneway=yes.

Proposed Solution
It might just be a matter of changing one if-statement, but I'm not sure where the relevant code is. (Is it https://github.com/streetcomplete/StreetComplete/blob/00f00f60b192bda332dae5249b5fdcd099148f29/app/src/main/java/de/westnordost/streetcomplete/view/controller/StreetSideSelectWithLastAnswerButtonViewController.kt?)

@westnordost
Copy link
Member

No. That would save you just one tap while burning its usefulness when you come back to a road with sidewalks on both sides.

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

Hm, on the other hand it works this way for overlays right now, so it would be consistent.

I remember now another reason why this was not done: Only one last-answer is saved. If you save it for just one side, and then open the quest for a road that has two sides, applying that last answer would not even fill the form completely (-> OK button does not appear), which doesn't really help the user. Fiddling about something like either saving two/three last-answers and checking which side... etc... is too complex.

For me the question is right now if for overlays the behavior should be the same as for quests, i.e. the selection is only saved if it specified for both.

@Henry00572
Copy link
Author

I understand why I wasn't done. The ideal solution I imagined would have saved multiple last-answers and showed the one fitting to the quest, but you have to judge whether this is worth the effort for saving one tap. For the user it would also help if the last-answer to a twosided quest was partially shown for the next onesided quest as well, with only one side, but again this might be too complex.

I think quests and overlays should be consistent in general. As far as I know the current overlays only apply to properties of the highway itself or of both sides, but never of one side only, so the problem only arises when the user saves a partially filled in quest and in this case the partial answer might as well be displayed for the next quest.

@westnordost
Copy link
Member

In quests, you cannot answer partially (e.g. only on one side). With overlays however, you can.

@Henry00572
Copy link
Author

That's reasonable.

For me the question is right now if for overlays the behavior should be the same as for quests, i.e. the selection is only saved if it specified for both.

I think for overlays it's especially useful if a partial answer is also saved and can be accessed by the shortcut, because most streets are split in many parts and if you answer partially for one of them, you will often give the same partial answer for the remaining parts.

@westnordost
Copy link
Member

Well, ... then I leave everything as-is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants