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

Make the preset for shop=yes unsearchable #1414

Open
matkoniecz opened this issue Dec 24, 2024 · 2 comments
Open

Make the preset for shop=yes unsearchable #1414

matkoniecz opened this issue Dec 24, 2024 · 2 comments

Comments

@matkoniecz
Copy link
Contributor

matkoniecz commented Dec 24, 2024

When you search for "shop" first match is shop=yes without even warning that it is an unspecific value, to be avoided

see also #1363 which requested opposite

screen02

screen01

@maro-21
Copy link

maro-21 commented Dec 24, 2024

Thanks for the issue. I was planning to report this at some point.

We can have this preset but it should either be hidden or be at the very end (as we have for the "Point" - as a vertex).

The problem is that when you type "shop" into the search engine, shop=yes shows up first. And underneath there are 20 others, in random order and they aren't the most popular shops... The user, after looking through this list, thinks there are only so many and none match what he wants to add so he selects shop=yes. When I started my adventure with OSM and iD this is what I thought - that these are all possible shops to choose from.

@tordans
Copy link
Collaborator

tordans commented Dec 25, 2024

Important

I reverted this change in #1416

I just merged #1415

This fallback preset was always present (at least 4 years). In the unsearchable shop=yes presets was introduced in 2022.

The discussion in #399 (comment) suggested that what I just merged in #1415 is a bad idea (I am re-reading that thread after merging it).

However, we now have the convention to use the "unspecified type" wording for various presets of unspecified type. And to me that is clear.

The search looked good to me as well since the matchScore makes this preset not show up really.

Please test the preview in that PR to see if you can find cases where this would be an unwanted change.

One other (or additional?) option we have is to add a ? to the Shop Icon just like we did with the Curb preset.

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

3 participants