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

Automatic building tag is not obvious #4668

Closed
ghost opened this issue Jan 8, 2018 · 6 comments
Closed

Automatic building tag is not obvious #4668

ghost opened this issue Jan 8, 2018 · 6 comments

Comments

@ghost
Copy link

ghost commented Jan 8, 2018

Like in #4639, but now (also) for marketplace.

screenshot_2018-01-08_17-23-48
This should not have been a building.

@bhousel
Copy link
Member

bhousel commented Jan 8, 2018

I think it was obvious in #4639, and I think it's still obvious here.

@bhousel bhousel closed this as completed Jan 8, 2018
@ghost
Copy link
Author

ghost commented Jan 8, 2018

Sorry to nag, but apparently it's not so obvious for the user who made the mistake. This could still be improved, although I hate to admit that I don't know how either.

@ghost
Copy link
Author

ghost commented Jan 8, 2018

FWIW I think marketplaces are an example of where it should definitely default to no building. Those are very often in open air.

@althio
Copy link
Contributor

althio commented Jan 8, 2018

@M1dgard

FWIW I think marketplaces are an example of where it should definitely default to no building. Those are very often in open air.

I agree.
Tag:amenity=marketplace reads

  • for both of them (indoor and outdoor)
  • as an area along the outline of the marketplace
  • [add] building=yes - if it is a market hall

@althio
Copy link
Contributor

althio commented Jan 18, 2018

this is related to issue #4559 PR #4568

@M1dgard do you think it can be closed?

@bhousel
Copy link
Member

bhousel commented Jan 18, 2018

this is related to issue #4559 PR #4568
@M1dgard do you think it can be closed?

Yes, sorry, this was already done and is closed (thank you for linking to the PR - I should have done that originally!)

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

No branches or pull requests

2 participants