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

Model Shop Preset Added #7779

Merged
merged 2 commits into from
Jul 14, 2020

Conversation

nisargshh
Copy link
Contributor

Added a model shop present when adding for a point or area. Has the same icon as Hobby.

@Adamant36
Copy link

Its questionable that manga should be a term. This tag is specifically for model shops. Not other types of shops that might sell models as a side thing. Manga related toys aren't really models as defined by the tag anyway. Same goes for figurines. There's actually shop=collector for that type of stuff already. Although, I don't think its supported in iD Editor yet. Anyway, this tag is purely confined to scale models.

@nisargshh
Copy link
Contributor Author

Could give some examples of tags that would be better suited for model shop?

@quincylvania quincylvania added the preset An issue with an OpenStreetMap preset or tag label Jul 12, 2020
@Adamant36
Copy link

I can't think of any off the top of my head. Maybe look at some locations on the map and see if there are any similarities in their names or something.

@nisargshh
Copy link
Contributor Author

I edited the tags and the icon. @quincylvania Was this a relevant change? If not would going back be relevant or would it face the issue of what @Adamant36 was saying.

@Adamant36
Copy link

Adamant36 commented Jul 14, 2020

I still don't think the terms work. Model shops don't really sell games and people who put together models don't consider them toys. Terms should be relevant to what people are going to be searching for and I don't think either of those are. Personally, I wouldn't do a search for either in iD Editor when adding a model shop, but people searching for a games store preset will just go with shop=model when it comes up because that's what searching for "games" comes up with. Same goes for crafts. What's wrong with just going with something like "model shop" for the preset and calling it at that? I think that would be fine. There isn't really any more specific synonyms for it that work IMO and there really doesn't have to be as long as people can find the preset when adding model shops.

BTW, I'm just asking and it's only general thoughts, but the Wiki is pretty clear that this should only be used for a specific thing and for instance the shop=shoes preset doesn't come up when you search for "clothes" in iD Editor when you search for it. Even though clothing store and a shoe store are more related then a gaming store and a model shop. Plus, if it's that un-clear where and how this tag should be used maybe there shouldn't be a preset for it at this point. At least until it's discussed and what it actually applies to is better hashed out.

@quincylvania quincylvania merged commit c364a18 into openstreetmap:develop Jul 14, 2020
@quincylvania quincylvania added this to the 2.18.0 milestone Jul 14, 2020
@quincylvania
Copy link
Collaborator

Thanks a lot, @nisargshh, welcome to the project!

@Adamant36 I know you're trying to be helpful, but it feels discouraging when you go into that much detail critiquing a minor aspect of someone else's work. The terms don't have to be perfect; they aren't shown in the UI and it's easy to update them later. The important thing is supporting new contributors.

@Adamant36
Copy link

@quincylvania sorry if it came off that way. It wasn't how I intended it to.

@quincylvania
Copy link
Collaborator

@Adamant36 Cool, no worries.

@boothym
Copy link
Contributor

boothym commented Jul 15, 2020

@nisargshh didn't notice this PR until now - thanks for adding the preset! 😄

@quincylvania and thanks for updating the terms, those look good

@nisargshh
Copy link
Contributor Author

@boothym Thanks, looking to do some more contributions.

@nisargshh nisargshh deleted the model-shop-preset branch July 15, 2020 22:05
@boothym
Copy link
Contributor

boothym commented Jul 16, 2020

Actually, it would be good if shop=craft had the term "hobby" as well because that can also be called a hobby shop.

https://github.com/openstreetmap/iD/blob/72a99673a0328ba4f44c58dd110797cc733f7007/data/presets/presets/shop/craft.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preset An issue with an OpenStreetMap preset or tag
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants