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

Don't suggest values from Taginfo for name:xx tags #10287

Open
Dimitar5555 opened this issue Jun 21, 2024 · 6 comments · May be fixed by #10521
Open

Don't suggest values from Taginfo for name:xx tags #10287

Dimitar5555 opened this issue Jun 21, 2024 · 6 comments · May be fixed by #10521
Labels
good first issue Best for first-time contributors. No experience necessary!

Comments

@Dimitar5555
Copy link
Contributor

URL

No response

How to reproduce the issue?

  1. Create an element
  2. Go to it's tags
  3. Add name:xx
  4. Switch to the value box
  5. Start typing
  6. (notice the suggestions below)

Screenshot(s) or anything else?

image

Which deployed environments do you see the issue in?

Released version at openstreetmap.org/edit

What version numbers does this issue effect?

2.29.0

Which browsers are you seeing this problem on?

Firefox

@1ec5
Copy link
Collaborator

1ec5 commented Jun 22, 2024

Related discussion: #7485.

There’s a hard-coded list of keys that don’t get taginfo suggestions. It could be supplemented by a regular expression for keys starting with name:.

_popularKeys = {
// manually exclude some keys – #5377, #7485
postal_code: true,
full_name: true,
loc_name: true,
reg_name: true,
short_name: true,
sorting_name: true,
artist_name: true,
nat_name: true,
long_name: true,
via: true,
'bridge:name': true
};

@1ec5 1ec5 added the good first issue Best for first-time contributors. No experience necessary! label Oct 15, 2024
@draunger
Copy link

draunger commented Oct 17, 2024

@1ec5 @Dimitar5555 can I start working on it if no one is doing it??

@1ec5
Copy link
Collaborator

1ec5 commented Oct 17, 2024

I don’t think anyone has submitted a pull request about this issue so far. (You’d see a link to it somewhere on this page.) Feel free to submit a PR once you’ve got something that you’d like feedback about, and we’ll try to respond when we’re able to.

@1ec5
Copy link
Collaborator

1ec5 commented Oct 17, 2024

By the way, I noted above that we’re hard-coding a list of “popular” keys to ignore, but there’s a large, technically unbounded number of possible name:* keys. So rather than trying to enumerate all of them, consider looking at where this _popularKeys variable is used and adjusting the logic there.

@draunger
Copy link

@1ec5 ok sir

@draunger
Copy link

draunger commented Nov 1, 2024

hey @1ec5 i have done this issue and here the PR of this issue #10521 you can review that PR as soon as possible.

@1ec5 1ec5 linked a pull request Nov 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Best for first-time contributors. No experience necessary!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants