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

ask the surface of roads with surface=cobblestone because it is generic #2237

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

westnordost
Copy link
Member

@matkoniecz what do you think? I was thinking to make a separate quest out of this, but I think this is easier and has no real downsides.

@westnordost
Copy link
Member Author

So, surface=cobblestone could be natural (unhewn) cobblestone or sett. In most cases it is sett. I was thinking to present these two options in a separate quest, plus a "show more" button that shows the rest, in case it has been tagged wrongly. But on the other hand, we had this discussion with DetailRoadSurface (paved/unpaved) and the solution was to just put everything into the same quest type.

@matkoniecz
Copy link
Member

The only downside that I see is that in a separate quest people are less likely to be confused and miss sett or unhewn_cobblestone option.

But probably it should not be a big problem, right?

@westnordost
Copy link
Member Author

I think not. I'll merge it then.

@westnordost westnordost merged commit e3b42b1 into master Nov 4, 2020
@westnordost westnordost deleted the define-cobblestone branch November 4, 2020 19:48
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 this pull request may close these issues.

None yet

2 participants