Skip to content

Add support for man_made=planter#7558

Closed
UgnilJoZ wants to merge 1 commit intoopenstreetmap:developfrom
UgnilJoZ:add-planter
Closed

Add support for man_made=planter#7558
UgnilJoZ wants to merge 1 commit intoopenstreetmap:developfrom
UgnilJoZ:add-planter

Conversation

@UgnilJoZ
Copy link
Copy Markdown

Good morning,

I felt like man_made=planter was missing in iD, so I added a preset.
Did I do everything right or is there additional work to do?

Cheers!

@UgnilJoZ
Copy link
Copy Markdown
Author

If my request is invalid for some reasons, just tell me :)

@quincylvania
Copy link
Copy Markdown
Collaborator

@UgnilJoZ I'm hesitating because I'm afraid this tag is too similar to landuse=flowerbed. A planter is essentially a raised flowerbed, no?

@quincylvania quincylvania added considering Not Actionable - still considering if this is something we want preset An issue with an OpenStreetMap preset or tag labels May 13, 2020
@UgnilJoZ
Copy link
Copy Markdown
Author

Planters are indeed mostly elevated - but they contain not only flowers, but also shrubs and small trees. There are many planters I wouldn't tag as flowerbed.

@quincylvania
Copy link
Copy Markdown
Collaborator

@UgnilJoZ Closing this since we actually moved iD's preset data to a new repository. I suggest opening an issue there for discussion if you'd still like this to be supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

considering Not Actionable - still considering if this is something we want preset An issue with an OpenStreetMap preset or tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants